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

Improved scaling speed of AzDO pipelines #3729

Merged
merged 4 commits into from
Oct 10, 2022
Merged

Improved scaling speed of AzDO pipelines #3729

merged 4 commits into from
Oct 10, 2022

Conversation

Eldarrin
Copy link
Contributor

@Eldarrin Eldarrin commented Oct 7, 2022

azure_pipelines_scalar.go has been modified to use a struct for the ADO JobRequest API to remove reliance on interface{} and a pre-run step has been introduced to remove previously completed jobs so only working jobs getting analysed for scaling

Checklist

  • When introducing a new scaler, I agree with the scaling governance policy
  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Tests have been added
  • A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
  • A PR is opened to update the documentation on (repo) (if applicable)
  • Changelog has been updated and is aligned with our changelog requirements

Fixes #

Issue (#3702)

Relates to #

@Eldarrin Eldarrin requested a review from a team as a code owner October 7, 2022 11:15
@JorTurFer
Copy link
Member

JorTurFer commented Oct 10, 2022

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

@JorTurFer JorTurFer enabled auto-merge (squash) October 10, 2022 11:49
@JorTurFer
Copy link
Member

Thanks a ton! Once the e2e test passes, the PR will be auto-merged

@JorTurFer JorTurFer merged commit 0e78994 into kedacore:main Oct 10, 2022
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.

2 participants