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

[receiver/solacereceiver]: Updated the format for generated metrics and included new Solace receiver metric attribute #34541

Merged
merged 11 commits into from
Aug 13, 2024

Conversation

oodigie
Copy link
Contributor

@oodigie oodigie commented Aug 8, 2024

Description: Updated the format for generated metrics. Included a receiver_name attribute that identifies the Solace receiver that generated the metrics.

Link to tracking Issue: N/A

Testing: Updated unit tests to accommodate passing of attributes for metrics

Documentation: N/A

@djaglowski
Copy link
Member

Seems there's still an error to resolve: Generated code is out of date, please run "make generate" and commit the changes in this PR.

@oodigie
Copy link
Contributor Author

oodigie commented Aug 9, 2024

Thanks @djaglowski, I have ran make generate and updated the PR. Should pass now.

@oodigie
Copy link
Contributor Author

oodigie commented Aug 9, 2024

Sorry I see some tests errors, I will fix them now.

@djaglowski djaglowski merged commit f1073ef into open-telemetry:main Aug 13, 2024
154 checks passed
@github-actions github-actions bot added this to the next release milestone Aug 13, 2024
@oodigie
Copy link
Contributor Author

oodigie commented Aug 13, 2024

Thanks @djaglowski!

f7o pushed a commit to f7o/opentelemetry-collector-contrib that referenced this pull request Sep 12, 2024
…nd included new Solace receiver metric attribute (open-telemetry#34541)

**Description:** Updated the format for generated metrics. Included a
`receiver_name` attribute that identifies the Solace receiver that
generated the metrics.

**Link to tracking Issue:** N/A

**Testing:** Updated unit tests to accommodate passing of attributes for
metrics

**Documentation:** N/A
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