-
Notifications
You must be signed in to change notification settings - Fork 58
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
Install argocd #1421
Install argocd #1421
Conversation
1f636a5
to
ae586b9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope you plan to squash all the tiny commits to smaller logical commits. I reviewed the entire changes since the commit looks like temporary commits.
8c2bbce
to
5bda9f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@par97 I rebased your PR on main for testing, and push it to ramen repo by mistake.
Re-basing on main when asking for review is a good idea in general, it makes it easier to review and test the PR.
9203b27
to
4ae91e1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The start hook looks good, added suggestions for minor improvements.
The test hook works but it install on the hub instead of the managed clusters, can be easily change to install on the managed clusters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but waiting for the hub is not needed.
Signed-off-by: jacklu <[email protected]>
Local test with argocd env
|
Local test with regional-dr env
|
We change the cache API in RamenDR#1389 but it was merge after the argocd was added. - Use cache.get() instead of removed cache.fetch() - Change current directory for start hook was missing in RamenDR#1421, not sure how the cache worked without it. Signed-off-by: Nir Soffer <[email protected]>
We change the cache API in #1389 but it was merge after the argocd was added. - Use cache.get() instead of removed cache.fetch() - Change current directory for start hook was missing in #1421, not sure how the cache worked without it. Signed-off-by: Nir Soffer <[email protected]>
This PR installs argocd operator on hub cluster, and enables ability to create argocd application CR and applicationset CR on hub, and the application could be auto deployed to the target dr cluster by argocd controller, using push mode.