Bug 1396667
| Summary: | [ironic] Include fix to upstream bug 1637671 "DRAC get_bios_config() vendor passthru causes AttributeError exception" in OSP 10 | |||
|---|---|---|---|---|
| Product: | Red Hat OpenStack | Reporter: | Richard Pioso <richard.pioso> | |
| Component: | openstack-ironic | Assignee: | Lucas Alvares Gomes <lmartins> | |
| Status: | CLOSED ERRATA | QA Contact: | Raviv Bar-Tal <rbartal> | |
| Severity: | high | Docs Contact: | ||
| Priority: | unspecified | |||
| Version: | 10.0 (Newton) | CC: | arkady_kanevsky, cdevine, christopher_dearborn, dcain, dtantsur, john.terpstra, John_walsh, jschluet, kasmith, kurt_hey, lmartins, mburns, morazi, nlevinki, randy_perryman, rbartal, rhel-osp-director-maint, richard.pioso, smerrow, sreichar, srevivo | |
| Target Milestone: | rc | Keywords: | OtherQA, Triaged | |
| Target Release: | 10.0 (Newton) | |||
| Hardware: | All | |||
| OS: | Unspecified | |||
| Whiteboard: | ||||
| Fixed In Version: | openstack-ironic-6.2.1-5.el7ost | Doc Type: | No Doc Update | |
| Doc Text: |
undefined
|
Story Points: | --- | |
| Clone Of: | 1396543 | |||
| : | 1396674 (view as bug list) | Environment: | ||
| Last Closed: | 2016-12-14 16:33:38 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: | 1335596, 1356451 | |||
|
Description
Richard Pioso
2016-11-18 22:31:59 UTC
Done The merged package version is verified. Test was not done downstream due to HA unavailability Why is that not verified on FV2 stamp? We deployed OSP 10 RC. I confirmed that that deployment contains the fix. /usr/lib/python2.7/site-packages/ironic/drivers/modules/drac/vendor_passthru.py has the patch that was backported to ironic's stable/newton branch. Thank you!
[root@director ~]# yum list
Loaded plugins: product-id, search-disabled-repos, subscription-manager,
: versionlock
Installed Packages
...
openstack-ironic-conductor.noarch
1:6.2.2-2.el7ost @RH7-RHOS-10.0-RC-2016-11-30.9
...
rhosp-director-images.noarch 10.0-20161130.1.el7ost
@RH7-RHOS-10.0-RC-2016-11-30.9
rhosp-director-images-ipa.noarch
10.0-20161130.1.el7ost
@RH7-RHOS-10.0-RC-2016-11-30.9
...
Available Packages
...
rhosp-release.noarch 10.0-3.el7ost RH7-RHOS-10.0-RC-2016-11-30.9
...
(In reply to Richard Pioso from comment #6) > We deployed OSP 10 RC. I confirmed that that deployment contains the fix. > /usr/lib/python2.7/site-packages/ironic/drivers/modules/drac/vendor_passthru. > py has the patch that was backported to ironic's stable/newton branch. Thank > you! > > [root@director ~]# yum list > > Loaded plugins: product-id, search-disabled-repos, subscription-manager, > : versionlock > Installed Packages > > ... > > openstack-ironic-conductor.noarch > 1:6.2.2-2.el7ost > @RH7-RHOS-10.0-RC-2016-11-30.9 > > ... > > rhosp-director-images.noarch 10.0-20161130.1.el7ost > > @RH7-RHOS-10.0-RC-2016-11-30.9 > rhosp-director-images-ipa.noarch > 10.0-20161130.1.el7ost > > @RH7-RHOS-10.0-RC-2016-11-30.9 > > ... > > Available Packages > > ... > > rhosp-release.noarch 10.0-3.el7ost > RH7-RHOS-10.0-RC-2016-11-30.9 > > ... I had overlooked that the "Fixed In Version" was changed by Jon Schlueter on 20-16-11-29 02:31:30 EST from openstack-ironic-conductor-6.2.1-5.el7ost to openstack-ironic-6.2.1-5.el7ost. I reexamined our OSP 10 RC installation and found the following openstack-ironic* packages installed. Note that there is no openstack-ironic-6.2.1-5.el7ost. [root@director ~]# yum list Loaded plugins: product-id, search-disabled-repos, subscription-manager, : versionlock Installed Packages ... openstack-ironic-api.noarch 1:6.2.2-2.el7ost @RH7-RHOS-10.0-RC-2016-11-30.9 openstack-ironic-common.noarch 1:6.2.2-2.el7ost @RH7-RHOS-10.0-RC-2016-11-30.9 openstack-ironic-conductor.noarch 1:6.2.2-2.el7ost @RH7-RHOS-10.0-RC-2016-11-30.9 openstack-ironic-inspector.noarch 4.2.1-1.el7ost @RH7-RHOS-10.0-RC-2016-11-30.9 ... Therefore, I guessed that the file that needs to be fixed, /usr/lib/python2.7/site-packages/ironic/drivers/modules/drac/vendor_passthru.py, might be in openstack-ironic-common.noarch. Sure enough, it is. And that file has the patch that was backported to ironic's stable/newton branch. Should the "Fixed In Version" be openstack-ironic-common-6.2.2-2.el7ost, instead of openstack-ironic-6.2.1-5.el7ost? Please see Comment 7 above. John, can you please clear up the confusion about which package has the fix for Richard? Yes I think I can clear it up. openstack-ironic-6.2.1-5.el7ost is the build NVR also name of srpm that patch was first included in. Since that point in time openstack-ironic-6.2.2-2.el7ost was built it also includes that patch. openstack-ironic-common-6.2.2-2.el7ost openstack-ironic-conductor-6.2.1-5.el7ost These are RPM built from open stack ironic package. Hope that helps. To make my life easier when wrangling OSP releases we try to keep fixed in version pointing at builds not specific packages or bare revisions. So we have tools we can use to find and translate to the build NVR that was referenced. 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/RHEA-2016-2948.html |