Skip to content

Commit

Permalink
teuthology-suite: no default value for suite branch
Browse files Browse the repository at this point in the history
By default suite branch must be equal to ceph one.

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@gmail.com>
  • Loading branch information
kshtsk committed Mar 20, 2019
1 parent a9f79f5 commit 8c37d3b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 0 additions & 2 deletions scripts/suite.py
Expand Up @@ -68,7 +68,6 @@
[default: qa]
--suite-branch <suite_branch>
Use this suite branch instead of the ceph branch
[default: {default_suite_branch}]
--suite-dir <suite_dir> Use this alternative directory as-is when
assembling jobs from yaml fragments. This causes
<suite_branch> to be ignored for scheduling
Expand Down Expand Up @@ -144,7 +143,6 @@
default_suite_repo=defaults('--suite-repo',
config.get_ceph_qa_suite_git_url()),
default_ceph_branch=defaults('--ceph-branch', 'master'),
default_suite_branch=defaults('--suite-branch', 'master'),
default_teuthology_branch=defaults('--teuthology-branch', 'master'),
)

Expand Down
2 changes: 2 additions & 0 deletions teuthology/suite/__init__.py
Expand Up @@ -54,6 +54,8 @@ def process_args(args):
key = key.lstrip('--').replace('-', '_')
# Rename the key if necessary
key = rename_args.get(key) or key
if key == 'suite_branch':
value = value or override_arg_defaults('--suite-branch', None)
if key == 'suite' and value is not None:
value = normalize_suite_name(value)
if key == 'suite_relpath' and value is None:
Expand Down

0 comments on commit 8c37d3b

Please sign in to comment.