Bug 1123292
Summary: | Rubygem-staypuft: HA: pacemaker cluster property are static and not reflect real status of the services. | ||
---|---|---|---|
Product: | Red Hat OpenStack | Reporter: | Leonid Natapov <lnatapov> |
Component: | openstack-foreman-installer | Assignee: | Jason Guiditta <jguiditt> |
Status: | CLOSED WONTFIX | QA Contact: | Leonid Natapov <lnatapov> |
Severity: | medium | Docs Contact: | |
Priority: | high | ||
Version: | 5.0 (RHEL 7) | CC: | abeekhof, cfeist, fdinitto, mburns, morazi, rhos-maint, yeylon |
Target Milestone: | --- | Keywords: | Reopened, ZStream |
Target Release: | Installer | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2016-04-04 15:00:35 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: |
Description
Leonid Natapov
2014-07-25 09:01:28 UTC
I'm not sure if this is something that ofi sets up or if it should be pacemaker itself. (In reply to Mike Burns from comment #1) > I'm not sure if this is something that ofi sets up or if it should be > pacemaker itself. This is something ofi sets. pacemaker doesn't set cluster properties after service names. The point here is: - pacemaker has no use for those properties - those properties are used by ofi only - they do not reflect the current status (if anything ofi has to update them) - they can be confusing for GSS/users. Best path forward is to understand why they are there and what's their purpose, then eventually handle them as necessary. (In reply to Fabio Massimo Di Nitto from comment #3) > (In reply to Mike Burns from comment #1) > > I'm not sure if this is something that ofi sets up or if it should be > > pacemaker itself. > > This is something ofi sets. pacemaker doesn't set cluster properties after > service names. > > The point here is: > > - pacemaker has no use for those properties > - those properties are used by ofi only > - they do not reflect the current status (if anything ofi has to update them) > - they can be confusing for GSS/users. Right. these properties mean nothing to pacemaker. They've been created and set outside of pacemaker. Pacemaker is just storing those values for us. > Best path forward is to understand why they are there and what's their > purpose, then eventually handle them as necessary. If you all need any help understanding how to reliably parse resource status information, please let me know. I'd be happy to review anything involving Pacemaker integration. -- Vossel These properties are entirely for ofi/quickstack, for determining cross-machine puppet orchestration. This is not affected by later stops to the service, but is needed to make sure puppet does not do something it shouldn't on a later run. This is not a bug. (In reply to Jason Guiditta from comment #5) > These properties are entirely for ofi/quickstack, for determining > cross-machine puppet orchestration. This is not affected by later stops to > the service, but is needed to make sure puppet does not do something it > shouldn't on a later run. This is not a bug. but the wording is misleading. "service: running" provides a status of a service that indeed might be failed. this is confusing at best for users and customer supports. Use all the keywords you need, but use words that will not generate confusion How about 'setup' or 'configured' instead of 'running'? (In reply to Jason Guiditta from comment #7) > How about 'setup' or 'configured' instead of 'running'? Sure, or use a prefix: service_foo: puppet_configured, puppet_setup.. Not a blocker, moving for consideration in an unnamed future release. This was never changed, and we are no longer implementing new features for ofi. |