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

Eliminate sleep pre-sync hook for CAPZ install #47

Open
dtzar opened this issue Apr 30, 2024 · 2 comments
Open

Eliminate sleep pre-sync hook for CAPZ install #47

dtzar opened this issue Apr 30, 2024 · 2 comments

Comments

@dtzar
Copy link
Collaborator

dtzar commented Apr 30, 2024

This issue is for a: (mark with an x)

- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Expected/desired behavior

The install of CAPZ relies on the natural available mechanisms for installation with ArgoCD versus the pre-sync sleep hook which points to sleep hook file.

This could be one or more of the following:

  1. Sync-Waves
  2. Progressive Syncs

For related discussion, see ArgoCD Application Dependencies thread and this blog post.

@dtzar
Copy link
Collaborator Author

dtzar commented May 24, 2024

@dtzar
Copy link
Collaborator Author

dtzar commented Sep 13, 2024

Attempted to make this work by:

  1. Setting readiness delay to 60 seconds
  2. Setting cert-manager to sync-wave -1, CAPI-Operator to sync-wave 2, namespaces inside chart to sync-wave 3/4 See code comparison here

The cert-manager did see the 60 second delay and ArgoCD did wait for it to state it was ready before proceeding to try and install CAPI-Operator. Unfortunately, CAPI-Operator failed to install due to not getting the certs needed.

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

No branches or pull requests

1 participant