[RFE]SUM based firmware update as manual cleaning step for iLO drivers

[Aparna]

This RFE proposes the SUM(Smart Update Manager) based firmware update as an inband manual clean step 'update_firmware_sum' for iLO drivers.
It is supported in the management interface of iLO drivers.

Clean step for 'update_firmware_sum':
[{
        "interface": "management",
        "step": "update_firmware_sum",
        "args": {
            "url": "/path/to/SPP.iso",
            "checksum": "xyz",
     "components": "xyz.exe,123.exe"
            }
}]
where 'URL' - Path to the SPP (Service Pack for Proliant) ISO
      'checksum' - Checksum of the SPP ISO
      'components' - List of firmware component file names

[OpenStack Infra (hudson-openstack)]

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

[Ruby Loo (rloo)]

We discussed this RFE briefly in our ironic meeting today [1]. You might want to read the logs to get an idea of what people's thoughts are wrt this proposal. This will require a spec since there are questions.

[1] starting at 17:32:15, http://eavesdrop.openstack.org/meetings/ironic/2016/ironic.2016-12-12-17.00.log.html

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/410161

[OpenStack Infra (hudson-openstack)]

Change abandoned by Aparna (<email address hidden>) on branch: master
Review: https://review.openstack.org/410161
Reason: Abandoning the spec based on the comments as all the major changes goes into HardwareManager. I will update the RFE based on the minor changes needed in ironic.

[Aparna (aparnavtce)]

I have abandoned the spec https://review.openstack.org/410161 based on the review comments since most of the changes can be added to the HardwareManager.

The description is updated to support `hpsum_firmware_update` as a valid manual clean step in ironic.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Aparna (<email address hidden>) on branch: master
Review: https://review.openstack.org/409271

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/495139

[OpenStack Infra (hudson-openstack)]

Change abandoned by Vladyslav Drok (<email address hidden>) on branch: stable/pike
Review: https://review.openstack.org/495139

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/422572
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=c725f541de3a9df995a489fc802b6b1c941b6e01
Submitter: Zuul
Branch: master

commit c725f541de3a9df995a489fc802b6b1c941b6e01
Author: Aparna <email address hidden>
Date: Wed Jan 18 15:48:17 2017 +0000

    Support SUM based firmware update as clean step for iLO drivers

    This commit adds support to perform Smart Update Manager (SUM)
    based firmware update as an inband manual clean step
    `update_firmware_sum` for iLO drivers.

    Closes-Bug: #1648448
    Change-Id: I581d5bc535420d4493394d16cf6cc2bb0ddeb8f7

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ironic 9.2.0 release.