-
Notifications
You must be signed in to change notification settings - Fork 183
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
[stress] Add LockDeletionForDays parameter to set PodDisruptionBudget and cleanup job #7002
[stress] Add LockDeletionForDays parameter to set PodDisruptionBudget and cleanup job #7002
Conversation
The following pipelines have been queued for testing: |
tools/stress-cluster/cluster/kubernetes/stress-test-addons/templates/_pod_disruption_budget.tpl
Show resolved
Hide resolved
The following pipelines have been queued for testing: |
ec812d6
to
c16e1dd
Compare
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
33308f0
to
37f8b2c
Compare
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#7002 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Ben Broderick Phillips <[email protected]>
Locking
This adds a parameter
LockDeletionForDays
to the stress deploy script, which will prevent a test and all its pods from being killed for a time period by the kubernetes api server. Possible reasons a pod may be killed:It also creates a cron job resource set to delete the pod disruption budget after
LockDeletionForDays
days, so that evictions can then occur (meaning the test pods won't necessarily be killed, but they will no longer be locked).Parallel
This updates the parallel jobs feature to use the matrix config instead of a separate addons template. This is both more intuitive and allows for setting different parallel values per scenario (rather than per test).