Skip to content

Remove sequential and only use concurrent tests#40103

Draft
aanm wants to merge 2 commits intomainfrom
pr/remove-ipsec-conformance
Draft

Remove sequential and only use concurrent tests#40103
aanm wants to merge 2 commits intomainfrom
pr/remove-ipsec-conformance

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Jun 18, 2025 •
edited
Loading

Remove sequential tests and only use concurrent tests.

@aanm aanm requested review from a team as code owners June 18, 2025 12:18
@aanm aanm added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Jun 18, 2025
@aanm aanm requested a review from ldelossa June 18, 2025 12:18
@aanm aanm added the feature/ipsec Relates to Cilium's IPsec feature label Jun 18, 2025
@aanm aanm requested a review from Artyop June 18, 2025 12:18
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 18, 2025

/test

@aanm aanm marked this pull request as draft June 18, 2025 13:02
@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 32c246d to c1f20ba Compare June 18, 2025 13:45
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 18, 2025

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from c1f20ba to 4a8c9c5 Compare June 18, 2025 15:24
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 18, 2025

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 4a8c9c5 to e215c84 Compare June 25, 2025 08:31
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 25, 2025

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch 2 times, most recently from 6f2af59 to eadc786 Compare June 25, 2025 10:39
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 25, 2025

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from eadc786 to 980e419 Compare June 25, 2025 13:31
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 25, 2025

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 980e419 to 10b82b5 Compare June 25, 2025 15:42
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 25, 2025

/ci-ipsec-e2e

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 10b82b5 to 08256d7 Compare June 25, 2025 20:00
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 25, 2025

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 08256d7 to 9646bfd Compare July 4, 2025 12:34
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jul 4, 2025

/test

@aanm aanm changed the title Remove conformance-ipsec-e2e.yaml in favor of upgrade workflow Remove sequential jobs and only use concurrent jobs Jul 9, 2025
@aanm aanm changed the title Remove sequential jobs and only use concurrent jobs Remove sequential and only use concurrent tests Jul 9, 2025
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Nov 27, 2025

/test

@aanm
Copy link
Copy Markdown
Member Author

aanm commented Nov 27, 2025 •
edited
Loading

Thank you @rastislavs it looks the bgp tests are fixed but now the egress tests are failing here 😭 /cc @tommyp1ckles

@aanm aanm assigned tommyp1ckles and unassigned rastislavs Nov 27, 2025
@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 01f0faf to b14b550 Compare November 27, 2025 10:35
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 29, 2025

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Dec 29, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 13, 2026

This pull request has not seen any activity since it was marked stale.
Closing.

@github-actions github-actions bot closed this Jan 13, 2026
@aanm aanm reopened this Feb 19, 2026
@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from b14b550 to 09516d6 Compare February 19, 2026 08:36
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Feb 19, 2026

/test

@tommyp1ckles tommyp1ckles self-requested a review February 19, 2026 17:10
@github-actions github-actions bot removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Feb 20, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 22, 2026

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Mar 22, 2026
@tommyp1ckles
Copy link
Copy Markdown
Contributor

tommyp1ckles commented Mar 30, 2026

/ci-e2e-upgrade

@tommyp1ckles
Copy link
Copy Markdown
Contributor

tommyp1ckles commented Mar 30, 2026

/test

@github-actions github-actions bot removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Mar 31, 2026
@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 09516d6 to 2c41c9d Compare April 1, 2026 14:39
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Apr 1, 2026

/test

@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 2c41c9d to 71ee692 Compare April 1, 2026 14:57
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Apr 1, 2026

/test

2 similar comments
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Apr 1, 2026

/test

@cilium-ariane
Copy link
Copy Markdown

cilium-ariane bot commented Apr 1, 2026

/test

aanm and others added 2 commits April 1, 2026 21:36
@aanm
cilium-cli: remove sequential tests
c9cf8be 
The sequential tests were introduced as they were causing the CI to be
flaky. Since they have been removed, they are no longer flaky and we can
run them in parallel again.

Signed-off-by: André Martins <andre@cilium.io>
@tommyp1ckles @aanm
cli: connectivity tests: only setup/teardown static routes once.
48d7c4b 
With the advent of concurrent e2e testing we no longer can have
individual tests racing over setting up & tearing down static
routes.

Instead, we check if the first ct requires the static routes.
If so, we enable them once and schedule the cleanup for later.

Currently the sets of connection tests are simply duplicates
of each other but running in different namespaces.
Therefore we only check the first set for the requirement.

Signed-off-by: Tom Hadlaw <tom.hadlaw@isovalent.com>
@aanm aanm force-pushed the pr/remove-ipsec-conformance branch from 71ee692 to 48d7c4b Compare April 1, 2026 19:36
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Apr 1, 2026

/test

1 similar comment
@cilium-ariane
Copy link
Copy Markdown

cilium-ariane bot commented Apr 1, 2026

/test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tamilmani1989 tamilmani1989 tamilmani1989 approved these changes

@ldelossa ldelossa ldelossa approved these changes

@Artyop Artyop Awaiting requested review from Artyop Artyop is a code owner automatically assigned from cilium/github-sec

@pippolo84 pippolo84 Awaiting requested review from pippolo84 pippolo84 is a code owner automatically assigned from cilium/egress-gateway

@youngnick youngnick Awaiting requested review from youngnick youngnick is a code owner automatically assigned from cilium/sig-servicemesh

@harsimran-pabla harsimran-pabla Awaiting requested review from harsimran-pabla harsimran-pabla is a code owner automatically assigned from cilium/sig-bgp

@tommyp1ckles tommyp1ckles Awaiting requested review from tommyp1ckles

At least 0 approving reviews are required to merge this pull request.

Assignees

@tommyp1ckles tommyp1ckles

Labels

area/CI Continuous Integration testing issue or flake cilium-cli This PR contains changes related with cilium-cli 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.

7 participants

@aanm @tommyp1ckles @harsimran-pabla @rastislavs @tamilmani1989 @ldelossa @msune