Bug 1027455

Summary: More recent puppet-pacemaker in packstack-modules-puppet
Product: Red Hat OpenStack Reporter: Crag Wolfe <cwolfe>
Component: openstack-packstackAssignee: Martin Magr <mmagr>
Status: CLOSED ERRATA QA Contact: Lon Hohberger <lhh>
Severity: medium Docs Contact:
Priority: high    
Version: 4.0CC: aortega, breeler, derekh, hateya, jguiditt, mmagr, yeylon
Target Milestone: rcKeywords: OtherQA, Triaged
Target Release: 4.0   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: openstack-packstack-2013.2.1-0.14.dev919.el6ost Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1027456 1031797 (view as bug list) Environment:
Last Closed: 2013-12-20 00:34:19 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:
Embargoed:
Bug Depends On:    
Bug Blocks: 998597, 1027456, 1031797, 1049121, 1049122    

Description Crag Wolfe 2013-11-06 21:33:14 UTC
Description of problem:

And older version of the puppet-pacemaker module exists in the latest openstack-puppet-modules rpm.  It should instead be equivalent to https://github.com/radez/puppet-pacemaker/commit/0b16aea22c8a03ddadf42837bcf0990355f961ff .

Comment 2 Alvaro Lopez Ortega 2013-12-04 17:01:47 UTC
Pacemaker isn't used in Packstack, so moving it to openstack-puppet-modules.

Comment 3 Alvaro Lopez Ortega 2013-12-04 17:23:48 UTC
*** Bug 1031797 has been marked as a duplicate of this bug. ***

Comment 4 Jason Guiditta 2013-12-04 17:39:26 UTC
But it _is_ used by foreman, so it is needed in packstack-modules-puppet.  We use that rpm for our puppet dependencies just like packstack does

Comment 5 Martin Magr 2013-12-05 11:07:28 UTC
Moving bug to the correct component doesn't mean we won't add it to packstack submodule set, since it is source for openstack-puppet-modules.

Comment 7 Scott Lewis 2013-12-09 15:33:42 UTC
Adding OtherQA for bugs in MODIFIED

Comment 10 Lon Hohberger 2013-12-11 20:02:02 UTC
I verified that this matches expected:

[root@localhost ~]# rpm -ql openstack-packstack | grep pacemaker | grep filesystem
/usr/lib/python2.6/site-packages/packstack/puppet/modules/pacemaker/manifests/resource/filesystem.pp
[root@localhost ~]# cat /usr/lib/python2.6/site-packages/packstack/puppet/modules/pacemaker/manifests/resource/filesystem.pp
class pacemaker::resource::filesystem($device,
                                      $directory,
                                      $fstype,
                                      $group='',
                                      $interval="30s",
                                      $ensure=present) {
  $resource_id = delete("fs-${directory}", '/')
  pacemaker::resource::base { $resource_id:
    resource_type   => "Filesystem",
    resource_params => "device=${device} directory=${directory} fstype=${fstype}",
    group           => $group,
    interval        => "30s",
    ensure          => $ensure,
  }
}

openstack-packstack-2013.1.1-0.37.dev705.el6ost.noarch

Comment 12 errata-xmlrpc 2013-12-20 00:34:19 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2013-1859.html