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

workflows: Pin conn-disrupt-test GH action to main #29402

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

pchaigno
Copy link
Member

If we use a specific SHA, then Renovate will keep bothering us, trying to update it every time we push to main. The intent is for those workflows to use the main version of this action anyway.

If we use a specific SHA, then Renovate will keep bothering us, trying
to update it every time we push to main. The intent is for those
workflows to use the main version of this action anyway.

Suggested-by: Gray Lian <gray.liang@isovalent.com>
Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno pchaigno added area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-note/ci This PR makes changes to the CI. labels Nov 27, 2023
@pchaigno pchaigno requested review from a team as code owners November 27, 2023 12:55
@pchaigno pchaigno enabled auto-merge November 27, 2023 12:58
@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 27, 2023
@youngnick youngnick disabled auto-merge November 28, 2023 03:29
@youngnick youngnick merged commit bf42b25 into cilium:main Nov 28, 2023
@pchaigno pchaigno deleted the pin-ipsec-action-to-main branch November 28, 2023 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants