Make check_xapi warn-only

Author: remibergsma

cloudstackops/xenserver.py

@@ -85,9 +85,8 @@ def check_connect(self, host):
     # Check if we can use xapi
     def check_xapi(self, host):
         try:
-            with settings(host_string=self.ssh_user + "@" + host.ipaddress):
-                with warn_only():
-                    result = fab.run("xe host-enable host=" + host.name)
+            with settings(warn_only=True, host_string=self.ssh_user + "@" + host.ipaddress):
+                result = fab.run("xe host-enable host=" + host.name)
                 if result.return_code == 0:
                     return True
                 else:

xenserver_post_empty_script.sh

@@ -53,11 +53,23 @@ done
 
 echo "Upgrading drivers"
 yum -y install bnx2x-* fnic* qla2* glnic* qlge* tg3* openvswitch-modules-xen*
+if [ $? -eq 0 ]; then
+    echo "Yum commnand returned non-zero"
+    exit 1
+fi
 
 SYSTEM_HARDWARE=$(dmidecode -s system-product-name | grep -v "#")
 if [[ "${SYSTEM_HARDWARE}" == "ProLiant DL380 G7" ]]; then
     echo "Skip HPSA on HP ProLiant DL380 G7 or else the box won't boot"
 else
     yum -y install hpsa*
+    if [ $? -eq 0 ]; then
+    echo "Yum commnand returned non-zero"
+    exit 1
+fi
 fi
 yum -y upgrade nicira-ovs-hypervisor-node
+if [ $? -eq 0 ]; then
+    echo "Yum commnand returned non-zero"
+    exit 1
+fi