Skip to content

ci/connectivity: Don't apply cnp to conn-disrupt pods in 1.15, 1.14 #36682

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
Dec 19, 2024
Merged

ci/connectivity: Don't apply cnp to conn-disrupt pods in 1.15, 1.14 #36682

merged 1 commit into from
Dec 19, 2024

Conversation

jschwinger233
Copy link
Member

@jschwinger233 jschwinger233 commented Dec 18, 2024 •
edited
Loading

See #36681 for details.

1.15 ci-ipsec-upgrade test: #36683

@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 18, 2024
@github-actions github-actions bot added cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary labels Dec 18, 2024
@jschwinger233 jschwinger233 mentioned this pull request Dec 18, 2024
Closed
jschwinger233 added a commit that referenced this pull request Dec 18, 2024
@jschwinger233
ci-ipsec-upgrade: use cilium-cli-ci built from #36682
a03ac0c 
Signed-off-by: gray <gray.liang@isovalent.com>
@jschwinger233 jschwinger233 mentioned this pull request Dec 18, 2024
Closed
@jschwinger233 jschwinger233 force-pushed the pr/gray/cli-skip-cnp-for-conn-disrupt-in-1.15 branch from a2b9906 to c50d65f Compare December 18, 2024 05:09
jschwinger233 added a commit that referenced this pull request Dec 18, 2024
@jschwinger233
ci-ipsec-upgrade: use cilium-cli-ci built from #36682
464ab79 
Signed-off-by: gray <gray.liang@isovalent.com>
@jschwinger233
ci/connectivity: Don't apply cnp to conn-disrupt pods in 1.15, 1.14
0ac7eba 
See #36681 for details.

Signed-off-by: gray <gray.liang@isovalent.com>
@jschwinger233 jschwinger233 force-pushed the pr/gray/cli-skip-cnp-for-conn-disrupt-in-1.15 branch from c50d65f to 0ac7eba Compare December 18, 2024 08:24
@jschwinger233 jschwinger233 changed the title ci/connectivity: Don't apply cnp to conn-disrupt pods in 1.15 ci/connectivity: Don't apply cnp to conn-disrupt pods in 1.15, 1.14 Dec 18, 2024
jschwinger233 added a commit that referenced this pull request Dec 18, 2024
@jschwinger233
ci-ipsec-upgrade: use cilium-cli-ci built from #36682
92e8309 
Signed-off-by: gray <gray.liang@isovalent.com>
@jschwinger233
Copy link
Member Author

/test

@jschwinger233 jschwinger233 added the release-note/misc This PR makes changes that have no direct user impact. label Dec 18, 2024
@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 18, 2024
@jschwinger233 jschwinger233 marked this pull request as ready for review December 18, 2024 13:28
@jschwinger233 jschwinger233 requested a review from a team as a code owner December 18, 2024 13:28
@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 Dec 18, 2024
@sayboras sayboras added this pull request to the merge queue Dec 19, 2024
Merged via the queue into main with commit 596cc3b Dec 19, 2024
219 of 220 checks passed
@sayboras sayboras deleted the pr/gray/cli-skip-cnp-for-conn-disrupt-in-1.15 branch December 19, 2024 05:02
This was referenced Dec 23, 2024
Merged
Merged
Merged
Merged
@github-actions github-actions bot mentioned this pull request Jan 13, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christarazi christarazi christarazi approved these changes

Assignees
No one assigned
Labels
cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jschwinger233 @christarazi @sayboras