Bug 1217916
Summary: | Refresh Power States Fails for OpenStack - No Cinder | |||
---|---|---|---|---|
Product: | Red Hat CloudForms Management Engine | Reporter: | Ken Evensen <kevensen> | |
Component: | Providers | Assignee: | Greg Blomquist <gblomqui> | |
Status: | CLOSED ERRATA | QA Contact: | Nandini Chandra <nachandr> | |
Severity: | high | Docs Contact: | ||
Priority: | high | |||
Version: | 5.3.0 | CC: | cpelland, jfrey, jhardy, kevensen, mfeifer, obarenbo | |
Target Milestone: | GA | Keywords: | ZStream | |
Target Release: | 5.5.0 | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | 5.5.0.1 | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 1258839 (view as bug list) | Environment: | ||
Last Closed: | 2015-12-08 13:07:03 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: | 1258839 |
Description
Ken Evensen
2015-05-02 21:49:52 UTC
Looking at the stack trace, this one is a bit sneakier than I first thought. We attempt to read the availability zones assigned to each instance and each volume when getting the list of availability zones. We already have a check in place to ignore volumes if the Cinder service isn't running, but we missed this check when looking for availability zones. This is a quick fix. New commit detected on manageiq/master: https://github.com/ManageIQ/manageiq/commit/2284d185f41ecd3cc00e1e675d9c2e33c546817a commit 2284d185f41ecd3cc00e1e675d9c2e33c546817a Author: Greg Blomquist <gblomqui> AuthorDate: Tue May 5 15:13:29 2015 -0400 Commit: Greg Blomquist <gblomqui> CommitDate: Sun Jul 19 22:14:01 2015 -0400 Do not get volumes when cinder isn't available The openstack inventory parser already has logic in place to avoid getting volumes from Cinder if the cinder service is not running. However, this check if bypassed when trying to get the list of Availability Zones. When the openstack inventory parser tries to get the list of Availability Zones, it checks what Availability Zones are assigned to instances as well as which are assigned to volumes. This check for Availability Zones assigned to volumes does not take into account the fact that cinder might not be running. https://bugzilla.redhat.com/show_bug.cgi?id=1217916 app/models/ems_refresh/parsers/openstack.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c90c96fdc0418edf99cca6dd7772cfacbbac6050 commit c90c96fdc0418edf99cca6dd7772cfacbbac6050 Author: Greg Blomquist <gblomqui> AuthorDate: Mon Aug 31 18:04:13 2015 -0400 Commit: Greg Blomquist <gblomqui> CommitDate: Tue Sep 1 10:08:57 2015 -0400 Do not get volumes when cinder isn't available The openstack inventory parser already has logic in place to avoid getting volumes from Cinder if the cinder service is not running. However, this check if bypassed when trying to get the list of Availability Zones. When the openstack inventory parser tries to get the list of Availability Zones, it checks what Availability Zones are assigned to instances as well as which are assigned to volumes. This check for Availability Zones assigned to volumes does not take into account the fact that cinder might not be running. https://bugzilla.redhat.com/show_bug.cgi?id=1258839 https://bugzilla.redhat.com/show_bug.cgi?id=1217916 vmdb/app/models/ems_refresh/parsers/openstack.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) New commit detected on cfme/5.4.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=545aba438035f14f739608af953d744a344485b0 commit 545aba438035f14f739608af953d744a344485b0 Merge: da6ae33 c90c96f Author: Gregg Tanzillo <gtanzill> AuthorDate: Mon Sep 28 15:07:16 2015 -0400 Commit: Gregg Tanzillo <gtanzill> CommitDate: Mon Sep 28 15:07:16 2015 -0400 Merge branch 'bz1258839-refresh-fails-with-no-cinder' into '5.4.z' Do not get volumes when cinder isn't available The openstack inventory parser already has logic in place to avoid getting volumes from Cinder if the cinder service is not running. However, this check if bypassed when trying to get the list of Availability Zones. When the openstack inventory parser tries to get the list of Availability Zones, it checks what Availability Zones are assigned to instances as well as which are assigned to volumes. This check for Availability Zones assigned to volumes does not take into account the fact that cinder might not be running. Upstream: https://github.com/ManageIQ/manageiq/pull/2845 Backport notes: this code was re-rooted and namespaced ... I just manually applied the changes because the changes were so trivial https://bugzilla.redhat.com/show_bug.cgi?id=1258839 https://bugzilla.redhat.com/show_bug.cgi?id=1217916 See merge request !241 vmdb/app/models/ems_refresh/parsers/openstack.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Verified in 5.5.0.9 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. https://access.redhat.com/errata/RHSA-2015:2551 |