-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
test: Delete Istio resources if install does not complete #16440
test: Delete Istio resources if install does not complete #16440
Conversation
If Istio install fails (e.g., times out) during BeforeAll(), the resource cleanups in AfterAll() are not performed, so have to clean up explicitly after failure in BeforeAll instead. Without this some Istio resources may be left behind, causing other tests to fail due to unexpected pods being present. Signed-off-by: Jarno Rajahalme <jarno@isovalent.com>
test-me-please |
test-1.21-4.9
|
test-1.21-4.9 |
test-1.21-4.9 |
1 similar comment
test-1.21-4.9 |
@nebril Do I need to rebase to get the CI to provision a VM? |
@jrajahalme I don't think so, this provisioning issue seems different then NFS ones that we were hitting. |
Looks like env vars were not being set properly because of |
test-1.21-4.9 |
This PR only changes a CI 2.0 (Ginkgo) and all Ginkgo CI runs passed. Marking this ready-to-merge. |
If Istio install fails (e.g., times out) during BeforeAll(), the
resource cleanups in AfterAll() are not performed, so have to clean up
explicitly after failure in BeforeAll instead.
Without this some Istio resources may be left behind, causing other
tests to fail due to unexpected pods being present.
Signed-off-by: Jarno Rajahalme jarno@isovalent.com