[RFE] Add firmware update support to manual cleaning for iLO drivers

Bug #1526216 reported by Vladyslav Drok
8
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
deray
proliantutils
Fix Released
Undecided
deray

Bug Description

Ironic is providing a framework to support certain functionalities as part of manual cleaning. It is proposed to extend the capability for iLO drivers to invoke firmware update as part of manual cleaning.

Tags: rfe
Vladyslav Drok (vdrok)
Changed in ironic:
status: New → Confirmed
importance: Undecided → Wishlist
tags: added: rfe
Revision history for this message
Vladyslav Drok (vdrok) wrote :

Copy of whiteboard contents:

Gerrit topic: https://review.openstack.org/#/q/topic:bp/ilo-firmware-update-in-manual-cleaning,n,z

Addressed by: https://review.openstack.org/211019
    Firmware update as part of iLO manual cleaning process

Added a manual clean step for iLO firmware update with this function signature:

    update_firmware(task, **kwargs)

As an example::

    update_firmware(task_object,
        firmware_update_method='ilo',
        firmware_urls=['http://gen9_firmwares/ilo_firmware.scexe', 'http://gen9_firmwares/rom_firmware.scexe' ...]
    )

Addressed by: https://review.openstack.org/#/c/203543
    common, ris, ribcl changes to support firmware update

Gerrit topic: https://review.openstack.org/#q,topic:bp/ilo-firmware-update-in-manual-cleaning,n,z

Changed in ironic:
assignee: nobody → deray (debayan-ray)
deray (debayan-ray)
Changed in proliantutils:
assignee: nobody → deray (debayan-ray)
status: New → Confirmed
Changed in ironic:
status: Confirmed → In Progress
Changed in proliantutils:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Revision history for this message
deray (debayan-ray) wrote :

Added a manual clean step for iLO drivers to provide out-of-band firmware update,
for supported hardware components. The function signature::

 def update_firmware(self, task, **kwargs):

As an example::

    'clean_steps': [{
        'interface': 'management',
        'step': 'update_firmware',
        'args': {
            'firmware_update_mode': 'ilo',
            'firmware_images':[
                {
                    'url': 'file:///firmware_images/ilo/1.5/CP024444.scexe',
                    'checksum': 'b64c8f7799cfbb553d384d34dc43fafe336cc889',
                    'component': 'ilo'
                },
                {
                    'url': 'swift://my_account/firmware_container/cpld/cpld2.3.rpm',
                    'checksum': '<sha1-checksum-of-this-file>',
                    'component': 'cpld'
                },
                {
                    'url': 'http://my_address:port/firmwares/bios_vLatest.scexe',
                    'checksum': '<sha1-checksum-of-this-file>',
                    'component': 'bios'
                },
                {
                    'url': 'https://my_secure_address_url/firmwares/chassis_vLatest.scexe',
                    'checksum': '<sha1-checksum-of-this-file>',
                    'component': 'chassis'
                },
                {
                    'url': 'file:///home/ubuntu/firmware_images/power_pic/pmc_v3.0.bin',
                    'checksum': '<sha1-checksum-of-this-file>',
                    'component': 'power_pic'
                }
            ]
        }
    }]

Project: openstack/ironic
~~~~~~~~~~~~~~~~~~~~~~~~~
Addressed by: https://review.openstack.org/211019
 Firmware update(iLO) as manual cleaning step

Addressed by: https://review.openstack.org/#/c/276765/
 Add documentation for firmware update(iLO) as manual clean step

Addressed by: https://review.openstack.org/#/c/277408
 Augmenting the hashing strategy

Project: openstack/proliantutils
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Addressed by: https://review.openstack.org/#/c/203543/
    common, ris, ribcl changes to support firmware update

Refer manual cleaning specification at:
http://specs.openstack.org/openstack/ironic-specs/specs/approved/manual-cleaning.html
old BP:
https://blueprints.launchpad.net/ironic/+spec/ilo-firmware-update-in-manual-cleaning
Gerrit topic:
https://review.openstack.org/#/q/topic:bp/ilo-firmware-update-in-manual-cleaning,n,z

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to proliantutils (master)

Reviewed: https://review.openstack.org/203543
Committed: https://git.openstack.org/cgit/openstack/proliantutils/commit/?id=e25520af18aeaadf8a7898c26b7728c29feea080
Submitter: Jenkins
Branch: master

commit e25520af18aeaadf8a7898c26b7728c29feea080
Author: Debayan Ray <email address hidden>
Date: Mon Jul 20 02:34:46 2015 -0700

    Support for iLO based firmware update

    Added support for iLO based firmware update for the supported
    devices through ribcl and ris, as applicable. Firmware update
    can be initiated as part of manual cleaning step in ironic.

    Partial-Bug: #1526216
    Change-Id: I1eca5a08d808df1c4bda0e91181a8389d053c379

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/211019
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=3e113386ce3cf5c1faacccc9b27b9b62353eaf4c
Submitter: Jenkins
Branch: master

commit 3e113386ce3cf5c1faacccc9b27b9b62353eaf4c
Author: Debayan Ray <email address hidden>
Date: Mon Aug 10 00:17:03 2015 -0700

    Firmware update(iLO) as manual cleaning step

    Added firmware file processing (download, checksum verification,
    extraction) logic. It uses proliantutils to do the actual
    firmware update through the iLO.

    Partial-Bug: #1526216
    Depends On: I0e34407133684e34c4ab9446b3521a24f3038f92
    Depends On: I1eca5a08d808df1c4bda0e91181a8389d053c379
    Change-Id: Id40bc3283b7cfb9bed5466bdf7f8bc43d49dda34

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/291069
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=3738e38316c086f553d401ebacd054ab3eaaa766
Submitter: Jenkins
Branch: master

commit 3738e38316c086f553d401ebacd054ab3eaaa766
Author: Debayan Ray <email address hidden>
Date: Thu Mar 10 01:41:44 2016 -0800

    Follow-up of firmware update(iLO) as manual cleaning step

    In 3e113386ce3cf5c1faacccc9b27b9b62353eaf4c, there were
    some outstanding review comments which are addressed as
    part of this follow-up patch.

    Change-Id: Ib98c338b9dfa3628d40f4fa644205abf0ab24400
    Partial-Bug: #1526216

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ironic (master)

Reviewed: https://review.openstack.org/276765
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=fa0c3e75acd17b72d1e8e05712989ff1e55b232d
Submitter: Jenkins
Branch: master

commit fa0c3e75acd17b72d1e8e05712989ff1e55b232d
Author: Debayan Ray <email address hidden>
Date: Fri Feb 5 06:16:55 2016 -0800

    Adds doc - firmware update(iLO) manual clean step

    Added the documentation for firmware update through iLO as a
    manual clean step. It also serves as a user guidance on proper
    usage of initiating firmware update as the manual clean step.

    Related-Bug: #1526216
    Change-Id: Ibcbbe1330df818705c4de33325878f5c816fe1d7

Changed in ironic:
status: In Progress → Fix Released
Changed in proliantutils:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.