Add stress test project template and documentation #38159
Conversation
github-actions
bot
added
the
EngSys
| mvn -f /stress/sdk/tools/pom.xml clean install -Dcheckstyle.skip -Dgpg.skip -Dmaven.javadoc.skip -Drevapi.skip -Dspotbugs.skip -Djacoco.skip -DskipTests && \ | ||
| mvn -f /stress/sdk/template/azure-template-stress/pom.xml clean install -Dcheckstyle.skip -Dgpg.skip -Dmaven.javadoc.skip -Drevapi.skip -Dspotbugs.skip -Djacoco.skip -DskipTests | ||
|
|
||
| FROM mcr.microsoft.com/openjdk/jdk:11-mariner |
There was a problem hiding this comment.
Future work that'll be useful: targeting the min version of JDK you support, max, etc...
lmolkova
force-pushed
the
stress-template
branch
from
4de4bff to
dd764c4
Compare
lmolkova
requested review from
hallipr,
weshaggard and
JimSuplizio
as code owners
| version: 0.1.1 | ||
| appVersion: v0.1 | ||
| annotations: | ||
| stressTest: 'false' # change it to true. This enables auto-discovery of this test via `find-all-stress-packages.ps1` |
There was a problem hiding this comment.
How come we are defaulting to false here? To prevent auto-discovery during development? It seems like it could be an odd behavior to debug if the script isn't finding one's test. Or is it to make sure someone edits this file?
There was a problem hiding this comment.
this is a template/example project which exist for documentation purposes. While the test is still useful, we don't really want to run it in stress pipeline
| - update test parameters for `test` container | ||
| 6. Define scenarios and parameters in `scenarios-matrix.yaml` | ||
|
|
||
| Now you're ready to run tests with `.\eng\common\scripts\stress-testing\deploy-stress-tests.ps1 -SearchDirectory .\sdk\<your service directory>`. |
There was a problem hiding this comment.
| Now you're ready to run tests with `.\eng\common\scripts\stress-testing\deploy-stress-tests.ps1 -SearchDirectory .\sdk\<your service directory>`. | |
| Now you're ready to run tests with `./eng/common/scripts/stress-testing/deploy-stress-tests.ps1 -SearchDirectory ./sdk/<your service directory>` |
| # simple and fast .NET Core HTTP server to run tests against | ||
| # When writing real stress test, you probably won't need it and would use corresponding Azure Service. | ||
| - name: server | ||
| image: stresspgs7b6dif73rup6.azurecr.io/stress/simplehttpserver |
There was a problem hiding this comment.
Is it safe to assume this will always be replaced? If not I should add a registry value to the helm context so we don't have to take a dependency on what is a cluster-specific ACR.
There was a problem hiding this comment.
Yes, this container is not needed for real SDK tests and only exists for demo purposes. At some point we probably will add azure-core tests where we'd need HHTP server, but would probably use storage with SAS connection string instead.
|
Love this! I have some minor nits and questions but LGTM, excited to see some templatization. Agree w/ Richard I would be happy to support making this a cross-language effort as well. |
lmolkova
force-pushed
the
stress-template
branch
from
5755cc0 to
678e7a7
Compare
lmolkova
force-pushed
the
stress-template
branch
from
d733c21 to
f80f8fc
Compare
No description provided.