Change in osmo-ci[master]: repo-install-test: verify DISTRO argument

[osmith]

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/22375 )


Change subject: repo-install-test: verify DISTRO argument
......................................................................

repo-install-test: verify DISTRO argument

Keep list of supported distros in one place in repo-install-test.sh.

Related: OS#4969
Change-Id: I1aa2564ff861d81a866302e44a1f99ff876c7b31
---
M scripts/repo-install-test.sh
1 file changed, 15 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/75/22375/1

diff --git a/scripts/repo-install-test.sh b/scripts/repo-install-test.sh
index 7585509..f4dcf1d 100755
--- a/scripts/repo-install-test.sh
+++ b/scripts/repo-install-test.sh
@@ -9,13 +9,25 @@
 . "$(dirname "$0")/common.sh"
 
 # Show usage
-if [ "$#" -ne 1 ]; then
+check_usage() {
+	local i
+	for i in $DISTROS; do
+		if [ "$DISTRO" = "$i" ]; then
+			return
+		fi
+	done
 	echo "usage: repo-install-test.sh DISTRO"
-	echo "DISTRO: debian or centos8"
+	echo "DISTRO: one of: $DISTROS"
 	exit 1
-fi
+}
 
 DISTRO="$1"
+DISTROS="
+	centos8
+	debian
+"
+
+check_usage
 docker_images_require "$DISTRO-repo-install-test"
 
 FEED="${FEED:-nightly}"

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/22375
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I1aa2564ff861d81a866302e44a1f99ff876c7b31
Gerrit-Change-Number: 22375
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210122/dec7b908/attachment.htm>