-
Notifications
You must be signed in to change notification settings - Fork 3.2k
CI: Add conn-disrupt-test action for reuse #27567
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
CI: Add conn-disrupt-test action for reuse #27567
Conversation
This commit makes conn-disrupt-test a github action, so upgrade test and IPsec key rotation test don't have to copy and paste everywhere. The idea is to allow caller workflow to specify the commands to execute, then this action will follow the steps: 1. Run "cilium-cli connectivity test --conn-disrupt-test-setup"; 2. Run whatever caller workflow passes: could be upgrade operation or IPsec key rotation; 3. Run "cilium-cli connectivity test --include-conn-disrupt-test"; Signed-off-by: Zhichuan Liang <gray.liang@isovalent.com>
@jschwinger233 @brb it looks like @brb was expecting to give a review, but you've set |
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I created #27647 for the flake affecting ConformanceGatewayAPI. It's clearly unrelated as the present PR only introduces a new GitHub Action (not even workflow) which isn't used anywhere yet. Marking ready to merge. |
This commit makes conn-disrupt-test a github action, so upgrade test and IPsec key rotation test don't have to copy and paste everywhere.
The idea is to allow caller workflow to specify the commands to execute, then this action will follow the steps:
Signed-off-by: Zhichuan Liang gray.liang@isovalent.com