Skip to content

Add argo sync-options annotation to retain PVs - #1307

Closed
dan-fernandes wants to merge 2 commits into
mainfrom
fix-persistent-volume-retention
Closed

dan-fernandes wants to merge 2 commits into
mainfrom
fix-persistent-volume-retention

Conversation

@dan-fernandes

Copy link
Copy Markdown
Contributor

Fixes #1227

@dan-fernandes
dan-fernandes requested a review from a team as a code owner December 16, 2025 08:06
@codecov

codecov Bot commented Dec 19, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.74%. Comparing base (ec820af) to head (9ee3f2e).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1307   +/-   ##
=======================================
  Coverage   94.74%   94.74%           
=======================================
  Files          42       42           
  Lines        2741     2741           
=======================================
  Hits         2597     2597           
  Misses        144      144           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dan-fernandes

Copy link
Copy Markdown
Contributor Author

Closed because these sync options do not work as I thought they did.

I was under the impression that they would cause the PV to be retained after the PVC was deleted. I am now under the impression that they cause the PVC to not be deleted when resources using it are deleted.

I may be incorrect, but I have contacted the cloud team for further clarification.

This branch had an error being deployed

1 failed and 1 inactive deployments
prod — 9ee3f2e2 Deployed Dec 19, 2025 by dan-fernandes via container / publish gcr #7736
release — 9ee3f2e2 Deployed Dec 19, 2025 by dan-fernandes via pypi / upload #7736
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.

ArgoCD ignores Helm reclaim policy and does not persist scratch area

1 participant