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.

Sign up for GitHub

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

Jump to bottom

[csi] CreateSnapshot implementation #86

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

[csi] CreateSnapshot implementation #86

wants to merge 16 commits into from

Conversation

astef
Copy link
Member

@astef astef commented Oct 2, 2024 •
edited
Loading

Description

  • implemented CREATE_DELETE_SNAPSHOT capability
  • support creating volumes from other volumes in CreateVolume for snapshot restore and volume clonning

Checklist

  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@astef
CreateSnapshot implementation
fad3653 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
specify source for snapshot volume
35f59df 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
avoid using reserved lv names for snapshot
f547143 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
snapshot deletion
1891708 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
volume creation from VolumeContentSource
f147e5f 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
snapshot capabilities
4ef3953 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
fix todos
f59a720 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
restore node name for snapshot
f852a63 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef astef added the enhancement New feature or request label Nov 6, 2024
@astef
lint
6f9f573 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
remove replace, go mod tidy
039fce6 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
fix go modules version
e32a745 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
use updated llvs properties
dff80ab 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef astef marked this pull request as ready for review November 12, 2024 11:13
@astef
fix review points
3ece44c 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@AleksZimin AleksZimin self-requested a review November 14, 2024 22:06
@astef
rbac
1cb09c0 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
align code with latest crd changes
ff1390e 
Signed-off-by: Alexandr Stefurishin <[email protected]>
@astef
fix linter
97536b3 
Signed-off-by: Alexandr Stefurishin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AleksZimin AleksZimin Awaiting requested review from AleksZimin

At least 1 approving review is required to merge this pull request.

Assignees

@astef astef

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@astef