alexlebens/external-snapshotter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
1,250 Commits 1 Branch 0 Tags
a4412e927cb941940813ef935d08ad9c45cb0d27
Commit Graph

81 Commits

Author SHA1 Message Date
Raunak Pradip Shah
875662b365 Add finalizer to indicate volume snapshot is part of a group 2023-12-06 16:16:30 -08:00
Raunak Pradip Shah
060ee23d6d Delete individual volume snapshots as part of the group snapshot delete API and prevent users from deleting individual volume snapshots if it is part of an existing group snapshot 
Also revert commit bb29899ca3.
 2023-11-17 14:24:55 +05:30
Jan Safranek
8a29bf5b21 Re-queue SnapshotContents that are readyToUse: false 
SnapshotContents with readyToUse: false should be periodically requeued
with exp. backoff until the CSI driver confirms the snapshot is ready.
 2023-11-16 15:37:48 +01:00
Jan Safranek
f9e125b994 Check sync error in unit test properly 
Tests must fail when syncContent() returns success and error was expected.
 2023-11-16 15:37:48 +01:00
Raunak Pradip Shah
42151b59aa change VolumeGroupSnapshotContentName to VolumeGroupSnapshotHandle in snapshot content status 2023-11-08 12:06:06 +05:30
Raunak Pradip Shah
facefbad6e add DeleteVolumeGroupSnapshot API to delete group snapshot 2023-11-03 13:14:00 +05:30
Shubham Pampattiwar
ff71329d8c Update VolumeSnapshot and VolumeSnapshotContent using JSON patch 
fix unit tests

operate on cloned copy

revert to update call for removeSnapshotFinalizer

fix content delete finalizer
 2023-09-15 12:17:38 -07:00
Sameer Shaikh
7fac9cdaf8 external-snapshotter constantly retrying CreateSnapshot calls on error w/o backoff 2023-06-29 21:17:27 +05:30
Sameer Shaikh
d8e698f609 external-snapshotter constantly retrying CreateSnapshot calls on error w/o backoff 2023-06-29 08:03:57 +05:30
Sameer Shaikh
ffaba0f81d external-snapshotter constantly retrying CreateSnapshot calls on error w/o backoff 2023-06-28 18:32:07 +05:30
Sameer Shaikh
a8c9b7b644 external-snapshotter constantly retrying CreateSnapshot calls on error w/o backoff 2023-06-28 15:59:21 +05:30
Sameer Shaikh
98939c02c5 external-snapshotter constantly retrying CreateSnapshot calls on error w/o backoff 
Signed-off-by: Sameer Shaikh <sameer.shaikh@ibm.com>
 2023-06-28 13:17:42 +05:30
Raunak Pradip Shah
21acc924d7 Fix typos 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
65fbb48cd7 Change timestamp format in volumesnapshot and content names 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
80893cb871 Add hash of PVC UUID to group snapshot names 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
e26501e510 Add reference details to volume group snapshot content status 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
9e2d4f6264 Add groupSnapshotNamePrefix and groupSnapshotNameUUIDLength CLI options 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
d8d01a38e4 Change all occurences of content to group snapshot content 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
871312988b Added create group volume snapshot functionality 2023-05-03 21:21:32 +05:30
Raunak Pradip Shah
299862c783 Add csi snapshotter functionality for volume group snapshots 2023-05-03 21:21:32 +05:30
Humble Chirammal
d34102faae make the formatting errors based on go 1.19 
Reference#
https://github.com/kubernetes-csi/csi-release-tools/pull/203

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
 2022-09-13 19:37:34 +05:30
Humble Chirammal
f30b14f8fd make use of short declaration of variables and cleanup code 
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
 2022-04-13 08:35:03 +05:30
Raunak Pradip Shah
5b3a927d2a Vendored deps 2022-03-24 21:40:29 +05:30
Raunak Pradip Shah
d300796fab - Update snapshotter module to v6 
- Update snapshotter client module to v5
- Update go version in client modules to v1.17
 2022-03-23 08:59:19 +05:30
Peter Schuurman
497a316b04 Bugfix: Check if newSnapContent.Status is set before checking if Status.Error is present 2022-03-08 09:06:37 -08:00
Peter Schuurman
edf2e03f35 Allow resync on pvc when there is no error on the object 2022-03-02 11:26:16 -08:00
zhucan
e46df1bdab external-snapshotter constantly retrying CreateSnapshot calls on error w/o backoff 
Signed-off-by: zhucan <zhucan.k8s@gmail.com>
 2022-02-14 10:38:57 +08:00
Grant Griffiths
f3d345340e Fix an issue where patch will fail when status is nil 
Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-10-20 19:36:48 -07:00
Grant Griffiths
d14e2eea8f Use patch for snapshot-controller when there are no finalizers 
- Also address PR feedback re: avoid a deepCopy for annotations patch

Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-09-29 16:43:02 -07:00
Grant Griffiths
b2d17cd6bb setAnnVolumeSnapshotBeingCreated patch impl 
Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-09-28 17:40:55 -07:00
Grant Griffiths
3b9f0e2c72 Fix unit tests and don't patch with empty finalizers 
Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-09-28 14:29:40 -07:00
Grant Griffiths
afff04176d WIP 
Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-09-23 14:05:17 -07:00
Grant Griffiths
0476dcedcc Refactor http server and register leaderelection health check 
- Needed to start the http server outside of pkg/metric
- We needed this because we want to add other endpoints to the server

Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-08-10 11:44:12 -07:00
Kubernetes Prow Robot
d6d967c79d Merge pull request #527 from ggriffiths/removeannvs_beingcreated_returncontent 
Return VolumeSnapshotContent from various functions instead of nil
 2021-06-17 01:54:18 -07:00
Grant Griffiths
5cf92fc01a Return VolumeSnapshotContent from various functions instead of nil 
Signed-off-by: Grant Griffiths <ggriffiths@purestorage.com>
 2021-06-14 10:47:05 -07:00
Humble Chirammal
9f7f993c50 Add ability to customize VolumeSnapshotContent workqueue 
Two new timeout values ( retryIntervalStart & retryIntervalMax )
have been added to set the ratelimiter for volumesnapshotcontent queue.

Fix# https://github.com/kubernetes-csi/external-snapshotter/issues/463

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>

```release-note
 `retry-interval-start` and `retry-interval-max` arguments are added to csi-snapshotter sidecar which controls retry interval of failed volume snapshot creation and deletion. These values set the ratelimiter for volumesnapshotcontent queue.
```

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
 2021-06-10 19:12:12 +05:30
Christian Huffman
c554274833 Correct snapshotContent error propagation 2021-04-21 18:38:40 -04:00
Timo Reimann
b31a29fd83 Retain CreateSnapshot error 
The error returned from CreateSnapshot was overwritten by the call to
removeAnnVolumeSnapshotBeingCreated. Retain the error through variable
shadowing so that it can be propagated and reported upwards the call
stack.

We also improve the error line by dropping an unnecessary comma and
dequoting the error string (since we already employ a colon separator).
 2021-02-07 21:58:37 +01:00
xing-yang
730e6f6458 Update to client/v4 2020-12-14 18:50:10 +00:00
xing-yang
5069c99ec3 Update controller based on snapshot v1 apis 2020-12-11 21:13:47 +00:00
Christian Huffman
0f3fa91d0b Don't check for snapshotClass when deleting content 2020-11-06 15:43:45 -05:00
Chris Henzie
f705b23bbd Pass snapshot metadata to CSI driver 2020-10-09 12:53:32 -07:00
xing-yang
a2ac03c740 Update to v3 2020-09-08 21:25:47 +00:00
Kubernetes Prow Robot
a69711cec2 Merge pull request #307 from Kartik494/updategomod 
Added go modules for Apis and Client
 2020-08-19 21:45:22 -07:00
xing-yang
2a990923fd Emit event even if status update fails so user can see it 2020-08-09 04:16:16 +00:00
kartik494
ae215218d1 Added go modules for Apis and Client 2020-07-29 16:52:45 +05:30
Christian Huffman
ffaeac1515 Move to klogv2 2020-07-27 14:39:56 -04:00
Max Jonas Werner
04848c755a chore: remove unused parameter from util funcs; add unit test 
Added the unit tests from
https://github.com/kubernetes/kubernetes/blob/master/pkg/util/slice/slice_test.go
(minus modifier tests) so we can be sure to not change behaviour.
 2020-07-22 09:36:22 +02:00
Max Jonas Werner
1f13709413 chore: get rid of k8s.io/kubernetes dep 
This dependency is problematic because of the many replaces it brings
with it. This makes it hard to import external-snapshotter in 3rd
party projects without having a lot of `replace` directives in those
projects as well.

The only usage of that import was to get two very simple utility
functions so I copied them over to this repo.
 2020-07-21 18:23:38 +02:00
Christian Huffman
daf0051f2b Updated sidecar to not require VolumeSnapshotClass for snapshot deletion 2020-06-04 17:16:45 -04:00