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

uninstall with --crds results in flux-system namespace stuck terminating #632

Closed
Nicklason opened this issue Dec 23, 2020 · 3 comments · Fixed by #891
Closed

uninstall with --crds results in flux-system namespace stuck terminating #632

Nicklason opened this issue Dec 23, 2020 · 3 comments · Fixed by #891

Comments

@Nicklason
Copy link

I have an RKE cluster and installed flux2 with the Flux CLI. When using the uninstall command with the --crds argument it removes all namespaces and CRDs as expected, but the flux-system namespace is stuck terminating. When removing the fluxcd labels from the namespace then the namespace is removed.

A workaround is to first run flux uninstall without the --crds argument. After that I can use flux install --crds to remove the CRDs.

@stefanprodan
Copy link
Member

I can't reproduce this on Kubernetes Kind nor on GKE/EKS.

@mhenniges
Copy link

Got bit by a similar issue today, where I had bootstrapped flux via terraform, and then changed some arguments, so all the objects tried to get replaced (including the namespace), which never succeeded.

I believe this is because the deployments were successfully destroyed before some other objects, just as helmrepositories and helm releases, which are guarded with a finalizer. Without the flux2 pods running, there's nothing to clear the finalizer, so the objects and therefore the namespace is blocked from deleting.

@stefanprodan
Copy link
Member

@mhenniges flux uninstall will not delete the pods until all finalizers are done, for Terraform see fluxcd/terraform-provider-flux#67

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 a pull request may close this issue.

3 participants