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

daemon-base: Add ceph-grafana-dashboards #1290

Closed
wants to merge 1 commit into from
Closed

Conversation

zmc
Copy link
Member

@zmc zmc commented Jan 11, 2019

This is needed for full functionality of the mgr dashboard module.

Signed-off-by: Zack Cerza zack@redhat.com

This is needed for full functionality of the mgr dashboard module.

Signed-off-by: Zack Cerza <zack@redhat.com>
@zmc zmc requested a review from leseb January 11, 2019 23:29
@zmc
Copy link
Member Author

zmc commented Jan 11, 2019

Please let me know if I made this change in the right place; as far as I know the package will only exist from nautilus on.

Copy link
Contributor

@ErwanAliasr1 ErwanAliasr1 left a comment

Choose a reason for hiding this comment

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

If you put that package in src/daemon-base/ that means that all releases will have to provide it. Including luminous/mimic/master.

Will you provide this package for each of this releases ?

If you only provide it to master then you'll have to override CEPH_BASE_PACKAGES in luminous & mimic

@dsavineau
Copy link
Contributor

@zmc any news on this one ?

fmount referenced this pull request May 29, 2019
This is needed to install the ceph dashboard json files in
`/etc/grafana/dashboards/ceph-dashboard/`.

Closes: ceph/ceph-ansible#4026

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
@guits
Copy link
Collaborator

guits commented May 29, 2019

addressed in #1384

@guits guits closed this May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants