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

App of App syncwaves #182

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

day0hero
Copy link
Contributor

@day0hero day0hero commented Oct 5, 2022

To control the flow of application deployments adding a count to the app
of apps pattern should help with deployments. Each application's counter
will +1 from the previous applications and theoretically should wait for
that application to be healthy/synced before deploying.

To control the flow of application deployments adding a count to the app
of apps pattern should help with deployments. Each application's counter
will +1 from the previous applications and theoretically should wait for
that application to be healthy/synced before deploying.
@beekhof
Copy link
Contributor

beekhof commented Oct 9, 2022

There are two possible problems:

  1. chart N needs something from chart M
  2. chart N needs something from chart M and chart M needs something from chart N

Scenario #1 is reasonably ok. I think most people, even without opening clusterGroup would see that M hasn't started syncing yet and try adjusting the order. Would adding ordering affect any existing patterns?

Would scenario #2 be just as broken without this change?

@soukron
Copy link
Contributor

soukron commented Apr 18, 2024

Can we rely on the order of appearance in the values file to make the assignation? What if it's just allowed to put the wave number in the values file so the end-user has the control of them so it can even ordered and group them as their convenience?

@soukron
Copy link
Contributor

soukron commented Apr 24, 2024

This issue was opened long time ago, and dependencies between apps is something that is a must. In example: we could need (in a not so theoretical use case) to configure Local Storage Operator before configuring Openshift Data Foundation before configuring Quay registry.

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