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

test/runtime: Look into log errors after test start #17351

Merged
merged 1 commit into from
Sep 10, 2021

Conversation

joamaki
Copy link
Contributor

@joamaki joamaki commented Sep 9, 2021

When running runtime tests locally sometimes the test fail as level=error
log entries are found that are the result of cilium-agent restarts during
provisioning.

This is similar to the fix done in #14529.

When running runtime tests locally sometimes the test fail as level=error
log entries are found that are the result of cilium-agent restarts during
provisioning.

This is similar to the fix done in cilium#14529.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki added the release-note/ci This PR makes changes to the CI. label Sep 9, 2021
@joamaki joamaki requested review from pchaigno and a team September 9, 2021 09:21
@joamaki joamaki requested a review from a team as a code owner September 9, 2021 09:21
@joamaki joamaki requested review from a team September 9, 2021 09:21
@joamaki joamaki requested a review from a team as a code owner September 9, 2021 09:21
@maintainer-s-little-helper maintainer-s-little-helper bot assigned glibsm and unassigned glibsm Sep 9, 2021
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.10.5 Sep 9, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.9.11 Sep 9, 2021
@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake kind/bug/CI This is a bug in the testing code. labels Sep 9, 2021
@pchaigno
Copy link
Member

pchaigno commented Sep 9, 2021

test-runtime

@joamaki joamaki added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 9, 2021
@kaworu kaworu merged commit 38994b0 into cilium:master Sep 10, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.11 Sep 14, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.10.5 Sep 14, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.11 Oct 2, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.11 Oct 2, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.10 in 1.10.5 Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake kind/bug/CI This is a bug in the testing code. 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
No open projects
1.10.5
Backport done to v1.10
1.9.11
Backport done to v1.9
Development

Successfully merging this pull request may close these issues.

None yet

9 participants