Bug 1231331

Summary:	pxe_drac fails with local boot enabled
Product:	Red Hat OpenStack	Reporter:	Imre Farkas <ifarkas>
Component:	openstack-ironic	Assignee:	Imre Farkas <ifarkas>
Status:	CLOSED ERRATA	QA Contact:	Toure Dunnon <tdunnon>
Severity:	unspecified	Docs Contact:
Priority:	medium
Version:	Director	CC:	athomas, christopher_dearborn, dariusz.smigiel, mburns, mlopes, sasha, yeylon
Target Milestone:	ga	Keywords:	OtherQA, Triaged
Target Release:	7.0 (Kilo)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	openstack-ironic-2015.1.0-6.el7ost	Doc Type:	Bug Fix
Doc Text:	Previously, the `pass_bootloader_install_info` method was missing from the DRAC `vendor_passthru interface`. Consequently, PXE deployment tasks failed when local boot was enabled. This fix adds the `pass_bootloader_install_info` from the standard PXE interface to `DRAC vendor_passthru`. As a result, deployment is expected to succeed when local boot is enabled.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-08-05 13:26:25 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 Imre Farkas 2015-06-12 17:21:38 UTC

Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:
This was reported upstream, and I have not personally reproduced the issue.

Steps to Reproduce:
1. nova flavor-key <flavor name> set capabilities:boot_option=local
2. ironic node-update <node uuid> add properties/capabilities='boot_option:local'
3. start deployment

Actual results:
ironic will fail with "No handler for method pass_bootloader_install_info"

Expected results:
deployment should be successful

Additional info:
pass_bootloader_install_info is missing from pxe_drac driver

Comment 3 Imre Farkas 2015-06-14 14:47:52 UTC

This only needs a fix downstream because upstream uses a different DRAC vendor_passthru interface (the BIOS and RAID management isn't there).

Proposed fix: https://code.engineering.redhat.com/gerrit/#/c/50679/

Couldn't test the patch yet, because I don't have the proper environment.

Comment 5 Mike Burns 2015-06-18 15:49:09 UTC

Chris,  Is this something you could test if we get you a scratch build?

Comment 6 Chris Dearborn 2015-06-18 16:49:02 UTC

Hey Mike,  I actually tested this yesterday!  I slicked and reloaded my undercloud, then hacked this change into the shipped ironic, and then attempted an overcloud deployment.

The result was that the nodes were successfully provisioned, but there was a failure further down the line.

Without this fix, the nodes were not being provisioned successfully.

So I think we're all set with this change.

Comment 7 Mike Burns 2015-06-18 18:07:03 UTC

Thanks for the quick turnaround!  Pushing this bug forward...

Comment 8 Imre Farkas 2015-06-19 12:47:53 UTC

Patch is merged.

Comment 9 Imre Farkas 2015-06-19 14:33:19 UTC

My bad, the patch isn't merged yet.

Comment 12 Mike Burns 2015-06-29 11:54:31 UTC

*** Bug 1235667 has been marked as a duplicate of this bug. ***

Comment 14 errata-xmlrpc 2015-08-05 13:26:25 UTC

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/RHEA-2015:1548