Browse code

Merge "Check that generate-subunit exists before using it"

Jenkins authored on 2017/09/24 00:26:40
Showing 1 changed files
... ...
@@ -509,14 +509,20 @@ function exit_trap {
509 509
 
510 510
     if [[ $r -ne 0 ]]; then
511 511
         echo "Error on exit"
512
-        generate-subunit $DEVSTACK_START_TIME $SECONDS 'fail' >> ${SUBUNIT_OUTPUT}
512
+        # If we error before we've installed os-testr, which will fail.
513
+        if type -p generate-subunit > /dev/null; then
514
+            generate-subunit $DEVSTACK_START_TIME $SECONDS 'fail' >> ${SUBUNIT_OUTPUT}
515
+        fi
513 516
         if [[ -z $LOGDIR ]]; then
514 517
             $TOP_DIR/tools/worlddump.py
515 518
         else
516 519
             $TOP_DIR/tools/worlddump.py -d $LOGDIR
517 520
         fi
518 521
     else
519
-        generate-subunit $DEVSTACK_START_TIME $SECONDS >> ${SUBUNIT_OUTPUT}
522
+        # If we error before we've installed os-testr, which will fail.
523
+        if type -p generate-subunit > /dev/null; then
524
+            generate-subunit $DEVSTACK_START_TIME $SECONDS >> ${SUBUNIT_OUTPUT}
525
+        fi
520 526
     fi
521 527
 
522 528
     exit $r