Closed chethanv28 closed 5 months ago
@chethanv28 do we clean up storagequota
CR status filed and remove policyQuotaUsage
status for the policy Id for which storagepolicyquota
CR is deleted?
/ok-to-test
@chethanv28 do we clean up
storagequota
CR status filed and removepolicyQuotaUsage
status for the policy Id for whichstoragepolicyquota
CR is deleted?
CSI Syncer does not perform any updates on storagequota
CR . I'm not sure if the status field cleanup is handled in StorageQuotaController repo.
@akankshapanse Can you answer this query ?
Started WCP block pre-checkin pipeline... Build Number: 6
Build ID: 6
WCP build status: SUCCESS
Stage before exit: finally
Jenkins E2E Test Results:
Ran 1 of 816 Specs in 271.244 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 815 Skipped
PASS
Ginkgo ran 1 suite in 5m35.032847777s
Test Suite Passed
make: Leaving directory `/home/worker/workspace/csi-wcp-precheckin_kavya/Results/6/vsphere-csi-driver`
Started WCP block pre-checkin pipeline... Build Number: 7
Build ID: 7
WCP build status: FAILURE
Stage before exit: null
Jenkins E2E Test Results:
FAIL! -- 10 Passed | 44 Failed | 0 Pending | 762 Skipped
--- FAIL: TestE2E (3972.50s)
FAIL
Ginkgo ran 1 suite in 1h7m17.802685179s
Test Suite Failed
make: Leaving directory `/home/worker/workspace/csi-wcp-precheckin_kavya/Results/7/vsphere-csi-driver`
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: chethanv28, divyenpatel
The full list of commands accepted by this bot can be found here.
The pull request process is described here
e clean up
storagequota
CR status filed and removepolicyQuota
storagequota CR status filed and remove policyQuotaUsage status for the policy Id for which storagepolicyquota CR is deleted will be done by Storage quota webhook
What this PR does / why we need it: Handle StoragePolicyUsageCR cleanup in StoragePolicyQuotaCR delete event
Testing done:
Before:
Unassigned storage policy from vSphere UI:
Verify quota CR is deleted:
Verify policy usage CR is deleted:
CSI Syncer logs:
Verification with 2 PolicyUsage CRs:
After removing the policy from the namespace:
Logs:
Special notes for your reviewer:
Release note: