Open arschles opened 3 years ago
@ajanth97 are you interested in implementing either one of these at some point?
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.
@arschles sorry I just saw this. yeah will do when I get some time
sounds good @ajanth97 - thanks!
I am currently working on this.
e2e test are executed on every PR/Push https://github.com/kedacore/http-add-on/pull/564
should we close this one if done?
Stress tests are still pending, we could cose this and open another issue for stress test if we want to have it more accuracy tracked
OK, thanks Jorge. That;s OK. We can keep it what it is.
@MrsDaehin , may you be interested in help with this too? (after core load tests are ready, ofc) This is easier to test indeed because this is closer to an API (and here the load should be higher)
we can discuss this. It will be easy once we got k6 running.
@arschles, since you proposed the definition of a stress test, shouldn't we stipulate a criteria for "acceptance"? like, "given that every request will take 1sec at the backend, for every pod of the http interceptor with 0.1 cpu and 128mb, the stress test will fail if: the interceptor crashes, the interceptor gets OOM killed, less than X requests are handled in parallel at any given time, less than Y requests are handled over the period of 10min" and so on?
We currently have mostly unit tests inside this repository, along with a few integration tests (like one for the interceptor coming in #206). I propose that we add more testing strategies to this project, including:
HTTPScaledObject
, and runs a quick suite of requests to trigger scale-from-zero, scale up, and scale back down to zero. this test should check that the app and the interceptor both scale properly, and that the external scaler, operator, and interceptor's admin endpoints all report the expected things (NOTE: KEDA does this, and there is a prototype of something similar in #206)