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

backport v1.14: IPsec upgrade tests #27175

Merged
merged 3 commits into from Aug 3, 2023

Conversation

brb
Copy link
Member

@brb brb commented Aug 1, 2023

Manual backport of #26983 and #27150.

@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Aug 1, 2023
@brb brb force-pushed the pr/brb/v1.14-ci-ipsec-upgrade branch from 7f33773 to 55570f0 Compare August 2, 2023 05:57
@brb
Copy link
Member Author

brb commented Aug 2, 2023

/ci-ipsec-upgrade

brb added 3 commits August 2, 2023 15:50
[ upstream commit e695db5 ]

This commit adds an option IPsec e2e upgrade test. The test does the
following:

* Install the latest stable release version of Cilium.
* Run the CLI's connectivity tests.
* Upgrade Cilium to a PR version.
* Run the CLI's connectivity tests.
* Downgrade to the previous version.
* Run the CLI's connectivity tests.

After each connectivity test case we flush CT to avoid the interference
[1][2] after running L7 proxy tests.

[1]: cilium/cilium-cli#367
[2]: #17459

The test runs pods which established long-lived connections.
The test checks whether they are not interrupted during upgrade /
downgrade.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
[ upstream commit ab8b413 ]

Fixes: e695db5 ("ci-ipsec-upgrade: Add IPsec upgrade tests")
Signed-off-by: Martynas Pumputis <m@lambda.lt>
[ upstream commit 031d886 ]

v1.13 -> PR build -> v1.13

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb force-pushed the pr/brb/v1.14-ci-ipsec-upgrade branch from 55570f0 to 426acb9 Compare August 2, 2023 13:52
@brb brb changed the title WIP: v1.14 tests-ipsec-upgrade backport v1.14: IPsec upgrade tests Aug 2, 2023
@brb
Copy link
Member Author

brb commented Aug 2, 2023

/ci-ipsec-upgrade

@brb brb marked this pull request as ready for review August 2, 2023 14:31
@brb brb requested review from a team as code owners August 2, 2023 14:31
@brb brb requested review from brlbil and pchaigno August 2, 2023 14:31
Copy link
Member

@jschwinger233 jschwinger233 left a comment

Choose a reason for hiding this comment

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

Thanks!

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 3, 2023
@dylandreimerink dylandreimerink merged commit e726b9f into v1.14 Aug 3, 2023
48 checks passed
@dylandreimerink dylandreimerink deleted the pr/brb/v1.14-ci-ipsec-upgrade branch August 3, 2023 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants