Skip to content

removed owner reference for PVC owned by statefulset#1919

Merged
vrutkovs merged 1 commit intomasterfrom
removed-owner-reference-for-sts-pvc
Mar 3, 2026
Merged

removed owner reference for PVC owned by statefulset#1919
vrutkovs merged 1 commit intomasterfrom
removed-owner-reference-for-sts-pvc

Conversation

@AndrewChubatiuk
Copy link
Contributor

@AndrewChubatiuk AndrewChubatiuk commented Mar 3, 2026

fixes #1845


Summary by cubic

Stop attaching OwnerReference to StatefulSet PVCs during resize to prevent unintended garbage collection and match Kubernetes behavior. Fixes #1845.

  • Bug Fixes
    • Removed owner parameter from updateSTSPVC; updated caller and unit test.
    • updatePVC now receives nil owner for StatefulSet-claimed PVCs.

Written for commit 0c8306c. Summary will update on new commits.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

@vrutkovs vrutkovs force-pushed the removed-owner-reference-for-sts-pvc branch from 9a62780 to 0c8306c Compare March 3, 2026 13:42
@vrutkovs vrutkovs merged commit 61acd66 into master Mar 3, 2026
4 checks passed
@vrutkovs vrutkovs deleted the removed-owner-reference-for-sts-pvc branch March 3, 2026 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vmdistributed: PVC is owned by both StatefulSet and VMCluster

2 participants