Skip to content

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Apr 19, 2022

Recent test runs have taken perhaps up to five minutes just to compile
various parts of the agent code from scratch with recent compilers, in
addition to taking a further ten minutes or so to run the tests. This is
close to the timeout, which means that occasionally the test will just
fail because it hits the timeout. Bump it to mitigate the issue.

Fixes: #18958

Recent test runs have taken perhaps up to five minutes just to compile
various parts of the agent code from scratch with recent compilers, in
addition to taking a further ten minutes or so to run the tests. This is
close to the timeout, which means that occasionally the test will just
fail because it hits the timeout. Bump it to mitigate the issue.

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested a review from a team as a code owner April 19, 2022 20:16
@joestringer joestringer added the release-note/ci This PR makes changes to the CI. label Apr 19, 2022
@joestringer joestringer requested a review from christarazi April 19, 2022 20:16
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Apr 19, 2022
@tklauser
Copy link
Member

/test-runtime

@borkmann borkmann added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Apr 20, 2022
@borkmann borkmann merged commit 0e76fdb into cilium:master Apr 20, 2022
@joestringer joestringer deleted the submit/bump-privileged-test-timeout branch April 20, 2022 16:23
@qmonnet qmonnet added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: RuntimePrivilegedUnitTests Run Tests: context deadline exceeded
7 participants