Skip to content

fix: resolve openshift-gitops namespace stuck Active on destroy - #36

Merged
supernovae merged 1 commit into
mainfrom
fix/crb-destroy-reliability
Feb 16, 2026
Merged

fix: resolve openshift-gitops namespace stuck Active on destroy#36
supernovae merged 1 commit into
mainfrom
fix/crb-destroy-reliability

Conversation

@supernovae

Copy link
Copy Markdown
Owner

Replace kubernetes_namespace_v1 with kubectl_manifest for the openshift-gitops namespace. The native provider's delete state waiter treats "Active" as an unexpected state (errors at ~20s) instead of waiting for OLM to finalize CSV/OperatorGroup resources (1-3 min). kubectl_manifest handles deletion gracefully.

Also removes prevent_destroy from both CRBs (terraform_operator_crb, argocd_rbac) after confirming ROSA's webhook allows deletion:

  • rosa-terraform ns is not in the protected regex
  • openshift-gitops is in the webhook exception list
  • system: prefixed users bypass the webhook entirely

No manual terraform state rm steps needed for destroy.

Replace kubernetes_namespace_v1 with kubectl_manifest for the
openshift-gitops namespace. The native provider's delete state waiter
treats "Active" as an unexpected state (errors at ~20s) instead of
waiting for OLM to finalize CSV/OperatorGroup resources (1-3 min).
kubectl_manifest handles deletion gracefully.

Also removes prevent_destroy from both CRBs (terraform_operator_crb,
argocd_rbac) after confirming ROSA's webhook allows deletion:
- rosa-terraform ns is not in the protected regex
- openshift-gitops is in the webhook exception list
- system: prefixed users bypass the webhook entirely

No manual terraform state rm steps needed for destroy.

Co-authored-by: Cursor <cursoragent@cursor.com>
@supernovae
supernovae merged commit ee9da01 into main Feb 16, 2026
15 checks passed
@supernovae
supernovae deleted the fix/crb-destroy-reliability branch February 16, 2026 18:45
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.

1 participant