openstack/rally-openstack
Code Issues Proposed changes

Small improvements for rally verify import

Browse Source 
1. Optional arguement "--set-name" was renamed into "--set" for
   consistency. See `rally verify start`.

2. The @envutils.with_default_deployment decorator was added to
   the import_results method.

3. When we import some results and don't specify set name for tests,
   in the verification table set name for these tests will be specified
   as None. So the default value of the "set_name" parameter was changed
   from None to "" to avoid this issue.

Change-Id: Icd6659f1a78c6a426c50c67cc31b7cd85f9fbd11
This commit is contained in:
Yaroslav Lobankov 2015-10-02 17:09:21 +03:00
parent bc1ec4e3dc
commit 1642922bbe
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
etc/rally.bash_completion
View File

@ -49,7 +49,7 @@ _rally()
OPTS["verify_compare"]="--uuid-1 --uuid-2 --csv --html --json --output-file --threshold"
OPTS["verify_detailed"]="--uuid --sort-by"
OPTS["verify_genconfig"]="--deployment --tempest-config --override"
OPTS["verify_import"]="--deployment --set-name --file --no-use"
OPTS["verify_import"]="--deployment --set --file --no-use"
OPTS["verify_install"]="--deployment --source"
OPTS["verify_list"]=""
OPTS["verify_reinstall"]="--deployment --tempest-config --source"
@ -86,4 +86,4 @@ _rally()
return 0
}
complete -o filenames -F _rally rally
complete -o filenames -F _rally rally