Skip to content

Commit

Permalink
Disable pruning for automated sync of root app
Browse files Browse the repository at this point in the history
This ensures that we'll be able to roll out support for multiple root
apps without having to disable auto-sync for the default root app before
the roll out, since we already have some configuration that will
reassign existing component instances to another root app.

Later, we'll want to keep this setting, so we can move component
instances between different root apps without having to manually disable
auto-sync for the root apps on all clusters that use the component
that's being moved.
  • Loading branch information
simu committed Dec 30, 2024
1 parent 8c31821 commit cc35449
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 5 deletions.
5 changes: 5 additions & 0 deletions component/app.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,11 @@ local root_app = argocd.App('root', params.namespace, secrets=false) {
source+: {
path: 'manifests/apps/',
},
syncPolicy+: {
automated+: {
prune: false,
},
},
},
};

Expand Down
2 changes: 1 addition & 1 deletion tests/golden/defaults/argocd/apps/01_rootapp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ spec:
targetRevision: HEAD
syncPolicy:
automated:
prune: true
prune: false
selfHeal: true
2 changes: 1 addition & 1 deletion tests/golden/https-catalog/argocd/apps/01_rootapp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ spec:
targetRevision: HEAD
syncPolicy:
automated:
prune: true
prune: false
selfHeal: true
2 changes: 1 addition & 1 deletion tests/golden/openshift/argocd/apps/01_rootapp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ spec:
targetRevision: HEAD
syncPolicy:
automated:
prune: true
prune: false
selfHeal: true
2 changes: 1 addition & 1 deletion tests/golden/params/argocd/apps/01_rootapp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ spec:
targetRevision: HEAD
syncPolicy:
automated:
prune: true
prune: false
selfHeal: true
2 changes: 1 addition & 1 deletion tests/golden/prometheus/argocd/apps/01_rootapp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ spec:
targetRevision: HEAD
syncPolicy:
automated:
prune: true
prune: false
selfHeal: true

0 comments on commit cc35449

Please sign in to comment.