From 00bcaf068eec30f22c11ba776b0ca08355f2b118 Mon Sep 17 00:00:00 2001 From: "Haomeng, Wang" Date: Thu, 17 Mar 2016 02:09:56 +0000 Subject: [PATCH] Devstack: add check of chassis creating Current devstack scripts will not check if chassis creating is successfully or not, just go ahead to run next actions, so when we use chassis_uuid, if it is empty, will break the devstack, but user don't have idea why missing this chassis_uuid. With this patch, we add code to check if chassis creating is successfully or not, and exit the devstack if failed. Change-Id: I575578a05274fff0cffbb86e8780a8ef5a6366b6 Closes-Bug: #1558340 --- devstack/lib/ironic | 1 + 1 file changed, 1 insertion(+) diff --git a/devstack/lib/ironic b/devstack/lib/ironic index 9417648bf9..bd578357bd 100644 --- a/devstack/lib/ironic +++ b/devstack/lib/ironic @@ -669,6 +669,7 @@ function _clean_ncpu_failure { function enroll_nodes { local chassis_id chassis_id=$(ironic chassis-create -d "ironic test chassis" | grep " uuid " | get_field 2) + die_if_not_set $LINENO chassis_id "Failed to create chassis" if [[ "$IRONIC_IS_HARDWARE" == "False" ]]; then local ironic_node_cpu=$IRONIC_VM_SPECS_CPU