osmith has submitted this change. ( https://gerrit.osmocom.org/c/docker-playground/+/40784?usp=email )
Change subject: ttcn3-bts-test: show respawn count at the end ......................................................................
ttcn3-bts-test: show respawn count at the end
As mentioned in the last commit, we currently must use respawn.sh to be able to recover from clock skew errors seen in jenkins. Let's display the respawn count at the end as warning.
Related: OS#6794 Change-Id: I4e5c3dd46ec682bfd79f2328195b0050fe3f186d --- M common/respawn.sh M ttcn3-bts-test/jenkins.sh 2 files changed, 26 insertions(+), 1 deletion(-)
Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve
diff --git a/common/respawn.sh b/common/respawn.sh index 31a8c06..d9bf641 100755 --- a/common/respawn.sh +++ b/common/respawn.sh @@ -7,7 +7,7 @@ i=0 max_i=500 while [ $i -lt $max_i ]; do - echo "$i: starting: $*" + echo "respawn: $i: starting: $*" $* & LAST_PID=$! wait $LAST_PID diff --git a/ttcn3-bts-test/jenkins.sh b/ttcn3-bts-test/jenkins.sh index 5d5629d..1d4211f 100755 --- a/ttcn3-bts-test/jenkins.sh +++ b/ttcn3-bts-test/jenkins.sh @@ -229,6 +229,27 @@ docker_kill_wait ${BUILD_TAG}-bts }
+show_respawn_count() { + set +x + local count=$(grep -P 'respawn: \d+: starting: ' "$VOL_BASE_DIR"/bts/osmo-bts.log | wc -l) + # Currently we run generic/bts/oml configurations, which means 3 + # restarts are expected. + local count_exp=3 + + if [ "$count" = "$count_exp" ]; then + echo "osmo-bts was respawned $count times (as expected)" + return + fi + + printf '\033[0;31m' # set color to red + echo + echo "================================================================" + echo " WARNING: osmo-bts was respawned $count times (expected: $count_exp)! (OS#6794)" + echo "================================================================" + echo + printf '\033[0m' +} + network_create
mkdir $VOL_BASE_DIR/bts-tester-generic @@ -272,3 +293,7 @@ start_config_virtphy start_config_oml start_config_hopping + +# Show respawn count at the very end +clean_up_common +show_respawn_count
-
osmith