public_git
/
actions-runner-controller
mirror of https://github.com/actions-runner-controller/actions-runner-controller.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Resolves the erroneous webhook scale down due to check runs (#2119) 

Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
This commit is contained in:
Nathan Klick 2023-02-20 19:56:46 -06:00 committed by GitHub
parent 44c06c21ce
commit ced88228fc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
controllers/actions.summerwind.net/horizontal_runner_autoscaler_webhook.go
View File

@ -210,9 +210,9 @@ func (autoscaler *HorizontalRunnerAutoscalerGitHubWebhook) Handle(w http.Respons
if e.GetAction() == "queued" { if e.GetAction() == "queued" {
target.Amount = 1 target.Amount = 1
break break
} else if e.GetAction() == "completed" && e.GetWorkflowJob().GetConclusion() != "skipped" { } else if e.GetAction() == "completed" && e.GetWorkflowJob().GetConclusion() != "skipped" && e.GetWorkflowJob().GetRunnerID() > 0 {
// A nagative amount is processed in the tryScale func as a scale-down request, // A negative amount is processed in the tryScale func as a scale-down request,
// that erasese the oldest CapacityReservation with the same amount. // that erases the oldest CapacityReservation with the same amount.
// If the first CapacityReservation was with Replicas=1, this negative scale target erases that, // If the first CapacityReservation was with Replicas=1, this negative scale target erases that,
// so that the resulting desired replicas decreases by 1. // so that the resulting desired replicas decreases by 1.
target.Amount = -1 target.Amount = -1