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.

Sign up for GitHub

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

Jump to bottom

test: Quarantine Secondary nodeport device tests #18091

Merged
merged 1 commit into from
Dec 2, 2021
Merged

test: Quarantine Secondary nodeport device tests #18091

merged 1 commit into from
Dec 2, 2021

Conversation

joestringer
Copy link
Member

See issue #18072 for more details about the flaky test. This is regularly
failing on master branch builds in Jenkins.

@joestringer
test: Quarantine Secondary nodeport device tests
b0e6824 
See issue 18072 for more details about the flaky test.

Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested a review from a team as a code owner December 2, 2021 02:22
@joestringer joestringer requested review from a team and christarazi December 2, 2021 02:22
@joestringer joestringer added the release-note/ci This PR makes changes to the CI. label Dec 2, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 2, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Dec 2, 2021
@joestringer
Copy link
Member Author

joestringer commented Dec 2, 2021 •
edited
Loading

This is regularly failing on the master branch: https://jenkins.cilium.io/job/cilium-master-k8s-1.22-kernel-4.19/ . Running the equivalent PR job should highlight that this PR is OK to merge.

@joestringer
Copy link
Member Author

/test-1.22-4.19

@joestringer joestringer mentioned this pull request Dec 2, 2021
Merged
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.0 Dec 2, 2021
@nathanjsweet nathanjsweet merged commit 2d7602e into cilium:master Dec 2, 2021
@joestringer joestringer deleted the submit/quarantine-secondary-node-tests branch December 2, 2021 18:58
@nathanjsweet nathanjsweet mentioned this pull request Dec 2, 2021
Merged
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.11.0 Dec 2, 2021
@joestringer joestringer added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels Dec 3, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.11 in 1.11.0 Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nathanjsweet nathanjsweet nathanjsweet approved these changes

@christarazi christarazi Awaiting requested review from christarazi christarazi was automatically assigned from cilium/ci-structure

@joamaki joamaki Awaiting requested review from joamaki

Assignees

@joamaki joamaki

@christarazi christarazi

Labels
backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.11.0
Backport done to v1.11
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@joestringer @nathanjsweet @joamaki @christarazi