Skip to content

.github/ipsec-key-rotation: shorten to 1m#40032

Merged
aanm merged 1 commit intomainfrom
pr/aanm/speedupci
Jun 16, 2025
Merged

.github/ipsec-key-rotation: shorten to 1m#40032
aanm merged 1 commit intomainfrom
pr/aanm/speedupci

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Jun 13, 2025 •
edited
Loading

There's no point on having the IPSec Key rotation to take up to 5
minutes. We can shorten its time to 1 minute to speed up CI a little
bit.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 13, 2025
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 13, 2025

/ci-ipsec-e2e

@aanm aanm force-pushed the pr/aanm/speedupci branch from 5db74fa to 9b0cadb Compare June 13, 2025 18:41
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 13, 2025

/ci-ipsec-e2e

@aanm aanm force-pushed the pr/aanm/speedupci branch from 9b0cadb to 9da57f3 Compare June 16, 2025 07:59
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 16, 2025

/ci-ipsec-e2e

@aanm aanm force-pushed the pr/aanm/speedupci branch from 9da57f3 to 8d5b3e4 Compare June 16, 2025 09:30
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 16, 2025

/ci-ipsec-e2e

@aanm
.github/ipsec-key-rotation: shorten to 1m
ba65f98 
There's no point on having the IPSec Key rotation to take up to 5
minutes. We can shorten its time to 1 minute to speed up CI a little
bit.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/aanm/speedupci branch from 8d5b3e4 to ba65f98 Compare June 16, 2025 10:36
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 16, 2025

/test

@aanm aanm changed the title test .github/ipsec-key-rotation: shorten to 1m Jun 16, 2025
@aanm aanm added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. feature/ipsec Relates to Cilium's IPsec feature labels Jun 16, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jun 16, 2025
@aanm aanm added needs-backport/1.16 needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch labels Jun 16, 2025
@aanm aanm requested review from a team and ldelossa and removed request for a team June 16, 2025 10:38
@aanm aanm marked this pull request as ready for review June 16, 2025 10:38
@aanm aanm requested review from a team as code owners June 16, 2025 10:38
@aanm aanm requested a review from viktor-kurchenko June 16, 2025 10:38
@aanm aanm enabled auto-merge June 16, 2025 10:39
@aanm aanm added this pull request to the merge queue Jun 16, 2025
Merged via the queue into main with commit 44faced Jun 16, 2025
317 of 322 checks passed
@aanm aanm deleted the pr/aanm/speedupci branch June 16, 2025 14:12
@joamaki joamaki mentioned this pull request Jun 17, 2025
Merged
4 tasks
@joamaki joamaki mentioned this pull request Jun 17, 2025
Merged
5 tasks
@joamaki joamaki added backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. and removed needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch labels Jun 17, 2025
@aanm aanm removed backport-pending/1.16 backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. labels Jun 18, 2025
@julianwiedmann julianwiedmann mentioned this pull request Nov 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@viktor-kurchenko viktor-kurchenko viktor-kurchenko approved these changes

@ldelossa ldelossa Awaiting requested review from ldelossa ldelossa was automatically assigned from cilium/ipsec

Assignees

No one assigned

Labels

area/CI Continuous Integration testing issue or flake feature/ipsec Relates to Cilium's IPsec feature release-note/ci This PR makes changes to the CI.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@aanm @viktor-kurchenko @joamaki @msune