Skip to content

Commit

Permalink
Fix regular expression for firewall rules.
Browse files Browse the repository at this point in the history
Previous regular expression did not match typical names used by CI, e.g. 'test-allow-docker-7340-0'.
New one does.
  • Loading branch information
marccarre committed Jan 13, 2017
1 parent c5c23ce commit bbe3844
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scheduler/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ def avg(test):
schedule = Schedule.get_or_insert(schedule_id, shards=shards)
return flask.json.jsonify(tests=schedule.shards[str(shard)])

FIRE_RE = re.compile(r'^(?P<network>\d+)-allow-(?P<type>\d+)-(?P<build>\d+)-(?P<shard>\d+)$')
FIRE_RE = re.compile(r'^(?P<network>\w+)-allow-(?P<type>\w+)-(?P<build>\d+)-(?P<shard>\d+)$')
NAME_REGEXES = [
re.compile(r'^host(?P<index>\d+)-(?P<build>\d+)-(?P<shard>\d+)$'),
re.compile(r'^test-(?P<build>\d+)-(?P<shard>\d+)-(?P<index>\d+)$'),
Expand Down

0 comments on commit bbe3844

Please sign in to comment.