actions/actions-runner-controller
1
0
Fork 0
mirror of https://github.com/actions/actions-runner-controller.git synced 2025-12-11 12:06:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: case-insensitive webhook label matching (#2302)

Browse Source 
Co-authored-by: Yusuke Kuoka <ykuoka@gmail.com>
This commit is contained in:
Andrei Vydrin
2023-02-21 07:37:42 +07:00
committed by GitHub
parent 4103fe35df
commit 44c06c21ce
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -22,6 +22,7 @@ import (
"fmt" "fmt"
"io" "io"
"net/http" "net/http"
"strings"
"sync" "sync"
"time" "time"
@@ -608,7 +609,7 @@ HRA:
// TODO labels related to OS and architecture needs to be explicitly declared or the current implementation will not be able to find them. // TODO labels related to OS and architecture needs to be explicitly declared or the current implementation will not be able to find them.
for _, l2 := range rs.Spec.Labels { for _, l2 := range rs.Spec.Labels {
if l == l2 { if strings.EqualFold(l, l2) {
matched = true matched = true
break break
} }
@@ -639,7 +640,7 @@ HRA:
// TODO labels related to OS and architecture needs to be explicitly declared or the current implementation will not be able to find them. // TODO labels related to OS and architecture needs to be explicitly declared or the current implementation will not be able to find them.
for _, l2 := range rd.Spec.Template.Spec.Labels { for _, l2 := range rd.Spec.Template.Spec.Labels {
if l == l2 { if strings.EqualFold(l, l2) {
matched = true matched = true
break break
} }