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

cephadm: Add ability to deploy grafana container #32491

Merged
merged 1 commit into from Jan 14, 2020
Merged

cephadm: Add ability to deploy grafana container #32491

merged 1 commit into from Jan 14, 2020

Conversation

pcuzner
Copy link
Contributor

@pcuzner pcuzner commented Jan 6, 2020

This patch adds initial support for deploying grafana. It uses a specific container image that has the grafana plugins and ceph-dashboards pre-installed, and relies on the caller passing appropriate SSL 'files' for the grafana instance to run under https (example in the samples dir)

Signed-off-by: Paul Cuzner pcuzner@redhat.com

@pcuzner pcuzner requested a review from a team as a code owner January 6, 2020 01:41
@pcuzner pcuzner requested a review from liewegas January 6, 2020 01:41
@pcuzner pcuzner added the cephadm label Jan 6, 2020
Copy link
Contributor

@p-se p-se left a comment

Choose a reason for hiding this comment

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

lgtm

This patch adds initial support for deploying grafana. It uses a specific container image that has the grafana
plugins and ceph-dashboards pre-installed, and relies on
the caller passing appropriate SSL 'files' for the grafana
instance to run under https (example in the samples dir)

Signed-off-by: Paul Cuzner <pcuzner@redhat.com>
@sebastian-philipp
Copy link
Contributor

@mgfritch do you know, how we can integrate this into the qa tests?

@sebastian-philipp sebastian-philipp added the wip-swagner-testing My Teuthology tests label Jan 13, 2020
@sebastian-philipp sebastian-philipp merged commit d4c4272 into ceph:master Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants