Skip to content

docs: clarify Helm-based App-of-Apps pattern with examples#25657

Open
nitishfy wants to merge 4 commits intoargoproj:masterfrom
nitishfy:helm-app-of-apps-docs
Open

docs: clarify Helm-based App-of-Apps pattern with examples#25657
nitishfy wants to merge 4 commits intoargoproj:masterfrom
nitishfy:helm-app-of-apps-docs

Conversation

@nitishfy
Copy link
Member

Closes #25648

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
  • The title of the PR states what changed and the related issues number (used for the release note).
  • The title of the PR conforms to the Title of the PR
  • I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
  • I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have signed off all my commits as required by DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My build is green (troubleshooting builds).
  • My new feature complies with the feature status guidelines.
  • I have added a brief description of why this PR is necessary and/or what this PR solves.
  • Optional. My organization is added to USERS.md.
  • Optional. For bug fixes, I've indicated what older releases this fix should be cherry-picked into (this may or may not happen depending on risk/complexity).

@nitishfy nitishfy requested review from a team as code owners December 15, 2025 07:22
@bunnyshell
Copy link

bunnyshell bot commented Dec 15, 2025

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

@nitishfy nitishfy force-pushed the helm-app-of-apps-docs branch from 1f22578 to 92920ad Compare December 17, 2025 12:50
@nitishfy nitishfy requested a review from afzal442 December 17, 2025 12:50
Copy link
Member

@afzal442 afzal442 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@nitishfy nitishfy force-pushed the helm-app-of-apps-docs branch from 785185f to 2fc159d Compare January 6, 2026 14:02
@nitishfy nitishfy requested a review from pjiang-dev January 6, 2026 14:03
Copy link
Member

@reggie-k reggie-k left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!
I think we should rewrite this section and recommend to perform cluster bootstrapping using AppSets instead, in order to avoid anti-pattern 16 here:
https://codefresh.io/blog/argo-cd-anti-patterns-for-gitops/

@kostis-codefresh
Copy link
Member

kostis-codefresh commented Jan 21, 2026

I actually added applicationsets here #26084

@nitishfy
Copy link
Member Author

nitishfy commented Feb 5, 2026

@kostis-codefresh that is pointing to the same PR, can you double check the link?

@kostis-codefresh
Copy link
Member

I fixed the link. I recommended Cluster generator and applicationsets here #26084

Signed-off-by: nitishfy <justnitish06@gmail.com>
Signed-off-by: nitishfy <justnitish06@gmail.com>
Signed-off-by: nitishfy <justnitish06@gmail.com>
@nitishfy nitishfy force-pushed the helm-app-of-apps-docs branch from 2fc159d to f24395c Compare February 13, 2026 11:07
@nitishfy nitishfy requested a review from reggie-k February 13, 2026 11:10
@nitishfy
Copy link
Member Author

@kostis-codefresh your changes are already present in the file so i don't think there's anything extra we need to do.

Signed-off-by: nitishfy <justnitish06@gmail.com>
@nitishfy nitishfy requested a review from jacobboykin February 16, 2026 11:57
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.

garbled language in the cluster bootstrapping docs

6 participants