Skip attach for non-attachable CSI volumes

[msau42]

Enhancement Description

• One-line enhancement description (can be used as a release note): Allow CSI drivers that don't support attach to not require the CSI external-attacher
• Primary contact (assignee): @jsafrane
• Responsible SIGs: sig-storage
• Design proposal link (community repo)/ KEP: Add KEP for csi skip attach #768 / https://github.com/kubernetes/enhancements/blob/master/keps/sig-storage/20190129-csi-skip-attach.md
• Link to e2e and/or unit tests:
• Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred: @msau42 @saad-ali
• Approver (likely from SIG/area to which enhancement belongs): @saad-ali
• Enhancement target (which target equals to which milestone):
  • Alpha release target (x.y): 1.12
  • Beta release target (x.y): 1.14
  • Stable release target (x.y)

API PRs:

• Make CSINodeInfo and CSIDriver Core APIs kubernetes#73883
• CSINodeInfo and CSIDriver Controller Changes kubernetes#74283

Please to keep this description up to date. This will help the Enhancement Team track efficiently the evolution of the enhancement

[msau42]

For some reason I couldn't find a feature issue for this, so I created this

[msau42]

@kubernetes/sig-storage-feature-requests

[msau42]

cc @claurence
This should be tracked as beta for 1.14.

[msau42]

/assign @bertinatto @jsafrane

[saad-ali]

KEP: https://github.com/kubernetes/enhancements/blob/master/keps/sig-storage/20190129-csi-skip-attach.md

[ameukam]

/stage beta

[simplytunde]

Hey @msau42 Just a friendly reminder we're looking for a PR against k/website (branch dev-1.14) due by Friday, March 1. It would be great if it's the start of the full documentation, but even a placeholder PR is acceptable. Let me know if you have any questions!

[claurence]

@msau42 Checking in as code freeze is approaching (March 7th) - I see a few PRs mentioned above but want to check which ones need to be merged by code freeze for this to be included in the 1.14 release (and if there are any additional PRs).

[msau42]

I updated the description with all the k/k prs required

[jsafrane]

This feature is not facing users and thus does not have docs on kubernetes.io. Its documentation is at https://kubernetes-csi.github.io/docs/. 1.14 updates are in progress at kubernetes-csi/docs#121

[kacole2]

Hello @msau42 @jsafrane , I'm the Enhancement Lead for 1.15. Is this feature going to be graduating alpha/beta/stable stages in 1.15? Please let me know so it can be tracked properly and added to the spreadsheet. Drop any relevant k/k PRs into this issue so they can be tracked as well. Thanks!

[irvifa]

Hello @msau42 @jsafrane I'm one of the v1.18 docs shadows.
Does this enhancement for (or the work planned for v1.18) require any new docs (or modifications to existing docs)? If not, can you please update the 1.18 Enhancement Tracker Sheet (or let me know and I'll do so)

If so, just a friendly reminder we're looking for a PR against k/website (branch dev-1.18) due by Friday, Feb 28th., it can just be a placeholder PR at this time. Let me know if you have any questions!

[irvifa]

Hello @msau42 @jsafrane I'm one of the v1.18 docs shadows.

Does this enhancement for (or the work planned for v1.18) require any new docs (or modifications to existing docs)? If not, can you please update the 1.18 Enhancement Tracker Sheet (or let me know and I'll do so)

If so, just a friendly reminder we're looking for a PR against k/website (branch dev-1.18) due by Friday, Feb 28th, which is roughly 1 weeks from now. It can just be a placeholder PR at this time. Let me know if you have any questions!

[jeremyrickard]

Hello @msau42 @jsafrane @gnufied,

Can you please link us to any k/k PRs for this enhancement so we can better track things. Code freeze is coming up quickly on 05 March 2020

[irvifa]

Hi @msau42 @jsafrane @gnufied I know you’re keeping tabs on this.. since the deadline is this Friday would you like to share when will you create a placeholder PR? Thanks..

[jeremyrickard]

/milestone clear

[VineethReddy02]

Hi, @msau42 @jsafrane @gnufied as there is no docs placeholder PR for this enhancement and didn't receive any update on the docs front. we have crossed the docs placeholder PR deadline. So we are removing this enhancement from the 1.18 release. If you want to request an exception. Please refer to https://github.com/kubernetes/sig-release/blob/master/releases/EXCEPTIONS.md

Thanks!

[gnufied]

so this enhancment and #603 are basically behind same feature gate which is called CSIDriverRegistry and it does make sense to open two different placeholder docs for documenting same feature-gate - kubernetes/website#19354

[VineethReddy02]

So @gnufied kubernetes/website#19354 will this placeholder PR contain docs for this enhancement or you want to create another placeholder PR? In the previous comment, you mentioned you want different placeholder PR

[msau42]

I think the same PR covers both

[jeremyrickard]

/milestone v1.18

[jeremyrickard]

/reopen @gnufied @msau42 you should leave this open until the KEP is updated to implemented

[gnufied]

Yeah. This is a side effect of PRs using direct ref rather than xref.

[jsafrane]

you should leave this open until the KEP is updated to implemented

Filed #1605

[palnabarun]

/close

(closing this since the KEP has been implemented)

[palnabarun]

/milestone clear

[k8s-ci-robot]

@palnabarun: Closing this issue.

In response to this:

/close

(closing this since the KEP has been implemented)

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.