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

Remove PData prefix/suffix from prometheus internal struct names #9767

Merged
merged 2 commits into from May 6, 2022

Conversation

bogdandrutu
Copy link
Member

Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@bogdandrutu bogdandrutu added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 5, 2022
@bogdandrutu bogdandrutu requested review from a team, Aneurysm9 and dashpole as code owners May 5, 2022 23:31
Co-authored-by: David Ashpole <dashpole@google.com>
@codeboten codeboten changed the title Remove PData prefix from prometheus internal struct names Remove PData prefix/suffix from prometheus internal struct names May 6, 2022
@codeboten codeboten added the ready to merge Code review completed; ready to merge by maintainers label May 6, 2022
@bogdandrutu bogdandrutu merged commit 68de32f into open-telemetry:main May 6, 2022
@bogdandrutu bogdandrutu deleted the rmpdata branch May 6, 2022 15:59
djaglowski pushed a commit to djaglowski/opentelemetry-collector-contrib that referenced this pull request May 10, 2022
…n-telemetry#9767)

* Remove PData prefix from prometheus internal struct names

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

* Update receiver/prometheusreceiver/internal/otlp_metricfamily.go

Co-authored-by: David Ashpole <dashpole@google.com>

Co-authored-by: Alex Boten <alex@boten.ca>
Co-authored-by: David Ashpole <dashpole@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants