Bug 1299812

Summary: crm_node output changed which breaks some openstack-foreman-installer scripts
Product: Red Hat Enterprise Linux 7 Reporter: GE Scott Knauss <sknauss>
Component: pacemakerAssignee: Ken Gaillot <kgaillot>
Status: CLOSED DUPLICATE QA Contact: cluster-qe <cluster-qe>
Severity: high Docs Contact:
Priority: high    
Version: 7.1CC: abeekhof, cfeist, cluster-maint, sknauss
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-01-20 08:56:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description GE Scott Knauss 2016-01-19 10:24:08 UTC 
Description of problem:

most recent version of pacemaker (1.1.13-10.el7) changes the output of crm_node which is breaking scripts in openstack foreman installer.

Version-Release number of selected component (if applicable):

1.1.13-10.el7

How reproducible:

attempt to deploy openstack using foreman installer after updating to latest pacemaker



Actual results:

deployment fails 

Expected results:

successful deployment

Additional info:

Earlier version 1.1.12-22.el7.1.4
[root@controller-1 ~]# /usr/sbin/crm_node -l
2 pcmk-controller-3
3 pcmk-controller-2
1 pcmk-controller-1

With new 1.1.13-10.el7
[root@controller-1 ~]# /usr/sbin/crm_node -l
2 pcmk-controller-3 member
3 pcmk-controller-2 member
1 pcmk-controller-1 member

I.e. there is a third field added which will confuse script

[root@controller-1 ~]# grep crm_node /tmp/ha-all-in-one-util.bash 
  /usr/sbin/crm_node -n
  members=$(/usr/sbin/crm_node -l | perl -p -e 's/^.*\s+(\S+)$/$1/g')
[root@controller-1 ~]# 

making it not get the cluster members but just "member" three times.

This script comes from 

# rpm -qf /usr/share/openstack-foreman-installer/puppet/modules/quickstack/templates/ha-all-in-one-util.erb
openstack-foreman-installer-3.0.26-1.el7ost.noarch
Comment 4 GE Scott Knauss 2016-01-19 16:46:51 UTC 
Chris,

   Done. 

BZ: 1299987

-Scott
Comment 5 GE Scott Knauss 2016-01-20 08:56:09 UTC 

*** This bug has been marked as a duplicate of bug 1290684 ***
Comment 7 Andrew Beekhof 2016-02-17 02:16:20 UTC 
Right. The mistake is mine. 

I assumed that people would be looking for the second field (and thus it would be safe to add a third).  It didn't occur to me that people would be looking for the 'last' field.

It appears I was/am in the minority.  Sorry.