Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[chore] Improve usage/setup of metadata.yaml #12343

Merged
merged 1 commit into from
Feb 13, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions cmd/mdatagen/internal/status.go
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,7 @@ func (s *Status) Validate() error {
if err := s.validateClass(); err != nil {
errs = errors.Join(errs, err)
}

if err := s.Stability.Validate(); err != nil {
errs = errors.Join(errs, err)
}
Expand Down
17 changes: 0 additions & 17 deletions exporter/exporterhelper/internal/metadata/generated_status.go

This file was deleted.

10 changes: 10 additions & 0 deletions pdata/README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,15 @@
# Pipeline data (pdata)

<!-- status autogenerated section -->
| Status | |
| ------------- |-----------|
| Stability | [stable]: traces, metrics, logs |
| Issues | [![Open issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector?query=is%3Aissue%20is%3Aopen%20label%3Apkg%2Fpdata%20&label=open&color=orange&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector/issues?q=is%3Aopen+is%3Aissue+label%3Apkg%2Fpdata) [![Closed issues](https://img.shields.io/github/issues-search/open-telemetry/opentelemetry-collector?query=is%3Aissue%20is%3Aclosed%20label%3Apkg%2Fpdata%20&label=closed&color=blue&logo=opentelemetry)](https://github.com/open-telemetry/opentelemetry-collector/issues?q=is%3Aclosed+is%3Aissue+label%3Apkg%2Fpdata) |
| [Code Owners](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/CONTRIBUTING.md#becoming-a-code-owner) | [@BogdanDrutu](https://www.github.com/BogdanDrutu), [@dmitryax](https://www.github.com/dmitryax) |

[stable]: https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/component-stability.md#stable
<!-- end autogenerated section -->

Pipeline data (pdata) implements data structures that represent telemetry data in-memory. All data received
is converted into this format, travels through the pipeline in this format, and is converted from this format by
exporters when sending.
Expand Down
7 changes: 7 additions & 0 deletions pdata/doc.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
// Copyright The OpenTelemetry Authors
// SPDX-License-Identifier: Apache-2.0

//go:generate mdatagen metadata.yaml

// Package pdata provides the data model definitions for all supported pipeline data.
package pdata // import "go.opentelemetry.io/collector/pdata"
13 changes: 13 additions & 0 deletions pdata/generated_package_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions pdata/metadata.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,11 @@
type: pdata
github_project: open-telemetry/opentelemetry-collector

status:
class: pkg
codeowners:
active:
- BogdanDrutu
- dmitryax
stability:
stable: [traces, metrics, logs]
songy23 marked this conversation as resolved.
Show resolved Hide resolved
9 changes: 0 additions & 9 deletions receiver/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ require (
go.opentelemetry.io/collector/consumer v1.25.0
go.opentelemetry.io/collector/consumer/consumertest v0.119.0
go.opentelemetry.io/collector/pipeline v0.119.0
go.opentelemetry.io/collector/receiver/receivertest v0.119.0
go.opentelemetry.io/otel v1.34.0
go.opentelemetry.io/otel/metric v1.34.0
go.opentelemetry.io/otel/sdk/metric v1.34.0
Expand All @@ -28,11 +27,9 @@ require (
github.com/modern-go/reflect2 v1.0.2 // indirect
github.com/pmezard/go-difflib v1.0.0 // indirect
go.opentelemetry.io/auto/sdk v1.1.0 // indirect
go.opentelemetry.io/collector/consumer/consumererror v0.119.0 // indirect
go.opentelemetry.io/collector/consumer/xconsumer v0.119.0 // indirect
go.opentelemetry.io/collector/pdata v1.25.0 // indirect
go.opentelemetry.io/collector/pdata/pprofile v0.119.0 // indirect
go.opentelemetry.io/collector/receiver/xreceiver v0.119.0 // indirect
go.opentelemetry.io/otel/sdk v1.34.0 // indirect
go.uber.org/multierr v1.11.0 // indirect
go.uber.org/zap v1.27.0 // indirect
Expand Down Expand Up @@ -61,12 +58,6 @@ replace go.opentelemetry.io/collector/consumer/xconsumer => ../consumer/xconsume

replace go.opentelemetry.io/collector/consumer/consumertest => ../consumer/consumertest

replace go.opentelemetry.io/collector/receiver/xreceiver => ./xreceiver

replace go.opentelemetry.io/collector/receiver/receivertest => ./receivertest

retract v0.76.0 // Depends on retracted pdata v1.0.0-rc10 module

replace go.opentelemetry.io/collector/pipeline => ../pipeline

replace go.opentelemetry.io/collector/consumer/consumererror => ../consumer/consumererror

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 1 addition & 2 deletions receiver/receiverhelper/metadata.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ type: receiverhelper
github_project: open-telemetry/opentelemetry-collector

status:
class: receiver
not_component: true
class: pkg
stability:
beta: [traces, metrics, logs]

Expand Down
4 changes: 0 additions & 4 deletions receiver/xreceiver/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,3 @@ replace go.opentelemetry.io/collector/pdata => ../../pdata
replace go.opentelemetry.io/collector/consumer/consumertest => ../../consumer/consumertest

replace go.opentelemetry.io/collector/pipeline => ../../pipeline

replace go.opentelemetry.io/collector/consumer/consumererror => ../../consumer/consumererror

replace go.opentelemetry.io/collector/receiver/receivertest => ../receivertest
2 changes: 1 addition & 1 deletion scraper/metadata.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
type: service
type: scraper
github_project: open-telemetry/opentelemetry-collector

status:
Expand Down
Loading