Skip to content

[chore][receiver/googlecloudpubsubreceiver] sort metadata.yaml entries#43579

Merged
codeboten merged 5 commits into
open-telemetry:mainfrom
paulojmdias:chore/mdata_fix_receiver/googlecloudpubsubreceiver
Oct 17, 2025
Merged

[chore][receiver/googlecloudpubsubreceiver] sort metadata.yaml entries#43579
codeboten merged 5 commits into
open-telemetry:mainfrom
paulojmdias:chore/mdata_fix_receiver/googlecloudpubsubreceiver

Conversation

@paulojmdias
Copy link
Copy Markdown
Member

Description

Sort metadata.yaml file in preparation for sort validation using mdatagen. Context: open-telemetry/opentelemetry-collector#13782

Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
@dmitryax
Copy link
Copy Markdown
Member

@paulojmdias the CI is failing. Please update this

@paulojmdias
Copy link
Copy Markdown
Member Author

@dmitryax updated, it should pass now.

@codeboten
Copy link
Copy Markdown
Contributor

Still some failures

Generated code is out of date, please run "make generate" and commit the changes in this PR.

Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
@paulojmdias
Copy link
Copy Markdown
Member Author

@codeboten fix pushed now 🎉

@codeboten codeboten merged commit eaef7cb into open-telemetry:main Oct 17, 2025
186 checks passed
@paulojmdias paulojmdias deleted the chore/mdata_fix_receiver/googlecloudpubsubreceiver branch October 17, 2025 16:08
@github-actions github-actions Bot added this to the next release milestone Oct 17, 2025
@codeboten
Copy link
Copy Markdown
Contributor

Thanks @paulojmdias!

ChrsMark pushed a commit to ChrsMark/opentelemetry-collector-contrib that referenced this pull request Oct 20, 2025
open-telemetry#43579)

#### Description

Sort metadata.yaml file in preparation for sort validation using
mdatagen. Context:
open-telemetry/opentelemetry-collector#13782

---------

Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
Co-authored-by: Alex Boten <223565+codeboten@users.noreply.github.com>
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