Allow pxe_drac as an available drop-down option for registering nodes in UI

Bug #1609912 reported by Dan Trainor on 2016-08-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Medium
Julie Pichon

Bug Description

When "Upload From File" is used to Register nodes in the UI, the dialogue provides post-import configuration options. One of these options is for the value of 'Driver'. The option only seems to support the 'pxe_ssh' and 'pxe_ipmitool' Drivers.

Allowing the specification of the 'pxe_drac' Driver in this drop-down - or better yet, dynamically sourcing from Ironic a list of supported drivers - should be added.

Dan Trainor (dtrainor) wrote :
Download full text (7.6 KiB)

This might be better approached by providing values in the UI that 'ironic driver-list' provides:

[stack@macd43d7ebcad02 ~]$ ironic driver-list
+---------------------+-----------------------+
| Supported driver(s) | Active host(s) |
+---------------------+-----------------------+
| fake_pxe | localhost.localdomain |
| pxe_amt | localhost.localdomain |
| pxe_drac | localhost.localdomain |
| pxe_ilo | localhost.localdomain |
| pxe_ipmitool | localhost.localdomain |
| pxe_ssh | localhost.localdomain |
| pxe_wol | localhost.localdomain |
+---------------------+-----------------------+
[stack@macd43d7ebcad02 ~]$

The UI can build form fields based on the available parameters of each driver, as provided by the 'ironic driver-properties' command:

[stack@macd43d7ebcad02 ~]$ ironic driver-properties fake_pxe
+--------------------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| Property | Description |
+--------------------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| deploy_forces_oob_reboot | Whether Ironic should force a reboot of the Node via the out-of-band channel after deployment is complete. Provides compatiblity with older deploy ramdisks. Defaults to False. Optional. |
| deploy_kernel | UUID (from Glance) of the deployment kernel. Required. |
| deploy_ramdisk | UUID (from Glance) of the ramdisk that is mounted at boot time. Required. |
+--------------------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
[stack@macd43d7ebcad02 ~]$ ironic driver-properties pxe_ipmitool
+--------------------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| Property | Description |
+--------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------...

Read more...

Jason E. Rist (jason-rist) wrote :

Might need to include another project in this since it's not directly driven by the UI.

Changed in tripleo-ui:
status: New → Triaged
importance: Undecided → Medium
tags: added: ui
affects: tripleo-ui → tripleo
Dan Trainor (dtrainor) wrote :

openstack-ironic manages the /etc/ironic/ironic.conf file which specifies the allowed drivers via enabled_drivers.

Having re-read this from when I last filed the bug, I want to clarify that by "allowed driver" I mean "having pxe_drac an available drop-down option for the node during node definition time in the UI". Updated the bug title to clarify this.

summary: - Allow pxe_drac as an allowed driver for registering nodes
+ Allow pxe_drac as an available drop-down option for registering nodes in
+ UI
Julie Pichon (jpichon) wrote :

It kinda feels like the two bugs have their body and contents reversed!

Because the title is clearer, I will take this one on to propose a small stopgap fix that allows pxe_drac as a driver option. The other bug Dan linked to can be used for the better, long-term solution.

Changed in tripleo:
assignee: nobody → Julie Pichon (jpichon)
milestone: none → ocata-2
tags: added: newton-backport-potential

Fix proposed to branch: master
Review: https://review.openstack.org/399512

Changed in tripleo:
status: Triaged → In Progress

Reviewed: https://review.openstack.org/399512
Committed: https://git.openstack.org/cgit/openstack/tripleo-ui/commit/?id=555f5bbba0e9f67c587f52cec80b30d60821d0b3
Submitter: Jenkins
Branch: master

commit 555f5bbba0e9f67c587f52cec80b30d60821d0b3
Author: Julie Pichon <email address hidden>
Date: Fri Nov 18 10:38:03 2016 +0000

    Add pxe_drac to the node registration driver list

    Also fix a typo along the way.

    Change-Id: I438ba40f87294707adc7b7e853814d9a2245b0f1
    Closes-Bug: #1609912

Changed in tripleo:
status: In Progress → Fix Released

Reviewed: https://review.openstack.org/400845
Committed: https://git.openstack.org/cgit/openstack/tripleo-ui/commit/?id=e9f848692bcb8b1a56bcc63b67129d84bc64d25c
Submitter: Jenkins
Branch: stable/newton

commit e9f848692bcb8b1a56bcc63b67129d84bc64d25c
Author: Julie Pichon <email address hidden>
Date: Fri Nov 18 10:38:03 2016 +0000

    Add pxe_drac to the node registration driver list

    Also fix a typo along the way.

    Change-Id: I438ba40f87294707adc7b7e853814d9a2245b0f1
    Closes-Bug: #1609912
    (cherry picked from commit 555f5bbba0e9f67c587f52cec80b30d60821d0b3)

tags: added: in-stable-newton

This issue was fixed in the openstack/tripleo-ui 2.0.1 release.

This issue was fixed in the openstack/tripleo-ui 1.1.0 release.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers