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

Increase maximum number of tries in WaitForTracingPolicy #2547

Merged
merged 2 commits into from
Jun 12, 2024

Conversation

tpapagian
Copy link
Member

@tpapagian tpapagian commented Jun 12, 2024

Thanks for contributing! Please ensure your pull request adheres to the following guidelines:

  • All commits contain a well written commit message and are signed-off (see Submitting a pull request).
  • All code is covered by unit and/or end-to-end tests where feasible.
  • All generated files are updated if needed (see Making changes).
  • Provide a title or release-note blurb suitable for the release notes (see guidelines).
  • Update documentation and write an upgrade note if needed (see guidelines).
  • Are you a user of Tetragon? Please add yourself to the Users doc in the Cilium repository.

After merging #2506 it seems that there are cases where 10 * 1 sec is not enough when waiting for a tracing policy to load (mainly in e2e tests).

This patch increases that to 20 * 1 sec to avoid timeouts.

After merging #2506 it seems that
there are cases where 10 * 1 sec is not enough when waiting for a
tracing policy to load (mainly in e2e tests).

This patch increases that to 20 * 1 sec to avoid timeouts.

Signed-off-by: Anastasios Papagiannis <[email protected]>
@tpapagian tpapagian added the release-note/ci This PR makes changes to the CI. label Jun 12, 2024
@tpapagian tpapagian marked this pull request as ready for review June 12, 2024 16:33
@tpapagian tpapagian requested a review from a team as a code owner June 12, 2024 16:33
@tpapagian tpapagian requested a review from jrfastab June 12, 2024 16:33
@tpapagian tpapagian merged commit 23b041e into main Jun 12, 2024
40 of 45 checks passed
@tpapagian tpapagian deleted the pr/apapag/increase-timeout branch June 12, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants