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

Remove duplicate self-hosted condition (#1016) 

Duplicate condition caused after merge of #953 and #1012
This commit is contained in:
Felipe Galindo Sanchez 2021-12-20 16:08:21 -08:00 committed by GitHub
parent 4ebec38208
commit 608c56936e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
controllers/horizontal_runner_autoscaler_webhook.go
View File

@ -655,9 +655,6 @@ HRA:
// Ensure that the RunnerSet-managed runners have all the labels requested by the workflow_job. // Ensure that the RunnerSet-managed runners have all the labels requested by the workflow_job.
for _, l := range labels { for _, l := range labels {
if l == "self-hosted" {
continue // label is automatically added to self-hosted runners
}
var matched bool var matched bool
// ignore "self-hosted" label as all instance here are self-hosted // ignore "self-hosted" label as all instance here are self-hosted
@ -689,9 +686,6 @@ HRA:
// Ensure that the RunnerDeployment-managed runners have all the labels requested by the workflow_job. // Ensure that the RunnerDeployment-managed runners have all the labels requested by the workflow_job.
for _, l := range labels { for _, l := range labels {
if l == "self-hosted" {
continue // label is automatically added to self-hosted runners
}
var matched bool var matched bool
// ignore "self-hosted" label as all instance here are self-hosted // ignore "self-hosted" label as all instance here are self-hosted