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

[Dan Trainor]

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)]

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 |
+--------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------...

[Jason E. Rist (jason-rist)]

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

[Dan Trainor (dtrainor)]

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.

[Dan Trainor (dtrainor)]

Possibly related to https://bugs.launchpad.net/tripleo/+bug/1642663

[Julie Pichon (jpichon)]

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.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/400845

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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