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.13: IPsec upgrade tests #27174

Merged
merged 3 commits into from Aug 3, 2023
Merged

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.13 This PR represents a backport for Cilium 1.13.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.13-ci-ipsec-upgrade branch 2 times, most recently from 0d1c99d to 1fbc862 Compare August 1, 2023 06:53
@brb
Copy link
Member Author

brb commented Aug 1, 2023

/ci-ipsec-upgrade

@brb brb force-pushed the pr/brb/v1.13-ci-ipsec-upgrade branch from 1fbc862 to 13c2cb7 Compare August 1, 2023 06:56
@brb
Copy link
Member Author

brb commented Aug 1, 2023

/ci-ipsec-upgrade

@brb brb force-pushed the pr/brb/v1.13-ci-ipsec-upgrade branch from 13c2cb7 to fc87b8e Compare August 2, 2023 13:48
@brb brb changed the title WIP: v1.13 tests-ipsec-upgrade backport v1.12: IPsec upgrade tests Aug 2, 2023
@brb brb changed the title backport v1.12: IPsec upgrade tests backport v1.13: 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
brb added 3 commits August 3, 2023 12:07
[ 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.

Backport NB: disabled the endpoint routes feature in the 3rd
configuration, as the documented limitation was fixed only in >=v1.13.

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.12 -> PR build -> v1.12

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 3, 2023
@aanm aanm merged commit ea5e2d3 into v1.13 Aug 3, 2023
35 checks passed
@aanm aanm deleted the pr/brb/v1.13-ci-ipsec-upgrade branch August 3, 2023 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.13 This PR represents a backport for Cilium 1.13.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