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

CSI: Add support for CSIDriver object creation #4169

Merged
merged 1 commit into from
Oct 24, 2019

Conversation

Madhu-1
Copy link
Member

@Madhu-1 Madhu-1 commented Oct 24, 2019

Description of your changes:
This will be helpful to list the CSI drivers deployed in the kubernetes cluster. if we have N number of
CSI drivers deployed in the cluster, the user can query kubectl get csidriver and get the list of
registered CSI drivers and it requires kube 1.14+ and also the functionality is disabled by default

More info here: ceph/ceph-csi#698

Signed-off-by: Madhu Rajanna [email protected]

Which issue is resolved by this Pull Request:
Resolves #

Checklist:

  • Reviewed the developer guide on Submitting a Pull Request
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.
  • Pending release notes updated with breaking and/or notable changes, if necessary.
  • Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
  • Code generation (make codegen) has been run to update object specifications, if necessary.
  • Comments have been added or updated based on the standards set in CONTRIBUTING.md
  • Add the flag for skipping the CI if this PR does not require a build. See here for more details.
    [skip ci]

This will be helpful to list the CSI drivers deployed
in the kubernetes cluster. if we have N number of
CSI drivers deployed in the cluster, the user can
query kubectl get csidriver and get  the list of
registered CSI drivers and it requires kube 1.14+
and also the functionality is disabled by default

More info here: ceph/ceph-csi#698

Signed-off-by: Madhu Rajanna <[email protected]>
@Madhu-1 Madhu-1 added csi do-not-merge DO NOT MERGE :) labels Oct 24, 2019
@Madhu-1
Copy link
Member Author

Madhu-1 commented Oct 24, 2019

Added DNM label as this requires cephcsi 1.2.2 release

Copy link
Member

@leseb leseb left a comment

Choose a reason for hiding this comment

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

If that's useful what's the reason not to enable it by default?

@Madhu-1
Copy link
Member Author

Madhu-1 commented Oct 24, 2019

If that's useful what's the reason not to enable it by default?

cephcsi is built for multiple CO not completely for kubernetes and this feature is specific to CO built around Kubernetes, that why we have not enabled by default in cephcsi. however, this will be enabled by default if cephcsi is deployed by rook and I have provided an option to turn it off in rook deployment

@leseb leseb merged commit 59a5078 into rook:master Oct 24, 2019
@leseb
Copy link
Member

leseb commented Oct 24, 2019

Oops, I guess I merged it :-( @Madhu-1 should we revert?

leseb added a commit to leseb/rook that referenced this pull request Oct 24, 2019
I merged rook#4169 too quickly and didn't
noticed we were supposed to wait for csi 1.2.2.

This reverts commit 520ad44.

Signed-off-by: Sébastien Han <[email protected]>
binoue pushed a commit to binoue/rook that referenced this pull request Apr 10, 2020
I merged rook#4169 too quickly and didn't
noticed we were supposed to wait for csi 1.2.2.

This reverts commit 520ad44.

Signed-off-by: Sébastien Han <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
csi do-not-merge DO NOT MERGE :)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants