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
Remove validation timeout in controlplane testing #26414
Conversation
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.
The change makes sense to me. I was reading the discussion in the linked issue and came across this, which made me hesitate on approving this PR. Raising it here for us to discuss whether that comment impacts this PR.
Yep, I'm working to verify what has been suggested. But I think we should consider removing the timeout anyway, to avoid possible further flakyness. |
This comment was marked as outdated.
This comment was marked as outdated.
4f39bca
to
48568a4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
48568a4
to
5bf0145
Compare
Due to the unpredictability of a CI environment, setting a validation timeout when running tests may lead to flakyness. At the same time, a short timeout is useful when developing locally to get faster feedback from the tests. The commit changes the controlplane test framework to run without a timeout by default. An additional method WithValidationTimeout is added to set a custom one when developing locally. Moreover, the result of each failed validation is always printed, to ease debuggability both locally and in a CI environment. Related: cilium#26082 Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
5bf0145
to
c67446c
Compare
/test |
The only affected tests ran successfully in Travis, so marking it ready-to-merge. |
Due to the unpredictability of a CI environment, setting a validation timeout when running tests may lead to flakyness. At the same time, a short timeout is useful when developing locally to get faster feedback from the tests.
The commit changes the controlplane test framework to run without a timeout by default. An additional method
WithValidationTimeout
is added to set a custom one when developing locally. Moreover, the result of each failed validation is always printed, to ease debuggability both locally and in a CI environment.Related: #26082