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

[Bug]: CFPackage resources may leak after app deletion #2679

Open
danail-branekov opened this issue Jul 12, 2023 · 0 comments
Open

[Bug]: CFPackage resources may leak after app deletion #2679

danail-branekov opened this issue Jul 12, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@danail-branekov
Copy link
Member

danail-branekov commented Jul 12, 2023

What happened?

CI e2e test failure: https://ci.korifi.cf-app.com/builds/14653297

The app deletion e2e test deletes the app, awaits for the deletion job to complete and then checks that the getting the app and the package returns HTTP 404. However, the check that the package is gone has failed in this occurance.

Upon app deletion the CFPackage referenced is not deleted explicitly. The way this is expected to work is that the package controller upon reconciling a package would set an ownership relation from the app to the package. This is somewhat racy as the app might be deleted prior the package controller manages to reconcile the package. Then, on app deletion the CFPackage resource would remain behind.

What you expected to happen

  • The test not to flake
  • CFPackages do not remain after app being deleted

Acceptance Criteria

See above

How to reproduce it (as minimally and precisely as possible)

This is hard to reproduce, could not achieve it locally. However, the CI managed to

Anything else we need to know?

Related slack thread: https://cloudfoundry.slack.com/archives/C0297673ASK/p1689169941354209

Environment

Revision of codebase: 6316626

@danail-branekov danail-branekov added the bug Something isn't working label Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🧊 Icebox
Development

No branches or pull requests

1 participant