Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BZ #1236180 - Missing redis-vip then ceilo-central pcs constraint. #542

Merged
merged 1 commit into from Jun 26, 2015
Merged

BZ #1236180 - Missing redis-vip then ceilo-central pcs constraint. #542

merged 1 commit into from Jun 26, 2015

Conversation

cwolferh
Copy link

Matches the ref arch constraint: https://github.com/beekhof/osp-ha-deploy/blob/f73eec96ddd9c7f2c85dbb0348ff909f144631ec/pcmk/ceilometer.scenario#L92

Results in puppet output:

Debug: Exec[Creating order constraint redis-vip-then-ceilo-central](provider=posix): Executing '/usr/sbin/pcs constraint order start ip-redis-pub-192.168.201.56 then start ceilometer-central-clone'
Debug: Executing '/usr/sbin/pcs constraint order start ip-redis-pub-192.168.201.56 then start ceilometer-central-clone'
Notice: /Stage[main]/Quickstack::Pacemaker::Constraints/Quickstack::Pacemaker::Constraint::Typical[redis-vip-then-ceilo-central]/Quickstack::Pacemaker::Constraint::Base[redis-vip-then-ceilo-central]/Pacemaker::Constraint::Base[redis-vip-then-ceilo-central]/Exec[Creating order constraint redis-vip-then-ceilo-central]/returns: executed successfully

Results in constraint:

# pcs constraint | grep ip-redis | grep ceilo
pcs constraint | grep ip-redis | grep ceilo
  start ip-redis-pub-192.168.201.56 then start ceilometer-central-clone (kind:Mandatory)

@cwolferh cwolferh force-pushed the redis_ceilo_central_constraint branch from de873cd to 9ab2bf1 Compare June 26, 2015 18:46
@cwolferh cwolferh changed the title Add redis-vip then ceilo-central pacemaker constraint. BZ #1180158 - Missing redis-vip then ceilo-central pcs constraint. Jun 26, 2015
@cwolferh cwolferh force-pushed the redis_ceilo_central_constraint branch from 9ab2bf1 to 66ede6b Compare June 26, 2015 18:47
@cwolferh cwolferh changed the title BZ #1180158 - Missing redis-vip then ceilo-central pcs constraint. BZ #1236180 - Missing redis-vip then ceilo-central pcs constraint. Jun 26, 2015
@cwolferh
BZ #1236180 - Missing redis-vip then ceilo-central pcs constraint.
66ede6b 
https://bugzilla.redhat.com/show_bug.cgi?id=1236180

Add the missing pacemaker constraint to ensure the redis vip (and
indirectly redis-master) are up before ceilometer-central.
jguiditta added a commit that referenced this pull request Jun 26, 2015
@jguiditta
Merge pull request #542 from cwolferh/redis_ceilo_central_constraint
d793eeb 
BZ #1236180 - Missing redis-vip then ceilo-central pcs constraint.
@jguiditta jguiditta merged commit d793eeb into redhat-openstack:master Jun 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@cwolferh @jguiditta