-
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
tests: rework custom calls's AfterEach
/AfterAll
blocks to skip if needed
#16651
tests: rework custom calls's AfterEach
/AfterAll
blocks to skip if needed
#16651
Conversation
test-only --focus="K8sCustomCalls" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
We should run the full net-next job though. The installations/cleanups in one test can have impacts on other tests in the suite.
test-me-please |
It previously failed due to the comment after the trigger phrase. test-me-please |
The AfterAll() and AfterEach() blocks in the test file for custom calls run everytime, even if the Context block for the actual tests is skipped. In that case, running the final blocks results in an attempt to remove deployments that have never been set up in the first place. This may lead to the blocks failing when the tests were in fact skipped, and may produce test artifacts even though Jenkins does not considered the test failed. Let's reorganise those blocks, to make sure they are called only when necessary. Note that we do need to keep both DeleteCilium() and DeleteAll(), even if they are now in the same block, as calling only DeleteAll() would not remove the Cilium ConfigMap. Fixes: 37f6192 ("test: add CI test for tail calls hooks for custom programs") Fixes: cilium#13191 Fixes: cilium#16633 Reported-by: Paul Chaignon <paul@cilium.io> Signed-off-by: Quentin Monnet <quentin@isovalent.com>
5167dd9
to
a8cdfa7
Compare
test-me-please |
The
AfterAll()
andAfterEach()
blocks in the test file for custom calls run everytime, even if the Context block for the actual tests is skipped. In that case, running the final blocks results in an attempt to remove deployments that have never been set up in the first place. This may lead to the blocks failing when the tests were in fact skipped, and may produce test artifacts even though Jenkins does not considered the test failed.Fixes: 37f6192 ("test: add CI test for tail calls hooks for custom programs")
Fixes: #13191
Fixes: #16633