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

(feat) better handle conflicts with Helm charts #740

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

gianlucam76
Copy link
Member

When Sveltos detects a conflict during ClusterSummary reconciliation, Sveltos requeues the ClusterSummary instance after a fixed retry interval (default: 60 seconds).

For Helm charts, Sveltos leverages its knowledge of competing ClusterSummaries. Only one ClusterSummary is granted management rights for a specific Helm chart in a given managed cluster. Others are denied and flagged as failures.

If the managing ClusterSummary is deleted, Sveltos requeues all contenders. With this PR, for Helm charts, a conflict is treated as non-retriable. Sveltos will retry reconciliation once the conflict is resolved.

When Sveltos detects a conflict during ClusterSummary reconciliation, Sveltos
requeues the ClusterSummary instance after a fixed retry interval (default: 60 seconds).

For Helm charts, Sveltos leverages its knowledge of competing ClusterSummaries.
Only one ClusterSummary is granted management rights for a specific Helm chart
in a given managed cluster. Others are denied and flagged as failures.

If the managing ClusterSummary is deleted, Sveltos requeues all contenders.
With this PR, for Helm charts, a conflict is treated as non-retriable.
Sveltos will retry reconciliation once the conflict is resolved.
@gianlucam76 gianlucam76 merged commit 260d9bd into projectsveltos:main Oct 23, 2024
5 checks passed
@gianlucam76 gianlucam76 deleted the helm-chart-conflict branch October 23, 2024 16:08
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.

1 participant