Skip to content

Conversation

rastislavs
Copy link
Contributor

BGP tests need to run sequentially since BGP CP uses cluster-scoped CRDs and the test inrfra uses same node-without-cilium to peer with. Run them sequentially and deploy the test FRR instance only when test concurrency is disabled.

@rastislavs rastislavs added area/CI Continuous Integration testing issue or flake area/cli Impacts the command line interface of any command in the repository. release-note/ci This PR makes changes to the CI. area/bgp Impacts the Border Gateway Protocol feature. labels Nov 4, 2024
@github-actions github-actions bot added cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary labels Nov 4, 2024
@rastislavs
Copy link
Contributor Author

/test

@rastislavs rastislavs marked this pull request as ready for review November 13, 2024 20:38
@rastislavs rastislavs requested review from a team as code owners November 13, 2024 20:38
BGP tests need to run sequentially since BGP CP uses cluster-scoped
CRDs and the test inrfra uses same node-without-cilium to peer with.
Run them sequentially and deploy the test FRR instance only when
test concurrency is disabled.

Signed-off-by: Rastislav Szabo <rastislav.szabo@isovalent.com>
@rastislavs
Copy link
Contributor Author

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 18, 2024
@tklauser tklauser added this pull request to the merge queue Nov 19, 2024
Merged via the queue into cilium:main with commit 6350e78 Nov 19, 2024
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/bgp Impacts the Border Gateway Protocol feature. area/CI Continuous Integration testing issue or flake area/cli Impacts the command line interface of any command in the repository. cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary 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