openstack/rally-openstack
Code Issues Proposed changes

[Verify] rally verify install/reinstall with --no-tempest-venv

Browse Source 
The `rally verify start` command can accept the --system-wide-install flag.
It is intended to provide a possibility to use the local environment instead
of Tempest virtual environment. The flag has a strange name, and it was decided
to use --no-tempest-venv for the `rally verify install/reinstall` commands
instead of --system-wide-install. The next patch will be dedicated to rename
--system-wide-install into --no-tempest-venv that is more logical.

In addition:
1. Some doc strings were changed to keep consistency between them;

2. Some doc strings were changed to allow users to better understand
for what options are.

3. The is_installed() method was changed to fix the following issue:
this method didn't work correctly when the --no-tempest-venv flag was passed
to `rally verify install` because it always checked existance of .venv directory.

Change-Id: I5541ac4b417152eb15b1f6277efa06c9879b4180
This commit is contained in:
Yaroslav Lobankov 2015-11-27 18:04:25 +03:00
parent a93e6663b1
commit 1450b09e8d
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
etc/rally.bash_completion
View File

@ -50,9 +50,9 @@ _rally()
OPTS["verify_detailed"]="--uuid --sort-by"
OPTS["verify_genconfig"]="--deployment --tempest-config --override"
OPTS["verify_import"]="--deployment --set --file --no-use"
OPTS["verify_install"]="--deployment --source"
OPTS["verify_install"]="--deployment --source --no-tempest-venv"
OPTS["verify_list"]=""
OPTS["verify_reinstall"]="--deployment --tempest-config --source"
OPTS["verify_reinstall"]="--deployment --tempest-config --source --no-tempest-venv"
OPTS["verify_results"]="--uuid --html --json --output-file"
OPTS["verify_show"]="--uuid --sort-by --detailed"
OPTS["verify_showconfig"]="--deployment"
@ -87,4 +87,4 @@ _rally()
return 0
}
complete -o filenames -F _rally rally
complete -o filenames -F _rally rally