Skip to content

Commit

Permalink
hasAutobake/hasBigtest - exact match
Browse files Browse the repository at this point in the history
While the current matches are exact, there was risk
in "in" as a comparision matching those builder names
with suffixes.

As this pattern was copied to other build conditions, lets
keep a good pattern to copy.
  • Loading branch information
grooverdan committed Dec 20, 2024
1 parent 1814e33 commit ba762e5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -553,7 +553,7 @@ def waitIfStaging(step):
def hasAutobake(props):
builderName = props.getProperty("buildername")
for b in builders_autobake:
if builderName in b:
if builderName == b:
return True
return False

Expand All @@ -580,7 +580,7 @@ def hasBigtest(props):
builderName = str(props.getProperty("buildername"))

for b in builders_big:
if builderName in b:
if builderName == b:
return True
return False

Expand Down

0 comments on commit ba762e5

Please sign in to comment.