-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
KEP-3314: CSI Changed Block Tracking #4082
Merged
Merged
Changes from 52 commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
8a3c2f8
Add draft of CSI CBT KEP
ihcsim b9d8121
Update KEP status
ihcsim ba1e4c3
Initial structure.
carlbraganza fdb94b1
Removed distracting links to common K8s definitions.
carlbraganza 7b5403e
More caveats. Better grammar.
carlbraganza 4dfcd49
Use "snapshot access session".
carlbraganza 5e50923
addressed most of the feedback in the PR.
carlbraganza 93f8f61
Updated role figure.
carlbraganza 66eb80c
More refinements.
carlbraganza 3efbfbc
Session figure. Renamed figure files.
carlbraganza 3df9a5b
Fix background of session figure.
carlbraganza ffe8562
Merge pull request #1 from ihcsim/carl-proposal-caveats-risks
carlbraganza 2dee59e
Updated figures and roles.
carlbraganza 690f801
Propose a new role for session data.
carlbraganza a42d41e
Merge pull request #1 from ihcsim/carl-proposal-caveats-risks
carlbraganza cd0f4c1
GRPC spec
carlbraganza 1d7062f
Merge pull request #4 from ihcsim/carl-grpc
carlbraganza 46e6d93
Don't propose roles.
carlbraganza db9715a
Add user stories in the proposal (#2)
PrasadG193 b687ecb
Merge pull request #3 from ihcsim/update-figures
carlbraganza 20645b9
Added details to the manager, sidecar and SP service sections.
carlbraganza 679d5d3
Called out UNRESOLVED issues.
carlbraganza 753dbd2
Resolved issues with expiry and advertising.
carlbraganza 6e71521
Merge pull request #6 from ihcsim/carl-components
carlbraganza ec44e47
Updated TOC
carlbraganza 8319039
Fixed typo and svg space rendering.
carlbraganza 6b2130e
Fixed typo in perms figure.
carlbraganza 320991b
Typo in session figure. More detail in user stories.
carlbraganza 0842a7a
Add SnapshotSession CRDs (#5)
PrasadG193 bf2d9f4
Added image of the flow when the TokenRequest and TokenReview APIs ar…
carlbraganza a4671c5
Fixed figure spacing
carlbraganza deffd7f
Updated permissions svg; removed session.
carlbraganza d085a74
Updated figures. Removed session figure.
carlbraganza 0ae8dc7
Added explanation of permissions.
carlbraganza 78b5204
Updated overview and risks.
carlbraganza c59ce97
Updated RPC and components.
carlbraganza 5558e47
Completed remaining rewrite.
carlbraganza 45d2aee
Merge pull request #7 from ihcsim/carl-sidecar-with-ataz
carlbraganza e58bb3d
Updated to CSI spec to reflect https://github.com/container-storage-i…
carlbraganza 2bfe89f
Updated to CSI spec to reflect https://github.com/container-storage-i…
carlbraganza 36afbfd
Removed the security_token and namespace from the gRPC spec.
carlbraganza bbabfef
Update sections on test plan, PRR and graduation criteria
ihcsim 4c66f27
More neutral language on passing the auth token.
carlbraganza 56b8f8d
Merge branch 'csi-cbt-kep' of https://github.com/ihcsim/enhancements …
carlbraganza 8ffa4d3
Updated to reflect changes in the CSI spec PR.
carlbraganza c048605
Use a separate gRPC API for the sidecar.
carlbraganza 19a88ba
Replaced authorization gRPC metadata with a security_token field in r…
carlbraganza 64163d4
Fixed typo.
carlbraganza ce7d567
Updated CSI spec; downplayed similarity between the K8s and CSI gRPC …
carlbraganza aba16ff
Add beta and GA graduation criteria
ihcsim 9702d1d
Updated CSI spec again - no unsigned numbers used.
carlbraganza ba1dc52
Update KEP milestone to v1.30
ihcsim 3e3d030
Update 'Scalability' section
ihcsim 7ff9422
Add sig-auth as participating sigs
ihcsim 35ff183
Require that the CR be named for the driver.
carlbraganza ffb7813
Removed the label requirement for the CR.
carlbraganza 0dc9264
Replaced johnbelamaric with soltysh for PRR approver.
carlbraganza a100f35
Bump up milestone to v1.31
PrasadG193 32d8982
Change KEP status to implementable
PrasadG193 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| kep-number: 3314 | ||
| alpha: | ||
| approver: "@johnbelamaric" | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been looking at this before as shadow, and John is overbooked. So let's switch to me with PRR approval on this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!