opera adapter fix2

* opera_adapter need sshpass which may cause deploy
fail
* compass log redirection is the last command which
always return 0 and thus cover deploy error

1. add installation of sshpass in prepare.sh
2. echo a string at the end of deploy/launch.sh into
log file to judge if deployment is complete
3. fix typing error in util/check_valid.py

Change-Id: Ic4bca50b2c316bf46cace45b9b3317ef9452a0ba
Signed-off-by: Harry Huang <huangxiangyu5@huawei.com>

diff --git a/deploy.sh b/deploy.sh
index 891eadf..e29b518 100755 (executable)
--- a/deploy.sh
+++ b/deploy.sh
@@ -39,6 +39,12 @@ if [[ -z $DEPLOY_COMPASS && -z $DEPLOY_HOST && -z $REDEPLOY_HOST ]]; then
 fi
 
 LOG_DIR=$COMPASS_DIR/work/deploy/log
+export LOG_DIR
+
 mkdir -p $LOG_DIR
 
 $COMPASS_DIR/deploy/launch.sh $* 2>&1 | tee $LOG_DIR/compass-deploy.log
+
+if [[ $(tail -1 $LOG_DIR/compass-deploy.log) != 'compass deploy success' ]]; then
+    exit 1
+fi

diff --git a/deploy/launch.sh b/deploy/launch.sh
index 5f1ee1c..3e523ad 100755 (executable)
--- a/deploy/launch.sh
+++ b/deploy/launch.sh
@@ -147,3 +147,5 @@ if [[ ${DHA##*/} =~ "openo" ]]; then
         exit 1
     fi
 fi
+
+echo 'compass deploy success'

diff --git a/deploy/prepare.sh b/deploy/prepare.sh
index c202322..b7e5bfa 100755 (executable)
--- a/deploy/prepare.sh
+++ b/deploy/prepare.sh
@@ -74,7 +74,7 @@ function  _prepare_python_env() {
         else
              sudo apt-get update -y
              sudo apt-get install -y --force-yes mkisofs bc curl ipmitool openvswitch-switch
-             sudo apt-get install -y --force-yes git python-dev python-pip figlet
+             sudo apt-get install -y --force-yes git python-dev python-pip figlet sshpass
              sudo apt-get install -y --force-yes libxslt-dev libxml2-dev libvirt-dev build-essential qemu-utils qemu-kvm libvirt-bin virtinst libmysqld-dev
              sudo apt-get install -y --force-yes libffi-dev libssl-dev
 

diff --git a/util/check_valid.py b/util/check_valid.py
index 9eaa100..e3ad6bc 100644 (file)
--- a/util/check_valid.py
+++ b/util/check_valid.py
@@ -135,7 +135,7 @@ if __name__ == "__main__":
     _, dha_file, network_file = sys.argv
 
     if not os.path.exists(dha_file):
-        err_print("DHA file doesn't exit")
+        err_print("DHA file doesn't exist")
         sys.exit(1)
     else:
         dha = load_file(dha_file)
@@ -148,7 +148,7 @@ if __name__ == "__main__":
                 has_invalid = True
 
     if not os.path.exists(network_file):
-        err_print("NETWORK file doesn't exit")
+        err_print("NETWORK file doesn't exist")
         sys.exit(1)
     else:
         network = load_file(network_file)