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

KIC 2.0 - Secret Finalizers Stuck #1515

Closed
2 tasks done
shaneutt opened this issue Jul 8, 2021 · 0 comments · Fixed by #1522
Closed
2 tasks done

KIC 2.0 - Secret Finalizers Stuck #1515

shaneutt opened this issue Jul 8, 2021 · 0 comments · Fixed by #1522
Assignees
Labels
bug Something isn't working priority/medium

Comments

@shaneutt
Copy link
Contributor

shaneutt commented Jul 8, 2021

Currently in KIC 2.0 secrets can get our finalizers attached to them:

 -n ingress get secrets default-token-mm99r -o yaml
apiVersion: v1
kind: Secret
metadata:
  annotations:
    kubernetes.io/service-account.name: default
    kubernetes.io/service-account.uid: 2c90e799-1f28-4020-991b-35d5b801e4e1
  finalizers:
  - configuration.konghq.com/ingress

However nothing automatically removes them, causing (among other things) namespaces to get stuck in deletion during CI cleanup.

Acceptance Criteria:

  • fix the finalizer issue, possibly by never setting the finalizer but more likely by cleaning it up
  • ensure the tests account for this going forward: tests should not pass if namespaces created for them can't clean up
@shaneutt shaneutt added bug Something isn't working priority/medium labels Jul 8, 2021
@shaneutt shaneutt assigned shaneutt and unassigned shaneutt Jul 8, 2021
@shaneutt shaneutt changed the title KIC 2.0 - Secret Finalizers Not Cleaned KIC 2.0 - Secret Finalizers Stuck Jul 8, 2021
@shaneutt shaneutt self-assigned this Jul 9, 2021
@shaneutt shaneutt linked a pull request Jul 9, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority/medium
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant