Addon: run Rook-Ceph inside Minikube#22826
Addon: run Rook-Ceph inside Minikube#22826bhavyaBeliever wants to merge 1 commit intokubernetes:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: bhavyaBeliever The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @bhavyaBeliever. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Can one of the admins verify this patch? |
- Made the `minikube addon enable rook-ceph` working - Made the `minikube addons disable rook-ceph` working - User can configure the rook-ceph addon: size and device(single) - Added the storage class for cephfs as well - Enabled for ceph dashboard - Rook-Ceph uses dataDirHostPath as /data
6ea53d3 to
2d58561
Compare
|
@bhavyaBeliever nobody can review 17k lines change, and the discussion on the issue barely started. We need to finish planning first before spending time on implementation. |
|
@nirs Out of 17k lines much of this is yaml files that are copied from the rook repository(crds.yaml, common.yaml, operator.yaml). The rook-ceph-crds.yaml and rook-ceph-operator.yaml are almost 16k |
|
Also, as we have taken this yaml files from rook repo what should be the license for such files? |
1k is much better than 17k but still a lot when we do't have any plan to discuss. I would not add 16k of rook resources to minikube. It does not make sense to keep this in minikube. We depend on rook images from the registry so we can depend on pulling rook resources from GitHub in the same way. Think abut updating rook version - the addon must update when rook updates. Pulling the resources from GitHub using kustomization makes update easier - you change the URL and maybe update the kustomization for new options added in the release. I think the modern way to implement addons is helm, which should be supported now in rook. This will keep the minikube code minimal. But the big question is not how to implement the addon, but why. Before we discuss the use cases we want to support there is no point to work on the implementation. |
Agreed, The whole point of making this PR is to test different configuration we might be looking forward with use cases we wanted inside minikube cluster by maybe just modify the rook-ceph-cluster.yaml. |
Steps to run:
Two things that can be configured.
extra-disksCheck if OSD pod is running with:
Current Implementation Details and configuration of Ceph cluster:
rook-cephfs,rook-ceph-block.dataDirHostPathis kept/data/rookinstead of default/var/lib/rook, supports persistant storage.minikube addons disable rook-cephis implemented, mapped custom teardown callback, used steps that were mentioned here.Pending Work:
fixes #21527