[chore] add codeowners to metadata (open-telemetry#24404)

**Description:** This is a minute set of changes, adding codeowners to every component and additional folders as designed by the CODEOWNERS file. Notable changes are renames: extension/encodingextension/metadata.yml -> extension/encodingextension/metadata.yaml cmd/mdatagen/metadata.yaml -> cmd/mdatagen/metadata-sample.yaml **Link to tracking Issue:** This is step 2 of open-telemetry#23367 **Testing:** I have used a tool I created to regenerate CODEOWNERS from this file.
ClickHouse · Jul 24, 2023 · 8a4348c · 8a4348c
1 parent d95d199
commit 8a4348c
Show file tree

Hide file tree

Showing 412 changed files with 839 additions and 156 deletions.
diff --git a/cmd/configschema/metadata.yaml b/cmd/configschema/metadata.yaml
@@ -0,0 +1,6 @@
+type: configschema
+
+status:
+ class: cmd
+ codeowners:
+ active: [mx-psi, dmitryax, pmcollins]
diff --git a/cmd/mdatagen/doc.go b/cmd/mdatagen/doc.go
@@ -2,6 +2,6 @@
 // SPDX-License-Identifier: Apache-2.0
 
 // Generate a test metrics builder from a sample metrics set covering all configuration options.
-//go:generate mdatagen metadata.yaml
+//go:generate mdatagen metadata-sample.yaml
 
 package main
diff --git a/cmd/mdatagen/loader_test.go b/cmd/mdatagen/loader_test.go
@@ -18,7 +18,7 @@ func Test_loadMetadata(t *testing.T) {
  wantErr string
  }{
  {
- name: "metadata.yaml",
+ name: "metadata-sample.yaml",
  want: metadata{
  Type: "file",
  SemConvVersion: "1.9.0",

diff --git a/cmd/mdatagen/main.go b/cmd/mdatagen/main.go
@@ -61,11 +61,13 @@ func run(ymlPath string) error {
  return err
  }
 
- if err = inlineReplace(
- filepath.Join(tmplDir, "readme.md.tmpl"),
- filepath.Join(ymlDir, "README.md"),
- md, statusStart, statusEnd); err != nil {
- return err
+ if _, err = os.Stat(filepath.Join(ymlDir, "README.md")); err == nil {
+ if err = inlineReplace(
+ filepath.Join(tmplDir, "readme.md.tmpl"),
+ filepath.Join(ymlDir, "README.md"),
+ md, statusStart, statusEnd); err != nil {
+ return err
+ }
  }
  }
  if len(md.Metrics) == 0 && len(md.ResourceAttributes) == 0 {
@@ -143,7 +145,11 @@ func templatize(tmplFile string, md metadata) *template.Template {
  "userLinks": func(elems []string) []string {
  result := make([]string, len(elems))
  for i, elem := range elems {
- result[i] = fmt.Sprintf("[@%s](https://www.github.com/%s)", elem, elem)
+ if elem == "open-telemetry/collector-approvers" {
+ result[i] = "[@open-telemetry/collector-approvers](https://github.com/orgs/open-telemetry/teams/collector-approvers)"
+ } else {
+ result[i] = fmt.Sprintf("[@%s](https://www.github.com/%s)", elem, elem)
+ }
  }
  return result
  },

diff --git a/cmd/mdatagen/metadata-sample.yaml b/cmd/mdatagen/metadata-sample.yaml
@@ -0,0 +1,105 @@
+# Sample metric metadata file with all available configurations.
+
+type: file
+
+sem_conv_version: 1.9.0
+
+status:
+ class: receiver
+ stability:
+ development: [logs]
+ beta: [traces]
+ stable: [metrics]
+ distributions: [contrib]
+ warnings:
+ - Any additional information that should be brought to the consumer's attention
+
+resource_attributes:
+ string.resource.attr:
+ description: Resource attribute with any string value.
+ type: string
+ enabled: true
+
+ string.enum.resource.attr:
+ description: Resource attribute with a known set of string values.
+ type: string
+ enum: [one, two]
+ enabled: true
+
+ optional.resource.attr:
+ description: Explicitly disabled ResourceAttribute.
+ type: string
+ enabled: false
+
+ slice.resource.attr:
+ description: Resource attribute with a slice value.
+ type: slice
+ enabled: true
+
+ map.resource.attr:
+ description: Resource attribute with a map value.
+ type: map 
+ enabled: true
+
+attributes:
+ string_attr:
+ description: Attribute with any string value.
+ type: string
+
+ overridden_int_attr:
+ name_override: state
+ description: Integer attribute with overridden name.
+ type: int
+
+ enum_attr:
+ description: Attribute with a known set of string values.
+ type: string
+ enum: [red, green, blue]
+
+ boolean_attr:
+ description: Attribute with a boolean value.
+ type: bool
+
+ slice_attr:
+ description: Attribute with a slice value.
+ type: slice
+
+ map_attr:
+ description: Attribute with a map value.
+ type: map
+
+metrics:
+ default.metric:
+ enabled: true
+ description: Monotonic cumulative sum int metric enabled by default.
+ extended_documentation: The metric will be become optional soon.
+ unit: s
+ sum:
+ value_type: int
+ monotonic: true
+ aggregation: cumulative
+ attributes: [string_attr, overridden_int_attr, enum_attr, slice_attr, map_attr]
+ warnings:
+ if_enabled_not_set: This metric will be disabled by default soon.
+
+ optional.metric:
+ enabled: false
+ description: "[DEPRECATED] Gauge double metric disabled by default."
+ unit: 1
+ gauge:
+ value_type: double
+ attributes: [string_attr, boolean_attr]
+ warnings:
+ if_configured: This metric is deprecated and will be removed soon.
+
+ default.metric.to_be_removed:
+ enabled: true
+ description: "[DEPRECATED] Non-monotonic delta sum double metric enabled by default."
+ extended_documentation: The metric will be will be removed soon.
+ unit: s
+ sum:
+ value_type: double
+ monotonic: false
+ aggregation: delta
+ warnings:
+ if_enabled: This metric is deprecated and will be removed soon.
diff --git a/cmd/mdatagen/metadata.yaml b/cmd/mdatagen/metadata.yaml
@@ -1,105 +1,6 @@
-# Sample metric metadata file with all available configurations.
-
-type: file
-
-sem_conv_version: 1.9.0
+type: mdatagen
 
 status:
- class: receiver
- stability:
- development: [logs]
- beta: [traces]
- stable: [metrics]
- distributions: [contrib]
- warnings:
- - Any additional information that should be brought to the consumer's attention
-
-resource_attributes:
- string.resource.attr:
- description: Resource attribute with any string value.
- type: string
- enabled: true
-
- string.enum.resource.attr:
- description: Resource attribute with a known set of string values.
- type: string
- enum: [one, two]
- enabled: true
-
- optional.resource.attr:
- description: Explicitly disabled ResourceAttribute.
- type: string
- enabled: false
-
- slice.resource.attr:
- description: Resource attribute with a slice value.
- type: slice
- enabled: true
-
- map.resource.attr:
- description: Resource attribute with a map value.
- type: map 
- enabled: true
-
-attributes:
- string_attr:
- description: Attribute with any string value.
- type: string
-
- overridden_int_attr:
- name_override: state
- description: Integer attribute with overridden name.
- type: int
-
- enum_attr:
- description: Attribute with a known set of string values.
- type: string
- enum: [red, green, blue]
-
- boolean_attr:
- description: Attribute with a boolean value.
- type: bool
-
- slice_attr:
- description: Attribute with a slice value.
- type: slice
-
- map_attr:
- description: Attribute with a map value.
- type: map
-
-metrics:
- default.metric:
- enabled: true
- description: Monotonic cumulative sum int metric enabled by default.
- extended_documentation: The metric will be become optional soon.
- unit: s
- sum:
- value_type: int
- monotonic: true
- aggregation: cumulative
- attributes: [string_attr, overridden_int_attr, enum_attr, slice_attr, map_attr]
- warnings:
- if_enabled_not_set: This metric will be disabled by default soon.
-
- optional.metric:
- enabled: false
- description: "[DEPRECATED] Gauge double metric disabled by default."
- unit: 1
- gauge:
- value_type: double
- attributes: [string_attr, boolean_attr]
- warnings:
- if_configured: This metric is deprecated and will be removed soon.
-
- default.metric.to_be_removed:
- enabled: true
- description: "[DEPRECATED] Non-monotonic delta sum double metric enabled by default."
- extended_documentation: The metric will be will be removed soon.
- unit: s
- sum:
- value_type: double
- monotonic: false
- aggregation: delta
- warnings:
- if_enabled: This metric is deprecated and will be removed soon.
+ class: cmd
+ codeowners:
+ active: [dmitryax]
diff --git a/cmd/mdatagen/templates/status.go.tmpl b/cmd/mdatagen/templates/status.go.tmpl
@@ -2,9 +2,11 @@
 
 package {{ .Package }}
 
+{{- if .Status.Stability }}
 import (
  "go.opentelemetry.io/collector/component"
 )
+{{- end }}
 
 const (
  Type = "{{ .Type }}"

diff --git a/cmd/mdatagen/validate.go b/cmd/mdatagen/validate.go
@@ -48,8 +48,10 @@ func (md *metadata) validateStatus() error {
  if err := md.Status.validateClass(); err != nil {
  errs = multierr.Append(errs, err)
  }
- if err := md.Status.validateStability(); err != nil {
- errs = multierr.Append(errs, err)
+ if md.Parent == "" {
+ if err := md.Status.validateStability(); err != nil {
+ errs = multierr.Append(errs, err)
+ }
  }
  return errs
 }

diff --git a/cmd/opampsupervisor/metadata.yaml b/cmd/opampsupervisor/metadata.yaml
@@ -0,0 +1,6 @@
+type: opampsupervisor
+
+status:
+ class: cmd
+ codeowners:
+ active: [evan-bradley, atoulme, tigrannajaryan]
diff --git a/cmd/otelcontribcol/metadata.yaml b/cmd/otelcontribcol/metadata.yaml
@@ -0,0 +1,6 @@
+type: otelcontribcol
+
+status:
+ class: cmd
+ codeowners:
+ active: []
diff --git a/cmd/oteltestbedcol/metadata.yaml b/cmd/oteltestbedcol/metadata.yaml
@@ -0,0 +1,6 @@
+type: oteltestbedcol
+
+status:
+ class: cmd
+ codeowners:
+ active: []
diff --git a/cmd/telemetrygen/README.md b/cmd/telemetrygen/README.md
@@ -6,6 +6,7 @@
 | Stability | [alpha]: traces |
 | | [development]: metrics, logs |
 | Issues | ![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Acmd%2Ftelemetrygen%20&label=open&color=orange&logo=opentelemetry) ![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Acmd%2Ftelemetrygen%20&label=closed&color=blue&logo=opentelemetry) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@mx-psi](https://www.github.com/mx-psi), [@codeboten](https://www.github.com/codeboten) |
 
 [alpha]: https://github.com/open-telemetry/opentelemetry-collector#alpha
 [development]: https://github.com/open-telemetry/opentelemetry-collector#development

diff --git a/cmd/telemetrygen/metadata.yaml b/cmd/telemetrygen/metadata.yaml
@@ -5,3 +5,5 @@ status:
  stability:
  alpha: [traces]
  development: [metrics, logs]
+ codeowners:
+ active: [mx-psi, codeboten]
diff --git a/confmap/provider/s3provider/metadata.yaml b/confmap/provider/s3provider/metadata.yaml
@@ -0,0 +1,3 @@
+status:
+ codeowners:
+ active: [Aneurysm9]
diff --git a/connector/countconnector/README.md b/connector/countconnector/README.md
@@ -4,6 +4,7 @@
 | ------------- |-----------|
 | Distributions | [contrib], [sumo] |
 | Issues | ![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aconnector%2Fcount%20&label=open&color=orange&logo=opentelemetry) ![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aconnector%2Fcount%20&label=closed&color=blue&logo=opentelemetry) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@djaglowski](https://www.github.com/djaglowski), [@jpkrohling](https://www.github.com/jpkrohling) |
 
 [development]: https://github.com/open-telemetry/opentelemetry-collector#development
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/connector/countconnector/metadata.yaml b/connector/countconnector/metadata.yaml
@@ -5,3 +5,5 @@ status:
  stability:
  development: [traces_to_metrics, metrics_to_metrics, logs_to_metrics]
  distributions: [contrib, sumo]
+ codeowners:
+ active: [djaglowski, jpkrohling]
diff --git a/connector/routingconnector/README.md b/connector/routingconnector/README.md
@@ -5,6 +5,7 @@
 | ------------- |-----------|
 | Distributions | [contrib] |
 | Issues | ![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aconnector%2Frouting%20&label=open&color=orange&logo=opentelemetry) ![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aconnector%2Frouting%20&label=closed&color=blue&logo=opentelemetry) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@jpkrohling](https://www.github.com/jpkrohling), [@kovrus](https://www.github.com/kovrus), [@mwear](https://www.github.com/mwear) |
 
 [development]: https://github.com/open-telemetry/opentelemetry-collector#development
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/connector/routingconnector/metadata.yaml b/connector/routingconnector/metadata.yaml
@@ -5,3 +5,5 @@ status:
  stability:
  development: [traces_to_traces, metrics_to_metrics, logs_to_logs]
  distributions: [contrib]
+ codeowners:
+ active: [jpkrohling, kovrus, mwear]
diff --git a/connector/servicegraphconnector/README.md b/connector/servicegraphconnector/README.md
@@ -5,6 +5,7 @@
 | ------------- |-----------|
 | Distributions | [contrib], [sumo] |
 | Issues | ![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aconnector%2Fservicegraph%20&label=open&color=orange&logo=opentelemetry) ![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aconnector%2Fservicegraph%20&label=closed&color=blue&logo=opentelemetry) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@jpkrohling](https://www.github.com/jpkrohling), [@mapno](https://www.github.com/mapno) |
 
 [alpha]: https://github.com/open-telemetry/opentelemetry-collector#alpha
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/connector/servicegraphconnector/metadata.yaml b/connector/servicegraphconnector/metadata.yaml
@@ -5,3 +5,5 @@ status:
  stability:
  alpha: [traces_to_metrics]
  distributions: [contrib, sumo]
+ codeowners:
+ active: [jpkrohling, mapno]
diff --git a/connector/spanmetricsconnector/README.md b/connector/spanmetricsconnector/README.md
@@ -5,6 +5,7 @@
 | ------------- |-----------|
 | Distributions | [contrib], [sumo] |
 | Issues | ![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aopen%20label%3Aconnector%2Fspanmetrics%20&label=open&color=orange&logo=opentelemetry) ![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector-contrib?query=is%3Aissue%20is%3Aclosed%20label%3Aconnector%2Fspanmetrics%20&label=closed&color=blue&logo=opentelemetry) |
+| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@albertteoh](https://www.github.com/albertteoh), [@kovrus](https://www.github.com/kovrus) |
 
 [alpha]: https://github.com/open-telemetry/opentelemetry-collector#alpha
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib

diff --git a/connector/spanmetricsconnector/metadata.yaml b/connector/spanmetricsconnector/metadata.yaml
@@ -5,3 +5,5 @@ status:
  stability:
  alpha: [traces_to_metrics]
  distributions: [contrib, sumo]
+ codeowners:
+ active: [albertteoh, kovrus]