Add Error Handling 02/14602/1 v0.31.0
authorTim Johnson <tijohnson@linuxfoundation.org>
Fri, 15 Feb 2019 18:21:52 +0000 (10:21 -0800)
committerTim Johnson <tijohnson@linuxfoundation.org>
Fri, 15 Feb 2019 18:21:52 +0000 (10:21 -0800)
Packer Cleanup Script was ignoring all errors

Change-Id: I881abf36420a1976c4f3af552aeeca4a4512c555
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>
shell/packer-clear-credentials.sh

index 093b16b..866edcf 100644 (file)
@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/bin/bash -ue
 # SPDX-License-Identifier: EPL-1.0
 ##############################################################################
 # Copyright (c) 2017 The Linux Foundation and others.
@@ -10,6 +10,6 @@
 ##############################################################################
 echo "---> packer-clear-credentials.sh"
 
-set +e  # DO NOT cause build failure if any of the rm calls fail.
-rm "$CLOUDENV"
-exit 0
+# OK if $CLOUDENV does not exist or empty
+# Fails if $CLOUDENV exists and rm is unable to delete it
+rm -rf "$CLOUDENV"