Minor update to wait-for-container script 09/93509/2
authorwaqas.ikram <waqas.ikram@est.tech>
Wed, 14 Aug 2019 14:27:41 +0000 (14:27 +0000)
committerWaqas Ikram <waqas.ikram@est.tech>
Wed, 14 Aug 2019 14:28:05 +0000 (14:28 +0000)
Change-Id: I60fe89b1c16b961718e371f5bcf54b9d884160a7
Issue-ID: SO-2144
Signed-off-by: waqas.ikram <waqas.ikram@est.tech>
plans/so/integration-etsi-testing/config/env
plans/so/integration-etsi-testing/config/wait-for-container.sh
plans/so/integration-etsi-testing/setup.sh

index 5334c8c..54d6720 100644 (file)
@@ -2,3 +2,4 @@ NEXUS_DOCKER_REPO_MSO=nexus3.onap.org:10001
 TAG=1.4.0-STAGING-latest
 TIME_OUT_DEFAULT_VALUE_SEC=1200
 PROJECT_NAME=etsiintegrationtesting
+DEFAULT_NETWORK_NAME=etsiintegrationtesting_default
index c2cbc26..598715d 100755 (executable)
@@ -32,11 +32,11 @@ usage()
  cat<<-EOF
  Command Arguments:
 
- -n, --name
+ -c, --container-name
  Mandatory argument. container name
 
- -p, --project-name
- Mandatory argument. project name
+ -n, --network-name
+ Mandatory argument. network name
 
  -t, --timeout
  Mandatory argument. time out value in seconds (must be number)
@@ -64,8 +64,8 @@ invalid_arguments()
 
 process_arguments()
 {
- SHORT_ARGS="n:p:t:"
- LONG_ARGS="help,name:,project-name:,timeout:"
+ SHORT_ARGS="c:n:t:"
+ LONG_ARGS="help,container-name:,network-name:,timeout:"
 
  args=$(getopt -o $SHORT_ARGS -l $LONG_ARGS -n "$0"  -- "$@"  2>&1 )
  [[ $? -ne 0 ]] && invalid_arguments $( echo " $args"| head -1 )
@@ -76,11 +76,11 @@ process_arguments()
 
  while true; do
     case "$1" in
-         -n|--name)
+         -c|--container-name)
           NAME=$2
           shift 2 ;;
-         -p|project-name)
-          PROJECT_NAME=$2
+         -n|--network-name)
+          NETWORK_NAME=$2
           shift 2 ;;
           -t|--timeout)
           TIME_OUT=$2
@@ -102,8 +102,8 @@ process_arguments()
    echo "$SCRIPT_NAME $(current_timestamp): error: Container name  must not be empty! $NAME" >&2; exit 1
  fi
 
- if [ -z "$PROJECT_NAME" ]; then
-   echo "$SCRIPT_NAME $(current_timestamp): error: project name  must not be empty! $PROJECT_NAME" >&2; exit 1
+ if [ -z "$NETWORK_NAME" ]; then
+   echo "$SCRIPT_NAME $(current_timestamp): error: network name  must not be empty! $NETWORK_NAME" >&2; exit 1
  fi
 
  regex='^[0-9]+$'
@@ -118,10 +118,10 @@ process_arguments()
    exit 1
  fi
 
- HOST_IP=$(docker inspect --format '{{ index .NetworkSettings.Networks "'$PROJECT_NAME'" "IPAddress"}}' $CONTAINER_NAME)
+ HOST_IP=$(docker inspect --format '{{ index .NetworkSettings.Networks "'$NETWORK_NAME'" "IPAddress"}}' $CONTAINER_NAME)
 
  if [ $? -ne 0 ]; then
-   echo "$SCRIPT_NAME $(current_timestamp) ERROR: Unable to find HOST IP using project name: $PROJECT_NAME and container name: $CONTAINER_NAME"
+   echo "$SCRIPT_NAME $(current_timestamp) ERROR: Unable to find HOST IP using network name: $NETWORK_NAME and container name: $CONTAINER_NAME"
    exit 1
  fi
 
index 6f0c010..ee0058e 100755 (executable)
@@ -47,6 +47,16 @@ echo "Running $SCRIPT_HOME/$SCRIPT_NAME ..."
 
 export $(egrep -v '^#' $ENV_FILE | xargs)
 
+MANDATORY_VARIABLES_NAMES=( "NEXUS_DOCKER_REPO_MSO" "TAG" "TIME_OUT_DEFAULT_VALUE_SEC" "PROJECT_NAME" "DEFAULT_NETWORK_NAME")
+
+for var in "${MANDATORY_VARIABLES_NAMES[@]}"
+ do
+   if [ -z "${!var}" ]; then
+     echo "Missing mandatory attribute $var in $ENV_FILE"
+     exit 1
+  fi
+done
+
 if [[ ! "$TEMP_DIR_PATH" || ! -d "$TEMP_DIR_PATH" ]]; then
         echo "Creating temporary directory $TEMP_DIR_PATH"
         mkdir $TEMP_DIR_PATH
@@ -154,7 +164,7 @@ fi
 
 API_INFRA_CONTAINER_NAME="api-handler-infra"
 echo "Will execute $WAIT_FOR_CONTAINER_SCRIPT to wait for $API_INFRA_CONTAINER_NAME container to start up"
-$WAIT_FOR_CONTAINER_SCRIPT -n "$API_INFRA_CONTAINER_NAME" -t "300" -p "${PROJECT_NAME}_default"
+$WAIT_FOR_CONTAINER_SCRIPT -c "$API_INFRA_CONTAINER_NAME" -t "300" -n "$DEFAULT_NETWORK_NAME"
 
 if [ $? -ne 0 ]; then
    echo "ERROR: $WAIT_FOR_CONTAINER_SCRIPT failed"