From 608c56936ef1a8e3f555fe62a590d937498c9a7f Mon Sep 17 00:00:00 2001 From: Felipe Galindo Sanchez Date: Mon, 20 Dec 2021 16:08:21 -0800 Subject: [PATCH] Remove duplicate self-hosted condition (#1016) Duplicate condition caused after merge of #953 and #1012 --- controllers/horizontal_runner_autoscaler_webhook.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/controllers/horizontal_runner_autoscaler_webhook.go b/controllers/horizontal_runner_autoscaler_webhook.go index 2bd3930a..f2fabd24 100644 --- a/controllers/horizontal_runner_autoscaler_webhook.go +++ b/controllers/horizontal_runner_autoscaler_webhook.go @@ -655,9 +655,6 @@ HRA: // Ensure that the RunnerSet-managed runners have all the labels requested by the workflow_job. for _, l := range labels { - if l == "self-hosted" { - continue // label is automatically added to self-hosted runners - } var matched bool // 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. for _, l := range labels { - if l == "self-hosted" { - continue // label is automatically added to self-hosted runners - } var matched bool // ignore "self-hosted" label as all instance here are self-hosted