laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
obs: check_builders: at least 10, fail on error

The number of builders connected to the OBS server seems to fluctuate a
bit (used to be 14, then 17 now 16). Make sure that we have at least 10
builders connected. Fix that the test didn't exit with 1 on error.

Related: https://obs.osmocom.org/monitor
Change-Id: Iedd506601a5450550e21bf701309b4ea79a3d897
---
M scripts/obs/check_builders.sh
1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/scripts/obs/check_builders.sh b/scripts/obs/check_builders.sh
index 609c467..65a0d76 100755
--- a/scripts/obs/check_builders.sh
+++ b/scripts/obs/check_builders.sh
@@ -1,11 +1,19 @@
#!/bin/sh -ex
-count=14
+min=10
+max=500
wget -q https://obs.osmocom.org -O index.html

-if ! grep -q " of $count build hosts" index.html; then
- grep "build hosts" index.html
- set +x
- echo
- echo "ERROR: expected $count builders to be connected to OBS!"
- echo
-fi
+set +x
+for i in $(seq $min $max); do
+ if grep -q " of $i build hosts" index.html; then
+ echo
+ echo "Check successful, $i builders are connected to OBS"
+ echo
+ exit 0
+ fi
+done
+
+echo
+echo "ERROR: expected at least $min builders to be connected to OBS!"
+echo
+exit 1

To view, visit change 29897. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Iedd506601a5450550e21bf701309b4ea79a3d897
Gerrit-Change-Number: 29897
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged