From: Steven Hardy <shardy@redhat.com>
Date: Wed, 24 Oct 2012 12:41:34 +0000 (+0100)
Subject: heat-keystone-setup workaround keystone output reordering
X-Git-Tag: 2014.1~1273
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=b3b56ff59d1402ba2188778e683d5b3725fb941e;p=openstack-build%2Fheat-build.git

heat-keystone-setup workaround keystone output reordering

Folsom version of keystoneclient orders the columns for
keystone user-list differently, so we need to detect the
new format to extract the correct field

Fixes #273

Change-Id: I46f653dd3a8f7b5a68648fbd72671f95b386a547
Signed-off-by: Steven Hardy <shardy@redhat.com>
---

diff --git a/bin/heat-keystone-setup b/bin/heat-keystone-setup
index a9085556..74691bd0 100755
--- a/bin/heat-keystone-setup
+++ b/bin/heat-keystone-setup
@@ -44,10 +44,21 @@ get_id () {
     get_data 1 id 2 "$@"
 }
 
+get_column_num() {
+    local name=$1
+    shift
+    $@ | awk -F'|' "NR == 2 { for (i=2; i<NF; i++) if (\$i ~ \"^ *${name} *\$\") print (i - 1) }"
+}
+
 get_user() {
     local username=$1
 
-    local user_id=$(get_data 4 $username 1 keystone user-list)
+    # Outut format of keystone user-list changed between essex and
+    # folsom - the columns have been re-ordered (!?), so detect what
+    # column to pass to get_data via get_column_num
+    namecol=$(get_column_num name keystone user-list)
+
+    local user_id=$(get_data $namecol $username 1 keystone user-list)
 
     if [ -n "$user_id" ]; then
         echo "Found existing $username user" >&2