Skip to content

mgr/prometheus: automatically discover RBD pools for stats gathering#35603

Merged
trociny merged 1 commit intoceph:masterfrom
dillaman:wip-46034
Jun 18, 2020
Merged

mgr/prometheus: automatically discover RBD pools for stats gathering#35603
trociny merged 1 commit intoceph:masterfrom
dillaman:wip-46034

Conversation

@dillaman
Copy link
Copy Markdown

@dillaman dillaman commented Jun 16, 2020

To simplify configuration, the existing 'rbd_stats_pools' config option
now accepts '*' as a wildcard for the pool and/or namespace portion of
the <pool>[/<namespace>] spec.

Fixes: https://tracker.ceph.com/issues/46034
Signed-off-by: Jason Dillaman dillaman@redhat.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@trociny
Copy link
Copy Markdown
Contributor

trociny commented Jun 16, 2020

Just thinking. What if instead of adding the additional config option (rbd_stats_all_pools) we just add a special value (e.g. *) for rbd_stats_pools?

@dillaman
Copy link
Copy Markdown
Author

What if instead of adding the additional config option (rbd_stats_all_pools) we just add a special value (e.g. *) for rbd_stats_pools?

Yeah, we could do that.

@dillaman dillaman force-pushed the wip-46034 branch 2 times, most recently from 3747109 to ff0c87b Compare June 16, 2020 18:25
Copy link
Copy Markdown
Contributor

@trociny trociny left a comment

Choose a reason for hiding this comment

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

@dillaman need to update the commit log message. Otherwise LGTM.

To simplify configuration, the existing 'rbd_stats_pools' config option
now accepts '*' as a wildcard for the pool and/or namespace portion of
the <pool>[/<namespace>] spec.

Fixes: https://tracker.ceph.com/issues/46034
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
@dillaman
Copy link
Copy Markdown
Author

Commit message updated

@trociny
Copy link
Copy Markdown
Contributor

trociny commented Jun 18, 2020

jenkins test dashboard backend

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.

2 participants