aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVratko Polak <vrpolak@cisco.com>2019-08-19 15:58:43 +0200
committerVratko Polak <vrpolak@cisco.com>2019-08-19 15:03:35 +0000
commit103d029def9e8feb0a89f25798bb3f2ac2beea31 (patch)
treec8f348a103c3ee23e1e6545580b74177f5e662a4
parentae7dc0011f376861ba9259b7cb5166777520799c (diff)
VPP perf per patch: Reverse the order of tests
If current VPP is broken, we want to fail early. If there is API change, we are more interested in current results, even if parent is likely to fail. Change-Id: I552485bbb9fe0053bae39de7939f94a5971bb63b Signed-off-by: Vratko Polak <vrpolak@cisco.com>
-rw-r--r--resources/libraries/bash/entry/per_patch_perf.sh9
1 files changed, 5 insertions, 4 deletions
diff --git a/resources/libraries/bash/entry/per_patch_perf.sh b/resources/libraries/bash/entry/per_patch_perf.sh
index afe5f97575..ad0ae90085 100644
--- a/resources/libraries/bash/entry/per_patch_perf.sh
+++ b/resources/libraries/bash/entry/per_patch_perf.sh
@@ -68,19 +68,20 @@ for ((iter=0; iter<iterations; iter++)); do
# but we need to clean it explicitly on subsequent iterations.
cleanup_topo
fi
- select_build "build_parent" || die
+ # Testing current first. Good for early failures or for API changes.
+ select_build "build_current" || die
check_download_dir || die
run_pybot || die
copy_archives || die
- archive_parse_test_results "csit_parent/${iter}" || die
+ archive_parse_test_results "csit_current/${iter}" || die
die_on_pybot_error || die
# TODO: Use less heavy way to avoid apt remove failures.
cleanup_topo
- select_build "build_current" || die
+ select_build "build_parent" || die
check_download_dir || die
run_pybot || die
copy_archives || die
- archive_parse_test_results "csit_current/${iter}" || die
+ archive_parse_test_results "csit_parent/${iter}" || die
die_on_pybot_error || die
done
untrap_and_unreserve_testbed || die