Skip to content

stress test addons version check #4543

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.

Sign up for GitHub

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

Merged
2 commits merged into from
Oct 28, 2022
Merged

stress test addons version check #4543

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
632167d
stress test addons version check
ckairen Oct 27, 2022
d8a2e38
cleanup
ckairen Oct 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions eng/common/scripts/stress-testing/find-all-stress-packages.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ function FindStressPackages(
}
foreach ($chartFile in $chartFiles) {
$chart = ParseChart $chartFile

VerifyAddonsVersion $chart
if (matchesAnnotations $chart $filters) {
$matrixFilePath = (Join-Path $chartFile.Directory.FullName $MatrixFileName)
if (Test-Path $matrixFilePath) {
Expand Down Expand Up @@ -73,6 +75,15 @@ function MatchesAnnotations([hashtable]$chart, [hashtable]$filters) {
return $true
}

function VerifyAddonsVersion([hashtable]$chart) {
foreach ($dependency in $chart.dependencies) {
if ($dependency.name -eq "stress-test-addons" -and
$dependency.version -lt "0.2.0") {
throw "The stress-test-addons version in use is $($dependency.version), please use versions >= 0.2.0"
}
}
}

function GetUsername() {
# Check GITHUB_USER for users in codespaces environments, since the default user is `codespaces` and
# we would like to avoid namespace overlaps for different codespaces users.
Expand Down
6 changes: 3 additions & 3 deletions tools/stress-cluster/chaos/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,8 +275,9 @@ For kubernetes manifests in the stress test helm chart `templates` directory tha
`stress-test-addons` (see [examples](#job-manifest)[below](#chaos-manifest)) templates, several special helper fields
are made available in the template context.

- `{{ .Values.image }}`
- The docker image published by the stress test deploy script
- `{{ .Stress.imageTag }}`
- The docker image published by the stress test deploy script.
- The docker image is referenced from the [scenarios-matrix.yaml](#scenarios-and-scenarios-matrixyaml).
- `{{ .Stress.Scenario }}`
- If using [Scenarios](#scenarios-and-scenarios-matrixyaml), this value maps to the individual scenario for which a
template is being generated.
Expand Down Expand Up @@ -401,7 +402,6 @@ Under the hood, the stress test tools will duplicate the job yaml for each scena
For example, given a stress test package with multiple tests represented as separate files each running on a different docker image:

```
values.yaml
templates/
src/
scenarioLongRunning.js
Expand Down