Bug 1472382 - discovery provision endpoint misses the pxe_loader option
Summary: discovery provision endpoint misses the pxe_loader option
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Discovery Plugin
Version: 6.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: Unspecified
Assignee: Rahul Bajaj
QA Contact: Roman Plevka
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-07-18 15:23 UTC by Roman Plevka
Modified: 2019-08-12 14:03 UTC (History)
6 users (show)

Fixed In Version: foreman_discovery-9.1.3
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-02-21 17:30:09 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 20337 0 None None None 2017-09-07 09:41:52 UTC
Foreman Issue Tracker 21122 0 None None None 2017-09-27 10:03:24 UTC

Description Roman Plevka 2017-07-18 15:23:57 UTC
Description of problem:
API (and CLI) miss the pxe_loader so it is not currently possible to change the pxe-handoff behavior for other variants than PXELINUX-BIOS via API and CLI.

Version-Release number of selected component (if applicable):
# rpm -qa satellite
satellite-6.3.0-16.0.beta.el7sat.noarch
# rpm -qa discovery
# rpm -qa | grep discovery
rubygem-smart_proxy_discovery-1.0.4-3.el7sat.noarch
tfm-rubygem-hammer_cli_foreman_discovery-1.0.0-1.el7sat.noarch
foreman-discovery-image-3.1.1-22.el7sat.noarch
tfm-rubygem-foreman_discovery-9.1.1-1.fm1_15.el7sat.noarch

How reproducible:
always

Actual results:
no pxe_loader parameter => no ability to specify the pxe loader on provisioning  the discovered hosts from API, CLI

Expected results:
option available

Additional info:

Comment 2 Brad Buckingham 2017-07-18 17:13:21 UTC
Lukas, thoughts on this one?  Should it be addressed in 6.3?

Comment 6 Lukas Zapletal 2017-09-22 08:32:44 UTC
Yes we are not there yet, broken tests in discovery upstream. THe other one will be CLI, if needed. Might be only one.

Comment 7 Lukas Zapletal 2017-09-26 06:33:26 UTC
We have a PR fix for unit tests now in discovery, merging today and then will push the PR this week.

https://github.com/theforeman/foreman/pull/4864

Comment 8 Rahul Bajaj 2017-09-27 09:58:23 UTC
@Lukas, will create a redmine issue for the missing CLI and will fix it in the meanwhile time :)

Comment 9 Lukas Zapletal 2017-10-04 07:26:07 UTC
commit 590d811374dea734ea4c23a78c4a2d50bf60cc4d
Author:     Rahul Bajaj <rahulrb0509>
AuthorDate: Wed Sep 6 11:58:53 2017 +0000
Commit:     Lukas Zapletal <lzap+git>
CommitDate: Wed Oct 4 09:12:23 2017 +0200

    Fixes #20337 - Add pxe loader into discovery v2 API

    (cherry picked from commit 38de3404134ad99111c62aa2e2159a0d8bbcddfb)

Comment 10 Roman Plevka 2017-10-10 13:22:04 UTC
VERIFIED
on sat6.3.0-19

tested all options and they correctly deploy files and update dhcp.lease file, pointing the host to the correct loader.

Comment 11 Bryan Kearney 2018-02-21 17:30:09 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/RHSA-2018:0336


Note You need to log in before you can comment on or make changes to this bug.