Skip to content

mgr/cephadm: allow custom images for grafana, prometheus, alertmanager and node_exporter#35106

Merged
sebastian-philipp merged 6 commits intoceph:masterfrom
p-se:wip-pse-cephadm-custom-monitoring-images
May 27, 2020
Merged

mgr/cephadm: allow custom images for grafana, prometheus, alertmanager and node_exporter#35106
sebastian-philipp merged 6 commits intoceph:masterfrom
p-se:wip-pse-cephadm-custom-monitoring-images

Conversation

@p-se
Copy link
Contributor

@p-se p-se commented May 18, 2020

Fixes: https://tracker.ceph.com/issues/45463

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

@p-se p-se requested a review from a team as a code owner May 18, 2020 13:34
@p-se
Copy link
Contributor Author

p-se commented May 18, 2020

If that turns out to be the right approach for being able to specify and use custom images, I'd also like to add a documentation update on how to use it.

@sebastian-philipp sebastian-philipp requested a review from tserong May 18, 2020 13:38
@tchaikov
Copy link
Contributor

@p-se needs rebase

@p-se p-se force-pushed the wip-pse-cephadm-custom-monitoring-images branch from 0f6cb1f to 8442400 Compare May 25, 2020 13:48
@p-se p-se removed the needs-rebase label May 25, 2020
p-se added 6 commits May 26, 2020 12:49
Namely,
- Grafana
- Prometheus
- Alertmanager and
- Node exporter.

Fixes: https://tracker.ceph.com/issues/45463

Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
@p-se p-se force-pushed the wip-pse-cephadm-custom-monitoring-images branch from 8442400 to 7ca3b47 Compare May 26, 2020 10:49
@sebastian-philipp
Copy link
Contributor

reconfig=False,
osd_uuid_map=None) -> str:
osd_uuid_map: Optional[Dict[str, Any]] = None,
redeploy=False,
Copy link
Contributor

Choose a reason for hiding this comment

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

Sigh. this PR introduced an unused redeploy parameter.

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