From: Alexandru Antone <Alexandru.Antone@enea.com>
Date: Thu, 30 Jan 2020 08:43:46 +0000 (+0200)
Subject: Fix jenkins remote-installer docker build
X-Git-Url: https://gerrit.akraino.org/r/gitweb?p=ta%2Fremote-installer.git;a=commitdiff_plain;h=da2e804b1c4fcb3d6e7ba4f4f830f268fe4a55b9

Fix jenkins remote-installer docker build

Moved arch checking code in common build.sh script used by both:
 - rpm build: remote-installer.spec
 - jenkins docker build: build-container.sh

Signed-off-by: Alexandru Antone <Alexandru.Antone@enea.com>
Change-Id: Icbfb05ddc8bad8c050b8f406f328ca28b6bfb411
---

diff --git a/remote-installer.spec b/remote-installer.spec
index 9806392..2dba6c7 100644
--- a/remote-installer.spec
+++ b/remote-installer.spec
@@ -14,7 +14,7 @@
 
 Name:           remote-installer
 Version:        %{_version}
-Release:        3%{?dist}
+Release:        4%{?dist}
 Summary:        Contains components for the remote-installer
 Group:          %{_platform_group}
 License:        %{_platform_licence}
@@ -30,14 +30,7 @@ Contains components for the remote-installer
 %prep
 
 %build
-BASEIMAGE_TAG='centos:7.6.1810'
-
-# For aarch64 use the closest available upstream version
-if [ "%{_arch}" = "aarch64" ]; then
-    BASEIMAGE_TAG='centos@sha256:df89b0a0b42916b5b31b334fd52d3e396c226ad97dfe772848bdd6b00fb42bf0'
-fi
-
-scripts/build.sh -t "${BASEIMAGE_TAG}"
+scripts/build.sh
 
 # Here hould be some registry but it should be handled by a Jenkis job
 docker image save remote-installer >remote-installer-image.tar
diff --git a/scripts/build.sh b/scripts/build.sh
index ee9d89d..99f8f7b 100755
--- a/scripts/build.sh
+++ b/scripts/build.sh
@@ -1,5 +1,6 @@
 #!/bin/bash
 # Copyright 2019 Nokia
+# Copyright 2020 ENEA
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -18,18 +19,25 @@ cd "$(dirname "$0")"/..
 TAR_IMAGE="remote-installer.tar"
 DOCKERFILE='docker-build/remote-installer/Dockerfile'
 
+BASEIMAGE_TAG='centos:7.6.1810'
+
+# For aarch64 use the closest available upstream version
+if [ "$(uname -m)" = "aarch64" ]; then
+    BASEIMAGE_TAG='centos@sha256:df89b0a0b42916b5b31b334fd52d3e396c226ad97dfe772848bdd6b00fb42bf0'
+fi
+
+
 help()
 {
-    echo -e "$(basename "$0") [-hs] -t <tag>"
+    echo -e "$(basename "$0") [-hs]"
     echo -e "   -h        display this help"
     echo -e "   -s        save image as tar to $TAR_IMAGE"
-    echo -e "   -t <tag>  specify docker base image tag"
     echo
     echo -e "Proxy configuration is taken from environment variables"
     echo -e "http_proxy, https_proxy and no_proxy"
 }
 
-while getopts "hst:" arg; do
+while getopts "hs" arg; do
     case $arg in
         h)
             help
@@ -38,9 +46,6 @@ while getopts "hst:" arg; do
         s)
             SAVE_IMAGE="yes"
             ;;
-        t)
-            BASEIMAGE_TAG="$OPTARG"
-            ;;
   esac
 done