-
Notifications
You must be signed in to change notification settings - Fork 3.3k
gh/workflows: Add IPsec key rotation action and use it in ci-eks / ci-ipsec-e2e #29704
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
Conversation
/ci-eks |
/ci-ipsec-e2e |
/ci-eks |
/ci-ipsec-e2e |
/ci-eks |
7 similar comments
/ci-eks |
/ci-eks |
/ci-eks |
/ci-eks |
/ci-eks |
/ci-eks |
/ci-eks |
8e00f2b
to
2f82a5f
Compare
/ci-eks |
1 similar comment
/ci-eks |
fb5ef81
to
d1ba4ab
Compare
/ci-eks |
3 similar comments
/ci-eks |
/ci-eks |
/ci-eks |
1c07081
to
99f46e1
Compare
/ci-eks |
99f46e1
to
0f369ed
Compare
/ci-eks |
0f369ed
to
1a430b6
Compare
/ci-eks |
/ci-ipsec-e2e |
First, this commit includes the IPsec key rotation tests action. Second, it changes the CLI exec name and path to "./cilium-cli", so that it can be used by the key rotation action and friends. Third, it runs the IPsec tests only if the matrix.ipsec is set to "true". A subsequent commit will extend the matrix configuration accordingly. Signed-off-by: Martynas Pumputis <m@lambda.lt>
The file name is non-ideal, but changing it would require changing many files :-( For each PR we will run 1.25 w/o IPsec and 1.28 w/ IPsec. Signed-off-by: Martynas Pumputis <m@lambda.lt>
4c62f3c
to
ee1e9ee
Compare
/test |
All passed after rerunning. |
@brb do you think the backport to v1.13 is doable / worth it? When I had a quick look a while ago, it seemed like we are missing a bunch of pre-req GH workflow PRs ... |
Yep, quite a few things need to be backported too. It's fine to skip considering that v1.13 will be EOL once we release v1.16. |
See commit msgs.