Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support poolName or poolID validation in Azure Pipelines Scaler #2370

Merged
merged 14 commits into from
Jan 3, 2022

Conversation

JorTurFer
Copy link
Member

@JorTurFer JorTurFer commented Nov 30, 2021

Signed-off-by: jorturfer jorge_turrado@hotmail.es

Provide a description of what has been changed

Checklist

Relates to #2342

Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
@tomkerkhove
Copy link
Member

Would you mind adding an issue for this @JorTurFer? Instead of using one or the other, maybe we should allow them to configure both and use what they want to use?

@tomkerkhove
Copy link
Member

Ok this is probably related to #2342

@JorTurFer
Copy link
Member Author

Ok this is probably related to #2342

Yes it is, I forgot adding the link :(

@JorTurFer JorTurFer added this to the v2.6.0 milestone Dec 1, 2021
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
@JorTurFer JorTurFer changed the title WIP - Use poolName instead of poolID in Azure Pipelines Scaler WIP - Support poolName or poolID validation in Azure Pipelines Scaler Dec 2, 2021
@JorTurFer JorTurFer changed the title WIP - Support poolName or poolID validation in Azure Pipelines Scaler Support poolName or poolID validation in Azure Pipelines Scaler Dec 13, 2021
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
Signed-off-by: jorturfer <jorge_turrado@hotmail.es>
@JorTurFer
Copy link
Member Author

JorTurFer commented Dec 14, 2021

/run-e2e azure-pipelines.test*
Update: You can check the progres here

@JorTurFer JorTurFer marked this pull request as ready for review December 14, 2021 07:18
@JorTurFer JorTurFer requested a review from a team as a code owner December 14, 2021 07:18
Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

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

Nit in Changelog, the rest looks good.

Jorge Turrado and others added 2 commits January 3, 2022 12:06
Co-authored-by: Zbynek Roubalik <726523+zroubalik@users.noreply.github.com>

Signed-off-by: Jorge Turrado <jorge.turrado@docplanner.com>
@zroubalik zroubalik merged commit 71d6c90 into kedacore:main Jan 3, 2022
@JorTurFer JorTurFer deleted the azure_devops branch January 3, 2022 14:21
alex60217101990 pushed a commit to dysnix/keda that referenced this pull request Jan 10, 2022
…edacore#2370)

Signed-off-by: Jorge Turrado <jorge.turrado@docplanner.com>

Co-authored-by: Zbynek Roubalik <726523+zroubalik@users.noreply.github.com>
Signed-off-by: alex60217101990 <alex6021710@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants