Skip to content

Conversation

@mikkeloscar
Copy link
Contributor

@mikkeloscar mikkeloscar commented Apr 29, 2025

This extends the existing load test to test internal path in the cluster e.g. to test the pod network via the path illustrated below:

image

The purpose of this is to test the pod network during a cluster upgrade e.g. during flannel rotation which can cause a short drop of network availability (#9272).

To make the test work it further improves the load test

  • Fix checking if daemonset pods are up to date in the wait-for-update.py script
  • Prevent scale-down of the vegeta/load test client to ensure it always running also when flannel is rotated.
  • Use kubectl apply instead of create to make it simpler to run start-load-test.sh locally.

TODO

  • Drop flannel change which is currently there to validate the test would catch it.

@mikkeloscar mikkeloscar added the minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call. label Apr 29, 2025
@MustafaSaber
Copy link
Member

👍

@mikkeloscar mikkeloscar force-pushed the loadtest-internal-path branch 2 times, most recently from 18fef65 to e51c266 Compare May 6, 2025 12:16
@mikkeloscar mikkeloscar force-pushed the loadtest-internal-path branch 3 times, most recently from 9d190db to 7be0770 Compare May 12, 2025 12:14
@mikkeloscar mikkeloscar force-pushed the loadtest-internal-path branch from 7be0770 to 62e3057 Compare July 3, 2025 13:34
Signed-off-by: Mikkel Oscar Lyderik Larsen <[email protected]>
Signed-off-by: Mikkel Oscar Lyderik Larsen <[email protected]>
@mikkeloscar mikkeloscar force-pushed the loadtest-internal-path branch 2 times, most recently from a5ecbc7 to 55eca2a Compare July 7, 2025 07:38
@mikkeloscar
Copy link
Contributor Author

👍

1 similar comment
@demonCoder95
Copy link
Member

👍

@demonCoder95 demonCoder95 merged commit b11819f into dev Jul 8, 2025
15 checks passed
@demonCoder95 demonCoder95 deleted the loadtest-internal-path branch July 8, 2025 12:10
@k8s-on-aws-manager-app k8s-on-aws-manager-app bot mentioned this pull request Jul 8, 2025
@k8s-on-aws-manager-app k8s-on-aws-manager-app bot mentioned this pull request Jul 9, 2025
@k8s-on-aws-manager-app k8s-on-aws-manager-app bot mentioned this pull request Jul 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged/alpha merged/beta merged/stable minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants