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

Add skeleton for new rabbitmq exporter component #31800

Merged
merged 8 commits into from Mar 26, 2024

Conversation

swar8080
Copy link
Contributor

@swar8080 swar8080 commented Mar 18, 2024

Description:
Sets-up the configuration format and common component boilerplate for the rabbitmq exporter

Implementation will be in other pull requests

Link to tracking Issue: 28891

Testing: Standard initial unit tests

Documentation: Created README for the component

@swar8080 swar8080 requested a review from a team as a code owner March 18, 2024 02:00
@swar8080 swar8080 requested a review from codeboten March 18, 2024 02:00
@atoulme
Copy link
Contributor

atoulme commented Mar 18, 2024

Very close. Just need to add a line to .github/CODEOWNERS and update the dependencies in go.mod with .github/workflows/scripts/check-collector-module-version.sh

@swar8080 swar8080 requested a review from atoulme March 19, 2024 01:10
@atoulme atoulme added the Accepted Component New component has been sponsored label Mar 20, 2024
@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Mar 21, 2024
@swar8080
Copy link
Contributor Author

Got the pipeline to pass - thanks @atoulme

@djaglowski djaglowski merged commit 9adb91e into open-telemetry:main Mar 26, 2024
147 checks passed
@github-actions github-actions bot added this to the next release milestone Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Component New component has been sponsored ready to merge Code review completed; ready to merge by maintainers reports/distributions/contrib.yaml
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants