Skip to content

workflows/ipsec: Cover Ingress #35476

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
Merged
merged 1 commit into from
Oct 28, 2024
Merged

workflows/ipsec: Cover Ingress #35476

merged 1 commit into from
Oct 28, 2024

Conversation

pchaigno
Copy link
Member

No description provided.

@pchaigno pchaigno added the release-note/ci This PR makes changes to the CI. label Oct 22, 2024
@pchaigno pchaigno force-pushed the pr/pchaigno/ipsec-workflows branch 2 times, most recently from 1ebf3ae to ec4bef2 Compare October 25, 2024 11:07
@pchaigno pchaigno changed the title workflows/ipsec: Upload artifacts workflows/ipsec: Cover Ingress Oct 25, 2024
@pchaigno pchaigno added feature/k8s-ingress feature/ipsec Relates to Cilium's IPsec feature labels Oct 25, 2024
@pchaigno pchaigno force-pushed the pr/pchaigno/ipsec-workflows branch 2 times, most recently from 128088f to 635ad6b Compare October 28, 2024 10:01
@pchaigno
workflows/ipsec: Cover Ingress
535bc2c 
This change is enough to cover IPsec+KPR+Ingress because Ingress tests
are already included in the CLI.

To double check this change, I modified the pull request to upload
sysdumps even in case of success and ran a quick sanity check on those
sysdumps.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno force-pushed the pr/pchaigno/ipsec-workflows branch from 635ad6b to 535bc2c Compare October 28, 2024 10:02
@pchaigno
Copy link
Member Author

/test

@pchaigno pchaigno marked this pull request as ready for review October 28, 2024 10:11
@pchaigno pchaigno requested review from a team as code owners October 28, 2024 10:11
@pchaigno pchaigno enabled auto-merge October 28, 2024 10:59
@pchaigno pchaigno added this pull request to the merge queue Oct 28, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 28, 2024
Merged via the queue into main with commit 9bf29a6 Oct 28, 2024
276 checks passed
@pchaigno pchaigno deleted the pr/pchaigno/ipsec-workflows branch October 28, 2024 11:27
@pchaigno pchaigno added the needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch label Nov 8, 2024
@viktor-kurchenko viktor-kurchenko mentioned this pull request Nov 12, 2024
Merged
13 tasks
@viktor-kurchenko viktor-kurchenko added backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. and removed needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch labels Nov 12, 2024
@github-actions github-actions bot removed the backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. label Nov 12, 2024
@github-actions github-actions bot added the backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. label Nov 12, 2024
@cilium-release-bot cilium-release-bot bot mentioned this pull request Nov 13, 2024
Merged
@joestringer joestringer mentioned this pull request Nov 20, 2024
Merged
@julianwiedmann
Copy link
Member

The v1.16 backport was reverted, adjusting the labels so it doesn't get lost.

@julianwiedmann julianwiedmann added backport/author The backport will be carried out by the author of the PR. needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch and removed backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. labels Nov 27, 2024
@julianwiedmann julianwiedmann mentioned this pull request Mar 24, 2025
Closed
@julianwiedmann
Copy link
Member

Second time lucky 🤞

@julianwiedmann julianwiedmann added backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. and removed needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch labels Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brlbil brlbil brlbil approved these changes

@jschwinger233 jschwinger233 jschwinger233 approved these changes

Assignees
No one assigned
Labels
backport/author The backport will be carried out by the author of the PR. backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. feature/ipsec Relates to Cilium's IPsec feature feature/k8s-ingress 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@pchaigno @julianwiedmann @brlbil @jschwinger233 @viktor-kurchenko