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

exporter: applying labels from monitoring section to ceph-exporter #13902

Merged
merged 2 commits into from Mar 13, 2024

Conversation

rkachach
Copy link
Contributor

@rkachach rkachach commented Mar 8, 2024

the labels listed under the 'monitoring' section are currently only being applied to the rook-ceph-mgr ServiceMonitor. This change extends those labels to also include the rook-ceph-exporter ServiceMonitor.

Fixes: #13774

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

the labels listed under the 'monitoring' section are currently only
being applied to the rook-ceph-mgr ServiceMonitor. This change extends
those labels to also include the rook-ceph-exporter ServiceMonitor.

Fixes: rook#13774

Signed-off-by: Redouane Kachach <rkachach@redhat.com>
@rkachach rkachach requested review from travisn and a team March 8, 2024 13:43
Copy link
Member

@travisn travisn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems simple enough to apply the monitoring labels to the exporter service monitor, in addition to the exporter labels.

@rkachach rkachach marked this pull request as ready for review March 12, 2024 15:39
@rkachach rkachach force-pushed the fix_issue_13774 branch 2 times, most recently from be98ecc to 1f7299c Compare March 12, 2024 16:24
Signed-off-by: Redouane Kachach <rkachach@redhat.com>
@travisn travisn merged commit c409a6a into rook:master Mar 13, 2024
51 checks passed
travisn added a commit that referenced this pull request Mar 13, 2024
exporter: applying labels from monitoring section to ceph-exporter (backport #13902)
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.

Monitoring labels aren't applied to rook-ceph-exporter
3 participants