Code Review
/
csit.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
VPP perf per patch: Reverse the order of tests
[csit.git]
/
resources
/
libraries
/
bash
/
entry
/
per_patch_perf.sh
diff --git
a/resources/libraries/bash/entry/per_patch_perf.sh
b/resources/libraries/bash/entry/per_patch_perf.sh
index
82dadc2
..
ad0ae90
100644
(file)
--- a/
resources/libraries/bash/entry/per_patch_perf.sh
+++ b/
resources/libraries/bash/entry/per_patch_perf.sh
@@
-40,6
+40,7
@@
source "${BASH_FUNCTION_DIR}/common.sh" || {
}
source "${BASH_FUNCTION_DIR}/per_patch.sh" || die "Source failed."
common_dirs || die
}
source "${BASH_FUNCTION_DIR}/per_patch.sh" || die "Source failed."
common_dirs || die
+check_prerequisites || die
set_perpatch_vpp_dir || die
build_vpp_ubuntu_amd64 "CURRENT" || die
set_aside_commit_build_artifacts || die
set_perpatch_vpp_dir || die
build_vpp_ubuntu_amd64 "CURRENT" || die
set_aside_commit_build_artifacts || die
@@
-52,33
+53,35
@@
get_test_code "${1-}" || die
get_test_tag_string || die
set_perpatch_dut || die
select_topology || die
get_test_tag_string || die
set_perpatch_dut || die
select_topology || die
+select_arch_os || die
activate_virtualenv "${VPP_DIR}" || die
activate_virtualenv "${VPP_DIR}" || die
-reserve_testbed || die
-select_tags || die
-compose_pybot_arguments || die
generate_tests || die
archive_tests || die
generate_tests || die
archive_tests || die
+reserve_and_cleanup_testbed || die
+select_tags || die
+compose_pybot_arguments || die
# Support for interleaved measurements is kept for future.
iterations=1 # 8
for ((iter=0; iter<iterations; iter++)); do
if ((iter)); then
# Support for interleaved measurements is kept for future.
iterations=1 # 8
for ((iter=0; iter<iterations; iter++)); do
if ((iter)); then
- #
Reserve
testbed has already cleaned it once,
+ #
Function reserve_and_cleanup_
testbed has already cleaned it once,
# but we need to clean it explicitly on subsequent iterations.
cleanup_topo
fi
# 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
check_download_dir || die
run_pybot || die
copy_archives || die
- archive_parse_test_results "csit_
pa
rent/${iter}" || die
+ archive_parse_test_results "csit_
cur
rent/${iter}" || die
die_on_pybot_error || die
# TODO: Use less heavy way to avoid apt remove failures.
cleanup_topo
die_on_pybot_error || die
# TODO: Use less heavy way to avoid apt remove failures.
cleanup_topo
- select_build "build_
cur
rent" || die
+ select_build "build_
pa
rent" || die
check_download_dir || die
run_pybot || die
copy_archives || die
check_download_dir || die
run_pybot || die
copy_archives || die
- archive_parse_test_results "csit_
cur
rent/${iter}" || die
+ archive_parse_test_results "csit_
pa
rent/${iter}" || die
die_on_pybot_error || die
done
untrap_and_unreserve_testbed || die
die_on_pybot_error || die
done
untrap_and_unreserve_testbed || die