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/k8s: update host policies for firewall tests. #25374

Merged
merged 1 commit into from
May 12, 2023
Merged

test/k8s: update host policies for firewall tests. #25374

merged 1 commit into from
May 12, 2023

Conversation

tommyp1ckles
Copy link
Contributor

Allow ICMP/ICMPv6 traffic on all nodes.

Fixes: #25344 #25343 #25342

@tommyp1ckles
test/k8s: update host policies for firewall tests.
08755bb 
Allow ICMP/ICMPv6 traffic on all nodes.

Fixes: cilium#25344 cilium#25343 cilium#25342

Signed-off-by: Tom Hadlaw <tom.hadlaw@isovalent.com>
@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 May 11, 2023
@tommyp1ckles
Copy link
Contributor Author

/test

@tommyp1ckles tommyp1ckles added the release-note/ci This PR makes changes to the CI. label May 11, 2023
@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 May 11, 2023
@youngnick youngnick mentioned this pull request May 11, 2023
Merged
@tommyp1ckles tommyp1ckles marked this pull request as ready for review May 11, 2023 17:23
@tommyp1ckles tommyp1ckles requested a review from a team as a code owner May 11, 2023 17:24
@tommyp1ckles
Copy link
Contributor Author

/test

@tommyp1ckles
Copy link
Contributor Author

Previous batch of tests passed, going to merge (reran by mistake)

@tommyp1ckles tommyp1ckles merged commit 8f7a537 into cilium:main May 12, 2023
48 of 50 checks passed
@pchaigno pchaigno added the area/host-firewall Impacts the host firewall or the host endpoint. label May 12, 2023
This was referenced May 12, 2023
Closed
Closed
@tommyp1ckles tommyp1ckles added the needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch label May 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.13.3 May 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.12.10 May 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from main in 1.12.10 May 15, 2023
@jibi jibi mentioned this pull request May 17, 2023
Merged
7 tasks
@jibi jibi added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels May 17, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.13 in 1.13.3 May 17, 2023
@jibi jibi added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. labels May 23, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.13 to Backport done to v1.13 in 1.13.3 May 23, 2023
@qmonnet qmonnet mentioned this pull request Jun 9, 2023
Closed
This was referenced Jun 9, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pchaigno pchaigno pchaigno approved these changes

@nbusseneau nbusseneau Awaiting requested review from nbusseneau nbusseneau is a code owner automatically assigned from cilium/ci-structure

Assignees
No one assigned
Labels
area/host-firewall Impacts the host firewall or the host endpoint. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.13.3
Backport done to v1.13
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CI: K8sDatapathConfig Host firewall With native routing and endpoint routes
3 participants
@tommyp1ckles @pchaigno @jibi