-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Scalability testing docs #39171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Scalability testing docs #39171
Conversation
ff7b1f2
to
202f3cd
Compare
202f3cd
to
a20a4ea
Compare
/test |
a20a4ea
to
1f5f937
Compare
/test |
c5d8b4c
to
12e7bec
Compare
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice doc!
Please find some grammar/formatting suggestions below.
My main concern is the usage of link to non-persistent test results (see near the end of the review, below).
12e7bec
to
a6d14b3
Compare
Thanks @qmonnet for amazing review, I've definitely learned a few tricks :) I went one more time through docs and fixed a couple of similar typos. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me now, thank you!
/test |
Enable agent prometheus metrics and pprof for local development. This allows for running scale test scenario locally as we are scraping agent metrics + profiling information. Additionally, expose /metrics endpoint in KCM and KS, as it is scrapped by CL2 automatically for kind environments. Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
Introduce ClusterLoader2, example scale-tests that we currently have, document current example use-cases. On top of that add documentation for local development of scale tests, how to access results in CI and how to debug scale test results. Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
a6d14b3
to
f1dcc22
Compare
/test |
Rebased as "Conformance Multi Pool IPAM" was failing consistently :/ |
/test |
Friendly ping @hemanthmalla @jshr-w :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the ping, this looks really awesome :) I didn't realize we collect profiling data from the scale tests this way, that's helpful to know for the future.
Let's start scalability testing documentation and also make it possible to run ClusterLoader2 easily in kind.