Skip to content

Conversation

giorio94
Copy link
Member

@giorio94 giorio94 commented Feb 19, 2025

Add a step at the end of the egress gateway scale test to measure the network performance for pod to external endpoint traffic, potentially flowing through the egress gateway. More specifically, we leverage the cilium connectivity perf tool and configure it to deploy the client on one of the cluster nodes (in pod network), and the server on the external node (in host network). The egress gateway policy is updated to additionally match the client pod.

@giorio94 giorio94 added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. feature/egress-gateway Impacts the egress IP gateway feature. labels Feb 19, 2025
@github-actions github-actions bot added the cilium-cli This PR contains changes related with cilium-cli label Feb 19, 2025
@giorio94 giorio94 force-pushed the pr/giorio94/main/scale-test-egw-2 branch 3 times, most recently from 1ddd00a to 447532b Compare February 20, 2025 17:17
It is never used, so let's save a bit of time by not enabling it.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 force-pushed the pr/giorio94/main/scale-test-egw-2 branch 2 times, most recently from 0a1f9cf to 8016f29 Compare February 27, 2025 14:09
@giorio94
Copy link
Member Author

/scale-egw

Let's leverage the dedicated egw_scale_test_failed_tests_total metric.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
Add a step at the end of the egress gateway scale test to measure
the network performance for pod to external endpoint traffic,
potentially flowing through the egress gateway. More specifically,
we leverage the cilium connectivity perf tool and configure it to
deploy the client on one of the cluster nodes (in pod network),
and the server on the external node (in host network). The egress
gateway policy is updated to additionally match the client pod.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 force-pushed the pr/giorio94/main/scale-test-egw-2 branch from 8016f29 to 62abf45 Compare February 27, 2025 15:02
@giorio94
Copy link
Member Author

/scale-egw

@giorio94
Copy link
Member Author

/test

@giorio94 giorio94 requested a review from marseel February 27, 2025 16:38
@giorio94 giorio94 marked this pull request as ready for review February 27, 2025 16:38
@giorio94 giorio94 requested review from a team as code owners February 27, 2025 16:38
@giorio94 giorio94 requested review from aanm and christarazi February 27, 2025 16:38
@giorio94 giorio94 enabled auto-merge February 27, 2025 16:38
Copy link
Contributor

@marseel marseel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks amazing, thanks!

@giorio94 giorio94 added this pull request to the merge queue Feb 28, 2025
@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 Feb 28, 2025
Merged via the queue into main with commit 95cdbff Feb 28, 2025
300 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake cilium-cli This PR contains changes related with cilium-cli feature/egress-gateway Impacts the egress IP gateway feature. 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