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

[BUG]:Extra VolumeSnapshots were created with VolumeGroup Snapshot. #67

Closed
phuongatemc opened this issue Oct 20, 2021 · 3 comments
Closed
Labels
area/csi-powerflex Issue pertains to the CSI Driver for Dell EMC PowerFlex type/bug Something isn't working. This is the default label associated with a bug issue.

Comments

@phuongatemc
Copy link

phuongatemc commented Oct 20, 2021

Extra VolumeSnapshots were created with VolumeGroup Snapshot.
We don't see this problem on all VolumeGroup Snapshots.

Steps to reproduce the behavior:

  1. Using dellemc/csi-volumegroup-snapshotter:v0.2.0 to create VolumeGroup Snapshot of 3 PVCs with same label "volume-group=volumeGroup1"

  2. Run VolumeGroup Snapshot:
    apiVersion: volumegroup.storage.dell.com/v1alpha2
    kind: DellCsiVolumeGroupSnapshot
    metadata:
    name: "vg1-snap1"
    namespace: "helmtest-vxflexos"
    spec:
    driverName: "csi-vxflexos.dellemc.com"
    memberReclaimPolicy: "Delete"
    volumesnapshotclass: "vxflexos-snapclass"
    pvcLabel: "volumeGroup1"

  3. Get the result and found there are 6 VolumeSnapshot instead of 3.
    phuong@irv-vm-90-22 ~/work/vproxy-kubernetes (PPDM-159574-create-volumesnapshot) # k get dellcsivolumegroupsnapshots.volumegroup.storage.dell.com -n helmtest-vxflexos vg1-snap1 -o yaml
    apiVersion: volumegroup.storage.dell.com/v1alpha2
    kind: DellCsiVolumeGroupSnapshot
    metadata:
    annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
    {"apiVersion":"volumegroup.storage.dell.com/v1alpha2","kind":"DellCsiVolumeGroupSnapshot","metadata":{"annotations":{},"name":"vg1-snap1","namespace":"helmtest-vxflexos"},"spec":{"driverName":"csi-vxflexos.dellemc.com","memberReclaimPolicy":"Delete","pvcLabel":"volumeGroup1","volumesnapshotclass":"vxflexos-snapclass"}}
    creationTimestamp: "2021-10-07T21:28:43Z"
    finalizers:

  • vgFinalizer
    generation: 1
    name: vg1-snap1
    namespace: helmtest-vxflexos
    resourceVersion: "17321187"
    uid: b540018a-d032-4ed2-b808-27e43007d521
    spec:
    driverName: csi-vxflexos.dellemc.com
    memberReclaimPolicy: Delete
    pvcLabel: volumeGroup1
    volumesnapshotclass: vxflexos-snapclass
    status:
    creationTime: "2021-10-07T21:27:58Z"
    readyToUse: true
    snapshotGroupID: 217b52c106138b0f-c56b7c7b00000003
    snapshotGroupName: vg1-snap1-100721-212844
    snapshots: vg1-snap1-100721-212844-0-pvol1,vg1-snap1-100721-212844-1-pvol0,vg1-snap1-100721-212844-2-pvol2
    status: Complete
    phuong@irv-vm-90-22 ~/work/vproxy-kubernetes (PPDM-159574-create-volumesnapshot) # k get volumesnapshot -n helmtest-vxflexosNAME READYTOUSE SOURCEPVC SOURCESNAPSHOTCONTENT RESTORESIZE SNAPSHOTCLASS SNAPSHOTCONTENT CREATIONTIME AGE
    vg1-snap1-100721-212843-0-pvol1 true snapcontent-0fa08d6e-62f8-44ef-860d-54d565da49fc 8Gi vxflexos-snapclass snapcontent-0fa08d6e-62f8-44ef-860d-54d565da49fc 56s 9s
    vg1-snap1-100721-212843-1-pvol0 true snapcontent-930aba6b-758b-4e66-9df1-c00a208d3b1e 8Gi vxflexos-snapclass snapcontent-930aba6b-758b-4e66-9df1-c00a208d3b1e 56s 9s
    vg1-snap1-100721-212843-2-pvol2 true snapcontent-57f03380-9715-40f5-a5de-25ec80f74a8e 8Gi vxflexos-snapclass snapcontent-57f03380-9715-40f5-a5de-25ec80f74a8e 56s 9s
    vg1-snap1-100721-212844-0-pvol1 true snapcontent-e5e9a2e0-caa7-4020-8a06-432ba4d27f60 8Gi vxflexos-snapclass snapcontent-e5e9a2e0-caa7-4020-8a06-432ba4d27f60 55s 9s
    vg1-snap1-100721-212844-1-pvol0 true snapcontent-61dc29f7-dddf-4c0a-9b0b-fb957188924d 8Gi vxflexos-snapclass snapcontent-61dc29f7-dddf-4c0a-9b0b-fb957188924d 55s 9s
    vg1-snap1-100721-212844-2-pvol2 true snapcontent-2ecfdd38-fa4a-4bb5-ab9e-0347b3b3fc78 8Gi vxflexos-snapclass snapcontent-2ecfdd38-fa4a-4bb5-ab9e-0347b3b3fc78 55s 9s
    phuong@irv-vm-90-22 ~/work/vproxy-kubernetes (PPDM-159574-create-volumesnapshot) #

Expected behavior
Only 3 VolumeSnapshot should be created.

@phuongatemc phuongatemc added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. labels Oct 20, 2021
@nb950
Copy link

nb950 commented Oct 20, 2021

please update to dellemc/csi-volumegroup-snapshotter:v0.3.0 , this has been fixed

@prablr79 prablr79 added the area/csi-powerflex Issue pertains to the CSI Driver for Dell EMC PowerFlex label Oct 21, 2021
@prablr79 prablr79 changed the title [BUG]: [BUG]:Extra VolumeSnapshots were created with VolumeGroup Snapshot. Oct 21, 2021
@mjsdell mjsdell removed the needs-triage Issue requires triage. label Oct 21, 2021
@jooseppi-luna
Copy link
Contributor

@phuongatemc when you get a chance, would you be able to update us on the status of this issue so we can address it or close it? Thanks!

@phuongatemc
Copy link
Author

I no longer seeing this issue in version 0.3.0. Close this issue.

csmbot pushed a commit that referenced this issue Aug 1, 2023
* [replication] Added upgrade page and updated install info (#57)

* Added note about repctl logs file

* Added upgrade instructions for both controller and sidecar

* modified installation\upgrade section

* Fixed couple of grammar mistakes

* Added new entry to troubleshooting page

* Addressed review comments

* Changed link address

Co-authored-by: Maxim Sklyarov <Maxim_Sklyarov@dell.com>

* Update deployment steps for CSM Authorization (#58)

* begin updating deployment

* fixed typos

* add auth upgrade doc

* updated powerscale with authorization

* updated authorization documentation for powermax, powerflex, and powerscale

* refactored for powermax

* added vxflexos related docs for auth deployment and configuration

* consolidated proxy server root cert

* fix grammar, notes, value.yaml parameters, update auth deployment

* added note for driver configurations with auth

* updated note

* add auth note to drivers

* update upgrade path

Co-authored-by: atye <tyeaaron@gmail.com>
Co-authored-by: sharmilarama <sharmila.ramamoorthy@dell.com>
Co-authored-by: Logan Jones <logan_jones2@dell.com>

* Fix operator install docs (#62)

* Small update to the contributing doc (#54)

* Update _index.md

* Update _index.md

* fixed sidecar instructions

* Update _index.md

* making changes requested by Aron

* trying to get rid of unwanted changes

Co-authored-by: gallacher <35462391+gallacher@users.noreply.github.com>

* add Volume Health Monitor section (#67)

* add Volume Health Monitor section

* PR feedback

* pv/pvc metrics csi-powerstore changes (#64)

* Added troubleshooting documentation about gateway timeout for authorization (#63)

* Upgrade and Rollback Support for CSM for Authorization proxy server (#66)

* added auth upgrade and rollback, updated auth notes for drivers

* fixed spacing

* [replication] Added uninstall page, updated repctl readme (#70)

* static provisioning and ephemeral changes (#71)

* Update uninstall.md

* updated auuth deployment steps (#72)

* add  healthMonitorInterval to values table (#79)

* Helm install update (#74)

* updating helm install instructions

* adding troubleshooting for helm update

* minor changes and updates

* more minor changes

* word change

* more minor changes

* addressing comments from Jacob

* fixing numbers

* update code owners (#76)

* Move health monitor section to correct file  (#81)

* update correct file

* remove feature from wrong file

* Removed older OpenShift and added new driver versions (#84)

* Feature rwop csi powerstore (#89)

* Documentation for RWOP - CSI Powerstore

* Addressed review comment

* Update powerstore.md

Co-authored-by: shanmydell <82038610+shanmydell@users.noreply.github.com>

* Feature rwop accessmode support for csi-powerscale (#90)

Co-authored-by: shanmydell <82038610+shanmydell@users.noreply.github.com>

* Tenant documentation for both csi-unity and operator (#85)

Co-authored-by: shanmydell <82038610+shanmydell@users.noreply.github.com>

* Replication prerequisites & troubleshooting (#93)

Co-authored-by: shanmydell <82038610+shanmydell@users.noreply.github.com>

* Feature/pvc metrics csi powerstore update (#91)

* volume health monitoring update (#92)

* volume health monitoring update

* Update powerscale.md

* update documentation for health monitoring

Co-authored-by: shanmydell <82038610+shanmydell@users.noreply.github.com>
Co-authored-by: Randeep Sharma <92301596+randeepsharma@users.noreply.github.com>
Co-authored-by: Bahubali Jain <bahubali_jain@dell.com>

* Changed replication support matrix (#94)

* Changed replication support matrix

* Changed to X

* Add health values (#95)

* add new values to values table

* Add note to features section

* fix typo

* Common changes (#86)

* Unity - RWOP Access Mode and Volume Health Monitoring (#77)

* RWOP support matrix change (#96)

* Added known issue for unity (#97)

* Update powerflex.md (#98)

* powerscale release notes updated (#99)

* Operator Docs changes related to Unity features (#102)

* Operator upgrade documentation for volume health monitor changes (#104)

* Added note about how to list volume snapshots (#101)

* restructured deployment docs (#106)

* Improve operator install steps (#107)

* Update versions (#100)

* Added note that clarifies keys for csm installer (#108)

* Added volume health monitor in CSI spec support (#109)

* updated sample update for topology usage (#112)

#82

Co-authored-by: Andrey Schipilo <superdron97@yandex.ru>
Co-authored-by: Maxim Sklyarov <Maxim_Sklyarov@dell.com>
Co-authored-by: shaynafinocchiaro <66699024+shaynafinocchiaro@users.noreply.github.com>
Co-authored-by: atye <tyeaaron@gmail.com>
Co-authored-by: sharmilarama <sharmila.ramamoorthy@dell.com>
Co-authored-by: Logan Jones <logan_jones2@dell.com>
Co-authored-by: Jooseppi Luna <jooseppi_luna@dell.com>
Co-authored-by: JacobGros <jacobgrosner4@gmail.com>
Co-authored-by: Ashish Verma <32611022+ashish2207@users.noreply.github.com>
Co-authored-by: Trevor Dawe <trevor.dawe@dell.com>
Co-authored-by: gilltaran <91598969+gilltaran@users.noreply.github.com>
Co-authored-by: hoppea2 <33433874+hoppea2@users.noreply.github.com>
Co-authored-by: Francis Nijay <francis.nijay@dell.com>
Co-authored-by: shanmydell <82038610+shanmydell@users.noreply.github.com>
Co-authored-by: Bahubali Jain <66621574+bpjain2004@users.noreply.github.com>
Co-authored-by: karthikk92 <92289639+karthikk92@users.noreply.github.com>
Co-authored-by: Sakshi-dell <75004921+Sakshi-dell@users.noreply.github.com>
Co-authored-by: Randeep Sharma <92301596+randeepsharma@users.noreply.github.com>
Co-authored-by: Bahubali Jain <bahubali_jain@dell.com>
Co-authored-by: rensyct <80810999+rensyct@users.noreply.github.com>
Co-authored-by: Rajendra Indukuri <82365588+rajendraindukuri@users.noreply.github.com>
Co-authored-by: abhi16394 <32352976+abhi16394@users.noreply.github.com>
Co-authored-by: panigs7 <92028646+panigs7@users.noreply.github.com>
Co-authored-by: Prasanna M <35757638+prablr79@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csi-powerflex Issue pertains to the CSI Driver for Dell EMC PowerFlex type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

5 participants