From d3d799e024c4f91cde50e7530ced44c0c12cc32b Mon Sep 17 00:00:00 2001 From: Oleksandr Savatieiev Date: Fri, 15 Jul 2016 18:53:59 +0300 Subject: [PATCH] [Reports] Significant improvements in verification report * ability to show several verifications results (this deprecates command `rally verify compare') * command `rally verify compare' is removed - new report compares results in better way * new, unified, AngularJS/Jinja2-based template * new verifications results processing * new module rally.ui.report which cares about report generation (this is a place where code from rally.task.processing.plot should be also moved) Examples: rally verify results\ --uuid --html > single_verification_result.html rally verify results\ --uuid --html > compare_3.html Co-Authored-By: Oleksandr Savatieiev Co-Authored-By: Alexander Maretskiy Change-Id: I942e0d9bf2094f3254dbeccbaa76dbbc3a3ca40e --- etc/rally.bash_completion | 4 ++-- tests/ci/rally_verify.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/etc/rally.bash_completion b/etc/rally.bash_completion index f7c47eb0..f17b52dd 100644 --- a/etc/rally.bash_completion +++ b/etc/rally.bash_completion @@ -46,7 +46,7 @@ _rally() OPTS["task_trends"]="--out --open --tasks" OPTS["task_use"]="--uuid" OPTS["task_validate"]="--deployment --task --task-args --task-args-file" - OPTS["verify_compare"]="--uuid-1 --uuid-2 --csv --html --json --output-file --threshold" + OPTS["verify_compare"]="" OPTS["verify_detailed"]="--uuid --sort-by" OPTS["verify_discover"]="--deployment --pattern --system-wide" OPTS["verify_genconfig"]="--deployment --tempest-config --add-options --override" @@ -56,7 +56,7 @@ _rally() OPTS["verify_list"]="" OPTS["verify_listplugins"]="--deployment --system-wide" OPTS["verify_reinstall"]="--deployment --source --version --system-wide" - OPTS["verify_results"]="--uuid --html --json --output-file" + OPTS["verify_results"]="--uuid --html --json --csv --output-file" OPTS["verify_show"]="--uuid --sort-by --detailed" OPTS["verify_showconfig"]="--deployment" OPTS["verify_start"]="--deployment --set --regex --load-list --skip-list --tempest-config --xfail-list --no-use --system-wide --concurrency --failing" diff --git a/tests/ci/rally_verify.py b/tests/ci/rally_verify.py index b952f1b8..2c24de4e 100755 --- a/tests/ci/rally_verify.py +++ b/tests/ci/rally_verify.py @@ -140,7 +140,7 @@ def do_compare(uuid_1, uuid_2): """Compare and save results in different formats.""" results = {} for output_format in ("csv", "html", "json"): - cmd = "verify compare --uuid-1 %(uuid-1)s --uuid-2 %(uuid-2)s" % { + cmd = "verify results --uuid %(uuid-1)s %(uuid-2)s" % { "uuid-1": uuid_1, "uuid-2": uuid_2 }