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

New repo for kubernetes-incubator/external-storage/ceph #324

Closed
cofyc opened this issue Dec 24, 2018 · 10 comments
Closed

New repo for kubernetes-incubator/external-storage/ceph #324

cofyc opened this issue Dec 24, 2018 · 10 comments
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@cofyc
Copy link
Member

cofyc commented Dec 24, 2018

New Repo, Staging Repo, or migrate existing

new repository

Requested name for new repository

sig-storage-ceph-provisioner

Which Organization should it reside

kubernetes-sigs

If not a staging repo, who should have admin access

@rootfs
@wongma7
@cofyc

If not a staging repo, who should have write access

@rootfs
@wongma7
@cofyc

If a new repo, who should be listed as approvers in OWNERS

@rootfs
@wongma7
@cofyc

If a new repo, who should be listed in SECURITY_CONTACTS

@rootfs
@wongma7
@cofyc

What should the repo description be

Ceph external provisioners for rbd and cephfs

What SIG and subproject does this fall under in sigs.yaml

this is part of the external-storage subproject for sig-storage

Approvals

Need sig-storage approval from: @saad-ali and @wongma7
See kubernetes-retired/external-storage#1083.

Additional context for request

Current, kubernetes-incubator/external-storage houses a lot of different provisioners. Different provisioners may have theirs own dependency/ci requirements or development/release workflow, etc. As for dependency management, it resulted that external-storage repo has huge dependency chain, and updating dependencies for one provisioner requires updating it for all provisioners which is not convenient and may have risk.
Xref: #244
Xref: #88

/area github-repo

@cofyc
Copy link
Member Author

cofyc commented Dec 24, 2018

cc @saad-ali @wongma7
What do you think?

@wongma7
Copy link
Contributor

wongma7 commented Jan 7, 2019

sgtm 👍

@saad-ali
Copy link
Member

Ceph external provisioners for rbd and cephfs

Why is this needed? Why add a ceph external-provisioner? There is already a Ceph CSI driver (https://github.com/ceph/ceph-csi) that handles dynamic provisioning.

@saad-ali
Copy link
Member

Oh I see it was a legacy external-provisioner (https://github.com/kubernetes-incubator/external-storage/tree/master/ceph) and because that repo is being split up, the ask is for a new home for this. My recommendation would be to deprecate this Ceph external-provisioner, and not find a new home for it, and recommend folks use the Ceph CSI driver.

@wongma7
Copy link
Contributor

wongma7 commented Feb 4, 2019

I think it is too soon to deprecate, the csi driver is still alpha. There could be a note in the README to say that someday it will be made obsolete by the CSI driver and to consider using that first, but we can't recommend switching yet. The provisioner project's still healthy and will benefit from having separate release scheme/cadence/branching (e.g. different images for mimic/luminous/etc...). When csi is matures it will be easier to steer people watching the repo in the right direction

@cblecker
Copy link
Member

@saad-ali is this still needed?

@nikhita
Copy link
Member

nikhita commented Mar 14, 2019

/lifecycle stale

Marking this as stale.. please advise if this is still required.

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 14, 2019
@wongma7
Copy link
Contributor

wongma7 commented Mar 14, 2019

ceph org may be a better home for this. @cofyc any progress on that?

@cofyc
Copy link
Member Author

cofyc commented Mar 15, 2019

/close

@k8s-ci-robot
Copy link
Contributor

@cofyc: Closing this issue.

In response to this:

/close

Instructions 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/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Development

No branches or pull requests

6 participants