Skip to content

Commit 547eba9

Browse files
pablomhpablomh
committed
feat(experiment/run-library.sh): Don't push rpm version to report portal
rh-pre-commit.version: 2.2.0 rh-pre-commit.check-secrets: ENABLED
1 parent 1f7fdba commit 547eba9

File tree

1 file changed

+45
-31
lines changed

1 file changed

+45
-31
lines changed

‎experiment/run-library.sh‎

Lines changed: 45 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,6 @@ opts_adhoc=${opts_adhoc:-"$opts"}
2323
logs="$marker"
2424
run_lib_dryrun=false
2525
hammer_opts="-u admin -p changeme"
26-
satellite_version="${satellite_version:-${sat_version}}" # will be determined automatically by run-bench.sh
27-
katello_version="${katello_version:-N/A}" # will be determined automatically by run-bench.sh
2826

2927
# Requirements check
3028
#if ! type bc >/dev/null; then
@@ -139,16 +137,16 @@ function generic_environment_check() {
139137
-m ansible.builtin.shell \
140138
-a "rpm -q katello" \
141139
satellite6
142-
katello_version=$( tail -n 1 $logs/00-info-rpm-q-katello.log ); echo "$katello_version" | grep '^katello-[0-9]\.' # make sure it was detected correctly
140+
katello_rpm="$( tail -n 1 $logs/00-info-rpm-q-katello.log )"; echo "$katello_rpm" | grep '^katello-[0-9]\.' # make sure it was detected correctly
143141

144142
a 00-info-rpm-q-satellite.log \
145143
-m ansible.builtin.shell \
146144
-a "rpm -q satellite || true" \
147145
satellite6
148-
satellite_version=$( tail -n 1 $logs/00-info-rpm-q-satellite.log )
146+
satellite_rpm="$( tail -n 1 $logs/00-info-rpm-q-satellite.log )"
149147

150-
log "katello_version = $katello_version"
151-
log "satellite_version = $satellite_version"
148+
log "katello_version = $katello_rpm"
149+
log "satellite_version = $satellite_rpm"
152150

153151
a 00-check-hammer-ping.log \
154152
-m ansible.builtin.shell \
@@ -199,16 +197,22 @@ function status_data_create() {
199197
sd_start="$( date -u -Iseconds -d @$4 )"
200198
sd_end="$( date -u -Iseconds -d @$5 )"
201199
sd_duration="$(( $( date -d @$5 +%s ) - $( date -d @$4 +%s ) ))"
202-
sd_kat_ver="$6"
203-
sd_kat_ver_short=$( echo "$sd_kat_ver" | sed 's/^katello-//' | sed 's/[^0-9.-]//g' | sed 's/^\([0-9]\+\.[0-9]\+\)\..*/\1/' | sed 's/^N\/A$/0.0/' ) # "katello-3.16.0-0.2.master.el7.noarch" -> "3.16"
204-
sd_sat_ver="$7"
205-
if [[ "${sd_sat_ver}" == 'stream' ]]; then
206-
sd_sat_ver_short=stream
207-
elif [[ "$(echo "${sd_sat_ver}" | awk -F'.' '{print $(NF-2)}')" == 'stream' ]]; then
208-
sd_sat_ver_short=stream
209-
else
210-
sd_sat_ver_short=$( echo "${sd_sat_ver}" | sed 's/^satellite-//' | sed 's/[^0-9.-]//g' | sed 's/^\([0-9]\+\.[0-9]\+\)\..*/\1/' | sed 's/^N\/A$/0.0/' ) # "satellite-6.6.0-1.el7.noarch" -> "6.6"
211-
fi
200+
sd_kat_rpm="$6"
201+
[[ -n $sd_kat_rpm ]] ||
202+
sd_kat_rpm="$( ansible $opts_adhoc \
203+
-m ansible.builtin.shell \
204+
-a 'rpm -q katello' \
205+
satellite6 2>/dev/null |
206+
tail -n 1 )"
207+
sd_kat_ver_short="$( echo $sd_kat_rpm | sed 's#^\(katello-\)\(.*\)\(-.*$\)#\2#g' )" # "katello-3.16.0-0.2.master.el7.noarch" -> "3.16.0"
208+
sd_sat_rpm="$7"
209+
[[ -n $sd_sat_rpm ]] ||
210+
sd_sat_rpm="$( ansible $opts_adhoc \
211+
-m ansible.builtin.shell \
212+
-a 'rpm -q satellite' \
213+
satellite6 2>/dev/null |
214+
tail -n 1 )"
215+
sd_sat_ver_short="$( echo $sd_sat_rpm | sed 's#^\(satellite-\)\(.*\)\(-.*$\)#\2#g' )" # "satellite-6.15.1-1.el8.noarch" -> "6.15.1"
212216
sd_run="$8"
213217
sd_additional="$9"
214218
if [ -n "$STATUS_DATA_FILE" -a -f "$STATUS_DATA_FILE" ]; then
@@ -235,9 +239,9 @@ function status_data_create() {
235239
"id=$sd_run" \
236240
"name=$sd_section/$sd_name" \
237241
"parameters.cli=$( echo "$sd_cli" | sed 's/=/__/g' )" \
238-
"parameters.katello_version=$sd_kat_ver" \
242+
"parameters.katello_version=$sd_kat_rpm" \
239243
"parameters.katello_version-y-stream=$sd_kat_ver_short" \
240-
"parameters.version=$sd_sat_ver" \
244+
"parameters.version=$sd_sat_rpm" \
241245
"parameters.version-y-stream=$sd_sat_ver_short" \
242246
"parameters.run=$sd_run" \
243247
"parameters.hostname=$sd_hostname" \
@@ -308,9 +312,14 @@ function status_data_create() {
308312
###status_data.py --status-data-file $sd_file --info
309313

310314
# Create "results-dashboard-data" data file
315+
if [[ "$sat_version" == 'stream' ]]; then
316+
sd_sat_release=stream
317+
else
318+
sd_sat_release="$( echo $sd_sat_ver_short | awk -F'.' '{print $1"."$2}' )"
319+
fi
311320
set -x
312321
jq -n \
313-
--arg version ${sd_sat_ver} \
322+
--arg version ${sd_sat_release} \
314323
--arg release ${sd_sat_ver_short} \
315324
--arg date ${sd_start} \
316325
--arg link ${sd_link} \
@@ -343,8 +352,8 @@ function status_data_create() {
343352
# Enhance log file
344353
tmp=$( mktemp )
345354
echo "command: $sd_cli" >>$tmp
346-
echo "satellite version: $sd_sat_ver" >>$tmp
347-
echo "katello version: $sd_kat_ver" >>$tmp
355+
echo "satellite version: $sd_sat_rpm" >>$tmp
356+
echo "katello version: $sd_kat_rpm" >>$tmp
348357
echo "hostname: $sd_hostname" >>$tmp
349358
if [ "$sd_result" != 'ERROR' ]; then
350359
echo "result determination log:" >>$tmp
@@ -380,8 +389,8 @@ function junit_upload() {
380389
# Determine ReportPortal launch name
381390
launch_name="${PARAM_reportportal_launch_name:-default-launch-name}"
382391
if echo "$launch_name" | grep --quiet '%sat_ver%'; then
383-
sat_ver="$( echo "$satellite_version" | sed 's/^satellite-//' | sed 's/^\([0-9]\+\.[0-9]\+\).*/\1/' )"
384-
[ -z "$sat_ver" ] && sat_ver="$( echo "$katello_version" | sed 's/^katello-//' | sed 's/^\([0-9]\+\.[0-9]\+\).*/\1/' )"
392+
sat_ver="$( echo $satellite_rpm | sed 's/^satellite-//' | sed 's/^\([0-9]\+\.[0-9]\+\).*/\1/' )"
393+
[ -z "$sat_ver" ] && sat_ver="$( echo $katello_rpm | sed 's/^katello-//' | sed 's/^\([0-9]\+\.[0-9]\+\).*/\1/' )"
385394
launch_name="$( echo "$launch_name" | sed "s/%sat_ver%/$sat_ver/g" )"
386395
fi
387396
launch_name="$( echo "$launch_name" | sed "s/[^a-zA-Z0-9._-]/_/g" )"
@@ -438,14 +447,15 @@ function c() {
438447
fi
439448
local end=$( date -u +%s )
440449
log "Finish after $(( $end - $start )) seconds with log in $out and exit code $rc"
450+
441451
measurement_add \
442452
"$@" \
443453
"$out" \
444454
"$rc" \
445455
"$start" \
446456
"$end" \
447-
"$katello_version" \
448-
"$satellite_version" \
457+
"$katello_rpm" \
458+
"$satellite_rpm" \
449459
"$marker"
450460
return $rc
451461
}
@@ -463,14 +473,15 @@ function a() {
463473
fi
464474
local end=$( date -u +%s )
465475
log "Finish after $(( $end - $start )) seconds with log in $out and exit code $rc"
476+
466477
measurement_add \
467478
"ansible $opts_adhoc $( _format_opts "$@" )" \
468479
"$out" \
469480
"$rc" \
470481
"$start" \
471482
"$end" \
472-
"$katello_version" \
473-
"$satellite_version" \
483+
"$katello_rpm" \
484+
"$satellite_rpm" \
474485
"$marker"
475486
return $rc
476487
}
@@ -497,14 +508,15 @@ function ap() {
497508
fi
498509
local end=$( date -u +%s )
499510
log "Finish after $(( $end - $start )) seconds with log in $out and exit code $rc"
511+
500512
measurement_add \
501513
"ansible-playbook $opts_adhoc $( _format_opts "$@" )" \
502514
"$out" \
503515
"$rc" \
504516
"$start" \
505517
"$end" \
506-
"$katello_version" \
507-
"$satellite_version" \
518+
"$katello_rpm" \
519+
"$satellite_rpm" \
508520
"$marker"
509521
return $rc
510522
}
@@ -556,14 +568,15 @@ function e() {
556568
local passed=$( grep "^$grepper" $log_report | tail -n 1 | cut -d ' ' -f 6 )
557569
local avg_duration=$( grep "^$grepper" $log_report | tail -n 1 | cut -d ' ' -f 8 )
558570
log "Examined $log for $grepper: $duration / $passed = $avg_duration (ranging from $started_ts to $ended_ts)"
571+
559572
measurement_add \
560573
"experiment/reg-average.py '$grepper' '$log'" \
561574
"$log_report" \
562575
"$rc" \
563576
"$started_ts" \
564577
"$ended_ts" \
565-
"$katello_version" \
566-
"$satellite_version" \
578+
"$katello_rpm" \
579+
"$satellite_rpm" \
567580
"$marker" \
568581
"results.items.duration=$duration results.items.passed=$passed results.items.avg_duration=$avg_duration results.items.report_rc=$rc"
569582
}
@@ -596,6 +609,7 @@ function task_examine() {
596609
duration="$( awk -F'"' '/^results.tasks.duration=/ {printf ("%.0f", $2)}' ${log_report} )"
597610
head_tail_perc="$( awk -F'"' '/^results.tasks.percentage_removed=/ {printf ("%.2f", $2)}' ${log_report} )"
598611
log "Examined task ${task_id} and it has ${head_tail_perc} % of head/tail (ranging from ${started_ts} to ${ended_ts}) and has taken $duration seconds"
612+
599613
measurement_add \
600614
"${command}" \
601615
"${log_report}" \

0 commit comments

Comments
 (0)