[RFE] Collect biosdevname info

Bug #1635351 reported by Annie Lezil
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ironic Inspector
Fix Released
Wishlist
Annie Lezil
ironic-python-agent
Fix Released
Wishlist
Ramamani Yeleswarapu

Bug Description

This is a proposal to add a collector for Ironic Python Agent. There are different collectors for Ironic Python Agent, which collects PXE boot device and root device and PCI devices, but there are no collectors that list the BIOS devices and their corresponding kernel names and MAC addresses assigned to it.

I would like to propose a collector for Ironic Python Agent to collect the info on all available BIOS devices using biosdevname –d (debug info). As an example, see the debug info for a device:

BIOS device: p9p1
Kernel name: p9p1
Permanent MAC: A0:36:9F:32:45:A8
Assigned MAC : A0:36:9F:32:45:A9
ifIndex: 2
Driver: ixgbe
Driver version: 3.15.1-k
Firmware version: 0x80000711
Bus Info: 0000:08:00.0
PCI name : 0000:08:00.0
PCI Slot : 9
Index in slot: 1

These data collected is useful for debugging the BIOS devices. As can be seen, we get the entire debug information for this particular device, which is helpful for troubleshooting.

affects: ironic-python-agent → ironic-inspector
affects: ironic-inspector → ironic-python-agent
Changed in ironic-inspector:
importance: Undecided → Wishlist
Changed in ironic-python-agent:
importance: Undecided → Wishlist
Changed in ironic-inspector:
assignee: nobody → Annie Lezil (annie-lezil)
Changed in ironic-python-agent:
assignee: nobody → Annie Lezil (annie-lezil)
Revision history for this message
Dmitry Tantsur (divius) wrote :

Hi! Could you please elaborate on your use case? Maybe an example of such debugging? Also please state the format of the resulting data. Actually, let's maybe propose a spec against ironic-inspector-specs similar to http://specs.openstack.org/openstack/ironic-inspector-specs/specs/generic-pci-resource.html

tags: added: needs-spec
Changed in ironic-python-agent:
status: New → Confirmed
Changed in ironic-inspector:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-inspector-specs (master)

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

Changed in ironic-inspector:
status: Confirmed → In Progress
Changed in ironic-python-agent:
status: Confirmed → In Progress
Revision history for this message
Annie Lezil (annie-lezil) wrote :

Do not affect this project

Changed in ironic-inspector:
status: In Progress → Won't Fix
status: Won't Fix → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-inspector-specs (master)

Reviewed: https://review.openstack.org/404372
Committed: https://git.openstack.org/cgit/openstack/ironic-inspector-specs/commit/?id=54eb73565953cca2899485d021c888c59b491e17
Submitter: Jenkins
Branch: master

commit 54eb73565953cca2899485d021c888c59b491e17
Author: Annie Lezil <email address hidden>
Date: Tue Nov 29 18:19:04 2016 +0000

    Collect NIC name given by BIOS

    Proposing a spec to add a extra field 'biosdevname' to network interface
    inventory collected by 'default' collector of ironic-python-agent.

    Change-Id: If4efd3ba5ed7bb2816696b853b1fc39d0b2340ef
    Partial-Bug: #1635351

Changed in ironic-inspector:
status: Invalid → In Progress
tags: added: rfe-approved
removed: needs-spec
Changed in ironic-python-agent:
assignee: Annie Lezil (annie-lezil) → Ramamani Yeleswarapu (ramamani-yeleswarapu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-python-agent (master)

Reviewed: https://review.openstack.org/396406
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=fdcb0922a5c4a4ecbb8cb1b307c6c750990699ff
Submitter: Jenkins
Branch: master

commit fdcb0922a5c4a4ecbb8cb1b307c6c750990699ff
Author: Annie Lezil <email address hidden>
Date: Thu Nov 10 21:29:40 2016 +0000

    Collect NIC name given by BIOS

    Adds an extra field ``biosdevname`` to network interface inventory
    collected by ``default`` inspection collector (which collects the whole
    inventory returned by hardware manager) of ironic-python-agent.

    This feature requires biosdevname utility to collect the bios given NIC
    names. The tooling module for tinyIPA is created for the same purpose.
    For CoreOS IPA pxe images, biosdevname tooling module is limited,
    because Docker repository is created and embedded into CoreOS pxe
    images. The Docker repository uses debian to download the packages.
    Debian does not have biosdevname package.

    Adds an export variable TINYIPA_REQUIRE_BIOSDEVNAME. Set this
    variable to ``true`` in your shell before building tinyIPA.

    Closes-Bug: #1635351
    Change-Id: Ia96af59e2a74868cac59e5a88cfbb3be60d85687

Changed in ironic-python-agent:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic-python-agent 2.1.0

This issue was fixed in the openstack/ironic-python-agent 2.1.0 release.

Revision history for this message
Iury Gregory Melo Ferreira (iurygregory) wrote :

The code change was in IPA and it was already released, inspector required a spec. Marking as Fix Released.

Changed in ironic-inspector:
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.