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

Installing crds via argocd #748

Closed
amarrella opened this issue Nov 6, 2018 · 6 comments
Closed

Installing crds via argocd #748

amarrella opened this issue Nov 6, 2018 · 6 comments

Comments

@amarrella
Copy link
Contributor

Hi, since argocd does a dry run before actually syncronizing a specific application, if i have crds defined in the application itself the sync will fail.

As a possible workaround, I tried creating a separate application pointing to a chart containing only the crds, but this fails the sync as the "application has no resources assigned".

Any solution to this catch-22?
Thanks :)

@jessesuen
Copy link
Member

@amarrella which version Argo CD are you running? We did have a bug exactly as you described about dry-run with CRDs, but this should be fixed in v0.10.

Basically, we now have special treatment for CRDs such that we will not fail the dry-run step if the resource Kind does not exist, with the assumption that the CRD is part of the install.

@amarrella
Copy link
Contributor Author

Thanks for the clarification! When I started looking into it it was still 0.9. Thanks for fixing it :)

@amarrella
Copy link
Contributor Author

Hmm actually I'm still having issues with dry-runs. Specifically installing Istio via argocd still does not work because it fails during the dry run :(
Using kubectl just works, but i would love to have it installed via argocd

helm template . --set ingress.enabled=false --set gateways.istio-egressgateway.enabled=false --set grafana-enabled=false --set prometheus.enabled=false --set global.disablePolicyChecks=true | kubectl apply -f -

@amarrella
Copy link
Contributor Author

I think this is due to the helm version being < 2.10. Would it be possible to upgrade helm?

@KarstenSiemer
Copy link
Contributor

i still experience this in version v0.12.0-rc3.
I have the prometheus operator and this registers the crds by itself. I cant be applied via argocd because of the dry run

@jessesuen
Copy link
Member

@KarstenSiemer can you share the output of argocd app get APPNAME -o yaml after the failed sync? I would like to see the errors happening during the sync.

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

3 participants