Commit 4cfbef5f authored by Takuto Ikuta's avatar Takuto Ikuta Committed by Chromium LUCI CQ

trigger_scripts: remove unnecessary code path

It seems --shard-index is always specified now.

This is partial re-land of https://crrev.com/c/2639441

Bug: 1127205
Cq-Include-Trybots: luci.chromium.try:chromeos-kevin-rel
Change-Id: If26fc49d61581635ae9260805d982768e285823a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640526
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: default avatarBrian Sheedy <bsheedy@chromium.org>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845716}
parent bb4118dd
...@@ -65,18 +65,10 @@ def parse_args(triggerer): ...@@ -65,18 +65,10 @@ def parse_args(triggerer):
dest='optional_dimensions', dest='optional_dimensions',
help='Optional dimensions which will result in additional task slices. ' help='Optional dimensions which will result in additional task slices. '
'Duplicated from the `swarming.py trigger` command.') 'Duplicated from the `swarming.py trigger` command.')
# BaseTestTriggerer's setup_parser_contract() takes care of adding needed base_test_triggerer.BaseTestTriggerer.setup_parser_contract(parser)
# swarming.py args if they're not already present. But only do this if args, additional_args = parser.parse_known_args()
# '--shard-index' is passed in. (The exact usage of trigger scripts are additional_args = triggerer.modify_args(
# currently changing. See crbug.com/926987 for more info.) additional_args, 0, args.shard_index, args.shards, args.dump_json)
if '--shard-index' in sys.argv:
base_test_triggerer.BaseTestTriggerer.setup_parser_contract(parser)
args, additional_args = parser.parse_known_args()
additional_args = triggerer.modify_args(
additional_args, 0, args.shard_index, args.shards, args.dump_json)
else:
base_test_triggerer.BaseTestTriggerer.add_use_swarming_go_arg(parser)
args, additional_args = parser.parse_known_args()
if additional_args[0] != 'trigger': if additional_args[0] != 'trigger':
parser.error( parser.error(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment