1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
|
# Copyright (c) 2018 Cisco and/or its affiliates.
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at:
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
set -exuo pipefail
# This library defines functions used mainly by "per_patch_perf.sh" entry script.
# Generally, the functions assume "common.sh" library has been sourced already.
# Keep functions ordered alphabetically, please.
# TODO: Add a link to bash style guide.
function build_vpp_ubuntu_amd64 () {
set -exuo pipefail
# TODO: Make sure whether this works on other distros/archs too.
# Arguments:
# - ${1} - String identifier for echo, can be unset.
# Variables read:
# - VPP_DIR - Path to existing directory, parent to accessed directories.
# Directories updated:
# - ${VPP_DIR} - Whole subtree, many files (re)created by the build process.
# - ${VPP_DIR}/build-root - Final build artifacts for CSIT end up here.
# - ${VPP_DIR}/dpdk - The dpdk artifact is built, but moved to build-root/.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
cd "${VPP_DIR}" || die "Change directory command failed."
echo 'Building using "make build-root/vagrant/build.sh"'
# TODO: Do we want to support "${DRYRUN}" == "True"?
make UNATTENDED=yes install-dep || die "Make install-dep failed."
# The per_patch script calls this function twice, first for the new commit,
# then for its parent commit. On Jenkins, no dpdk is installed at first,
# locally it might have been installed. New dpdk is installed second call.
# If make detects installed vpp-ext-deps with matching version,
# it skips building vpp-ext-deps entirely, but we need that file.
# On the other hand, if parent uses different dpdk version,
# The new vpp-ext-deps is built, but the old one is not removed
# from the build directory if present. (Further functions move both,
# and during test dpkg decides on its own which version gets installed.)
# As per_patch is too dumb (yet) to detect any of that,
# the only safe solution is to clean build directory and force rebuild.
# TODO: Make this function smarter and skip DPDK rebuilds if possible.
cmd=("dpkg-query" "--showformat='$${Version}'" "--show" "vpp-ext-deps")
installed_deb_ver="$(sudo "${cmd[@]}" || true)"
if [[ -n "${installed_deb_ver}" ]]; then
sudo dpkg --purge "vpp-ext-deps" || {
die "DPDK package uninstalation failed."
}
fi
make UNATTENDED=yes install-ext-deps || {
die "Make install-ext-deps failed."
}
build-root/vagrant/"build.sh" || die "Vagrant VPP build script failed."
# CSIT also needs the external dependency artifacts, which is not in
# build-root.
mv -v "build/external/vpp-ext-deps"*".deb" "build-root"/ || {
die "*.deb move failed."
}
echo "*******************************************************************"
echo "* VPP ${1-} BUILD SUCCESSFULLY COMPLETED" || {
die "Argument not found."
}
echo "*******************************************************************"
}
function compare_test_results () {
set -exuo pipefail
# Variables read:
# - VPP_DIR - Path to directory with VPP git repo (at least built parts).
# - ARCHIVE_DIR - Path to where robot result files are created in.
# - PYTHON_SCRIPTS_DIR - Path to directory holding comparison utility.
# Directories recreated:
# - csit_parent - Sibling to csit directory, for holding results
# of parent build.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
# - parse_bmrr_results - See definition in this file.
# Exit code:
# - 0 - If the comparison utility sees no regression (nor data error).
# - 1 - If the comparison utility sees a regression (or data error).
cd "${VPP_DIR}" || die "Change directory operation failed."
rm -rf "csit_parent" || die "Remove operation failed."
mkdir -p "csit_parent" || die "Directory creation failed."
for filename in "output.xml" "log.html" "report.html"; do
mv "${ARCHIVE_DIR}/${filename}" "csit_parent/${filename}" || {
die "Attempt to move '${filename}' failed."
}
done
parse_bmrr_results "csit_parent" || {
die "The function should have died on error."
}
# Reusing CSIT main virtualenv.
pip install -r "${PYTHON_SCRIPTS_DIR}/perpatch_requirements.txt" || {
die "Perpatch Python requirements installation failed."
}
python "${PYTHON_SCRIPTS_DIR}/compare_perpatch.py"
# The exit code determines the vote result.
}
function download_builds () {
set -exuo pipefail
# This is mostly useful only for Sandbox testing, to avoid recompilation.
#
# Arguments:
# - ${1} - URL to download VPP builds from.
# Variables read:
# - VPP_DIR - Path to WORKSPACE, parent of created directories.
# - DOWNLOAD_DIR - Path to directory pybot takes the build to test from.
# Directories created:
# - archive - Ends up empty, not to be confused with ${ARCHIVE_DIR}.
# - build_new - Holding built artifacts of the patch under test (PUT).
# - built_parent - Holding built artifacts of parent of PUT.
# - csit_new - (Re)set to a symlink to archive robot results on failure.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
cd "${VPP_DIR}" || die "Change directory operation failed."
rm -rf "build-root" "build_parent" "build_new" "archive" "csit_new" || {
die "Directory removal failed."
}
wget -N --progress=dot:giga "${1}" || die "Wget download failed."
unzip "archive.zip" || die "Archive extraction failed."
mv "archive/build_parent" ./ || die "Move operation failed."
mv "archive/build_new" ./ || die "Move operation failed."
cp -r "build_new"/*".deb" "${DOWNLOAD_DIR}" || {
die "Copy operation failed."
}
# Create symlinks so that if job fails on robot, results can be archived.
ln -s "${ARCHIVE_DIR}" "csit_new" || die "Symbolic link creation failed."
}
function parse_bmrr_results () {
set -exuo pipefail
# Currently "parsing" is just two greps.
# TODO: Re-use PAL parsing code, make parsing more general and centralized.
#
# Arguments:
# - ${1} - Path to (existing) directory holding robot output.xml result.
# Files read:
# - output.xml - From argument location.
# Files updated:
# - results.txt - (Re)created, in argument location.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
rel_dir="$(readlink -e "${1}")" || die "Readlink failed."
in_file="${rel_dir}/output.xml"
out_file="${rel_dir}/results.txt"
# TODO: Do we need to check echo exit code explicitly?
echo "Parsing ${in_file} putting results into ${out_file}"
echo "TODO: Re-use parts of PAL when they support subsample test parsing."
pattern='Maximum Receive Rate trial results in packets'
pattern+=' per second: .*\]</status>'
grep -o "${pattern}" "${in_file}" | grep -o '\[.*\]' > "${out_file}" || {
die "Some parsing grep command has failed."
}
}
function prepare_build_parent () {
set -exuo pipefail
# Variables read:
# - VPP_DIR - Path to existing directory, parent to accessed directories.
# Directories read:
# - build-root - Existing directory with built VPP artifacts (also DPDK).
# Directories updated:
# - ${VPP_DIR} - A local git repository, parent commit gets checked out.
# - build_new - Old contents removed, content of build-root copied here.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
cd "${VPP_DIR}" || die "Change directory operation failed."
rm -rf "build_new" || die "Remove operation failed."
mkdir -p "build_new" || die "Directory creation failed."
mv "build-root"/*".deb" "build_new"/ || die "Move operation failed."
# The previous build could have left some incompatible leftovers,
# e.g. DPDK artifacts of different version.
# "make -C dpdk clean" does not actually remove such .deb file.
# Also, there usually is a copy of dpdk artifact in build-root.
git clean -dffx "dpdk"/ "build-root"/ || die "Git clean operation failed."
# Finally, check out the parent commit.
git checkout HEAD~ || die "Git checkout operation failed."
# Display any other leftovers.
git status || die "Git status operation failed."
}
function prepare_test_new () {
set -exuo pipefail
# Variables read:
# - VPP_DIR - Path to existing directory, parent of accessed directories.
# - DOWNLOAD_DIR - Path to directory where Robot takes builds to test from.
# - ARCHIVE_DIR - Path to where robot result files are created in.
# Directories read:
# - build-root - Existing directory with built VPP artifacts (also DPDK).
# Directories updated:
# - build_parent - Old directory removed, build-root moved to become this.
# - ${DOWNLOAD_DIR} - Old content removed, files from build_new copied here.
# - csit_new - Currently a symlink to to archive robot results on failure.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
cd "${VPP_DIR}" || die "Change directory operationf failed."
rm -rf "build_parent" "csit_new" "${DOWNLOAD_DIR}"/* || die "Remove failed."
mkdir -p "build_parent" || die "Directory creation operation failed."
mv "build-root"/*".deb" "build_parent"/ || die "Move operation failed."
cp "build_new"/*".deb" "${DOWNLOAD_DIR}" || die "Copy operation failed."
# Create symlinks so that if job fails on robot, results can be archived.
ln -s "${ARCHIVE_DIR}" "csit_new" || die "Symbolic link creation failed."
}
function prepare_test_parent () {
set -exuo pipefail
# Variables read:
# - VPP_DIR - Path to existing directory, parent of accessed directories.
# - CSIT_DIR - Path to existing root of local CSIT git repository.
# - ARCHIVE_DIR and DOWNLOAD_DIR - Paths to directories to update.
# Directories read:
# - build_parent - Build artifacts (to test next) are copied from here.
# Directories updated:
# - csit_new - Deleted, then recreated and latest robot results copied here.
# - ${CSIT_DIR} - Subjected to git reset and git clean.
# - ${ARCHIVE_DIR} - Created if not existing (if deleted by git clean).
# - ${DOWNLOAD_DIR} - Created after git clean, parent build copied here.
# - csit_parent - Currently a symlink to csit/ to archive robot results.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
# - parse_bmrr_results - See definition in this file.
cd "${VPP_DIR}" || die "Change directory operation failed."
rm -rf "csit_new" "csit_parent" || die "Remove operation failed."
mkdir -p "csit_new" || die "Create directory operation failed."
for filename in "output.xml" "log.html" "report.html"; do
mv "${ARCHIVE_DIR}/${filename}" "csit_new/${filename}" || {
die "Move operation of '${filename}' failed."
}
done
parse_bmrr_results "csit_new" || {
die "The function should have died on error."
}
pushd "${CSIT_DIR}" || die "Change directory operation failed."
git reset --hard HEAD || die "Git reset operation failed."
git clean -dffx || die "Git clean operation failed."
popd || die "Change directory operation failed."
mkdir -p "${ARCHIVE_DIR}" "${DOWNLOAD_DIR}" || die "Dir creation failed."
cp "build_parent"/*".deb" "${DOWNLOAD_DIR}"/ || die "Copy failed."
# Create symlinks so that if job fails on robot, results can be archived.
ln -s "${ARCHIVE_DIR}" "csit_parent" || die "Symlink creation failed."
}
function set_perpatch_dut () {
set -exuo pipefail
# Variables set:
# - DUT - CSIT test/ subdirectory containing suites to execute.
# TODO: Detect DUT from job name, when we have more than just VPP perpatch.
DUT="vpp"
}
function set_perpatch_vpp_dir () {
set -exuo pipefail
# Variables read:
# - CSIT_DIR - Path to existing root of local CSIT git repository.
# Variables set:
# - VPP_DIR - Path to existing root of local VPP git repository.
# Functions called:
# - die - Print to stderr and exit, defined in common.sh
# In perpatch, CSIT is cloned inside VPP clone.
VPP_DIR="$(readlink -e "${CSIT_DIR}/..")" || die "Readlink failed."
}
|