http://gitlab.cloudforms.lab.eng.rdu2.redhat.com/cloudforms/cfme/merge_requests/241
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(-)
Pete, please retest per comment 6 ;)
Verified in 5.4.3.1
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://rhn.redhat.com/errata/RHBA-2015-1916.html