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] migrate metrics to mdatagen #33514

Conversation

codeboten
Copy link
Contributor

Added metadata.yaml file to allow for generation of internal metrics. Note the stability section of that file are not used anywhere, but mdatagen requires it.

Added metadata.yaml file to allow for generation of internal metrics.

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@codeboten codeboten force-pushed the codeboten/mdatagen-fileconsumer-metrics-2 branch from b1a0354 to b958ee1 Compare June 19, 2024 22:32
@codeboten codeboten merged commit d065d41 into open-telemetry:main Jun 19, 2024
154 checks passed
@codeboten codeboten deleted the codeboten/mdatagen-fileconsumer-metrics-2 branch June 19, 2024 22:55
@github-actions github-actions bot added this to the next release milestone Jun 19, 2024
codeboten pushed a commit that referenced this pull request Jun 20, 2024
`main` is currently failing the [`checks`
action](https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/9588994799/job/26442043847),
at the `Gen CODEOWNERS` step. This applies the required change.

This was broken by
#33514,
which added a `metadata.yaml` file for the `pkg/stanza/fileconsumer`
package, but didn't add an entry to the CODEOWNERS file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants