Skip to content
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

azure-pipelines scaler allows configuration of requireAllDemands #4138 #4203

Merged
merged 5 commits into from Feb 8, 2023

Conversation

patst
Copy link
Contributor

@patst patst commented Feb 6, 2023

Signed-off-by: Patrick Steinig patrick.steinig@googlemail.com

Introduced a new attribute for the azure-pipelines scaler named requireAllDemands which only scales a if the configured capabilities of a trigger match exactly what a job demands.

This allows more exact scaling, e.g. by not reacting to empty demands for jobs.

Checklist

Fixes: #4138

Relates to documentation PR kedacore/keda-docs#1063

@patst patst requested a review from a team as a code owner February 6, 2023 20:03
@patst patst force-pushed the 4138-azuredevops-capabilities branch 2 times, most recently from ea1f829 to baf0bce Compare February 6, 2023 20:09
Copy link
Member

@JorTurFer JorTurFer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good
I have left some comments inline

pkg/scalers/azure_pipelines_scaler.go Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
pkg/scalers/azure_pipelines_scaler.go Outdated Show resolved Hide resolved
@patst
Copy link
Contributor Author

patst commented Feb 7, 2023

@JorTurFer thanks for the excellent review and comments. I applied them and changed the docs PR as well

patst and others added 4 commits February 7, 2023 20:12
…core#4138

Signed-off-by: Patrick Steinig <patrick.steinig@googlemail.com>
Signed-off-by: patst <patrick.steinig@googlemail.com>
Co-authored-by: Jorge Turrado Ferrero <Jorge_turrado@hotmail.es>
Signed-off-by: patst <patrick.steinig@googlemail.com>
Co-authored-by: Jorge Turrado Ferrero <Jorge_turrado@hotmail.es>
Signed-off-by: patst <patrick.steinig@googlemail.com>
Signed-off-by: patst <patrick.steinig@googlemail.com>
Copy link
Member

@JorTurFer JorTurFer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
Thanks for the improvement ❤️

Signed-off-by: Jorge Turrado Ferrero <Jorge_turrado@hotmail.es>
@JorTurFer
Copy link
Member

JorTurFer commented Feb 8, 2023

/run-e2e pipeline*
Update: You can check the progress here

@JorTurFer JorTurFer merged commit 4a63a18 into kedacore:main Feb 8, 2023
@patst patst deleted the 4138-azuredevops-capabilities branch September 19, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

azure-pipelines scaler: exact match demands (don't match jobs without capability)
2 participants