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

Helm chart for cluster provisioning #2109

Closed
maver1ck opened this issue Sep 10, 2018 · 17 comments · Fixed by #7778
Closed

Helm chart for cluster provisioning #2109

maver1ck opened this issue Sep 10, 2018 · 17 comments · Fixed by #7778
Assignees
Labels

Comments

@maver1ck
Copy link

Is this a bug report or feature request?

  • Feature Request

What should the feature do:
I'd like to have helm chart for cluster creating. Both for sc and cluster itself
https://rook.io/docs/rook/master/block.html
https://github.com/rook/rook/blob/master/Documentation/ceph-quickstart.md#create-a-rook-cluster

What is use case behind this feature:
I can have all the parts of the env managed by helm

@stale
Copy link

stale bot commented Dec 9, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Dec 9, 2018
@kfox1111
Copy link

This could be useful.

@stale
Copy link

stale bot commented Mar 16, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Mar 16, 2019
@Bessonov
Copy link

@Stale bot leave it open

@travisn
Copy link
Member

travisn commented Apr 4, 2019

With #2820 merged, we can now plan on adding the cluster spec to the existing rook-ceph helm chart instead of creating a whole new chart. This will be a great simplification since users generally want to manage a single cluster with a single operator.

We can leverage the changes in #1676 and now apply it to the existing helm chart and we will be most of the way there. The cluster crd has changed quite a bit since that PR was opened, so we will also need to update to the latest crd schema.

@stale
Copy link

stale bot commented Jul 3, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jul 3, 2019
@travisn travisn removed the wontfix label Jul 3, 2019
@stale
Copy link

stale bot commented Oct 2, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Oct 2, 2019
@Bessonov
Copy link

Bessonov commented Oct 2, 2019

Activity!

@stale stale bot removed the wontfix label Oct 2, 2019
@stale
Copy link

stale bot commented Dec 31, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Dec 31, 2019
@Bessonov
Copy link

Happy New Year!

@stale
Copy link

stale bot commented Mar 30, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Mar 30, 2020
@stale stale bot removed the wontfix label Mar 30, 2020
@travisn travisn added the ceph main ceph tag label May 26, 2020
@maver1ck
Copy link
Author

maver1ck commented Oct 5, 2020

2 years later ? :)

@galexrt galexrt self-assigned this Oct 5, 2020
@galexrt
Copy link
Member

galexrt commented Oct 5, 2020

My plan would be to just have it be a "wrapper" for the CephCluster object in the values.yaml and keeping it separate to the rook-ceph-operator Chart so we can rely on the operator like that as a dependencies.

@maver1ck
Copy link
Author

maver1ck commented Oct 5, 2020

@galexrt
There is such a wrapper ready.
https://github.com/helm/charts/tree/master/incubator/raw

@galexrt
Copy link
Member

galexrt commented Oct 5, 2020

@maver1ck Yeah I know that one, but we need one specific for the Rook CephCluster object to be able to easily add "easy to use options / toggles" later down the road.

I'll start the chart as a specific for CephCluster wrapper and later on as users will hopefully contribute to it so that "easy use options" (e.g., "this is a test cluster" toggle, etc) can be added to it.

@maver1ck
Copy link
Author

maver1ck commented Oct 5, 2020

If you need any testing please let me know.

@galexrt
Copy link
Member

galexrt commented Oct 15, 2020

To update on this issue, I'm in the process of writing a design document to move all Helm charts to a separate repo, so the Helm charts are able to "move" / release without being bound to the Rook releases directly.
The Helm charts have the appVersion which can / should be used then to indicate the Rook version they are targeted for.
I should have that document finally ready tomorrow / weekendish for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
6 participants