Skip to content

chore(argocd-understack): switch to unified 'understack' AppProject#1959

Open
cardoe wants to merge 1 commit intomainfrom
update-appproject
Open

chore(argocd-understack): switch to unified 'understack' AppProject#1959
cardoe wants to merge 1 commit intomainfrom
update-appproject

Conversation

@cardoe
Copy link
Copy Markdown
Contributor

@cardoe cardoe commented Apr 20, 2026

This switches us to the unified 'understack' AppProject for ArgoCD. From
here we'll be able to grant controls and actions on all Applications
that are part of Understack and start to craft proper limiting rules.

This switches us to the unified 'understack' AppProject for ArgoCD. From
here we'll be able to grant controls and actions on all Applications
that are part of Understack and start to craft proper limiting rules.
@cardoe cardoe requested a review from a team April 20, 2026 18:54
Copy link
Copy Markdown
Collaborator

@skrobul skrobul left a comment

Choose a reason for hiding this comment

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

looks good syntax wise, but are we sure what ArgoCD is going to do when we move apps between projects or rename them ?:)

@ctria
Copy link
Copy Markdown
Contributor

ctria commented Apr 22, 2026

I don't see any issue although I'm still not sure that all those apps should be part of understack deployment and not dependencies of understack (i.e. cilium, cert-manager) that need to be fulfilled before installing understack.

Regarding what ArgoCD will do. Moving apps between projects is harmless but renaming is not. My understanding is that it depends on what will be synced first, the existence of the new app or the removal of the old (for ArgoCD there is no rename, it is an app that removed and an app that is added).

@cardoe
Copy link
Copy Markdown
Contributor Author

cardoe commented Apr 23, 2026

I don't see any issue although I'm still not sure that all those apps should be part of understack deployment and not dependencies of understack (i.e. cilium, cert-manager) that need to be fulfilled before installing understack.

So that's why I originally had the separate AppProject because there was one set that was dependencies and one that was the actual parts.

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.

3 participants