[RFE] : Add a field in node object table for pci_devices

Bug #1680780 reported by Nisha Agarwal
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Invalid
Wishlist
Unassigned

Bug Description

Create a field in node object for pci devices as node.'pci_devices'. This can be populated via inspection or manually by a user for pci devices list in the format nova understands it i.e. [{'vendor_id': <value>, 'product_id': <value>, 'count': <value>, 'device_type': <value>, }]

description: updated
Revision history for this message
Jay Faulkner (jason-oldos) wrote :

Please put up a spec for this. If it's related to an existing spec, please link that review in here.

Thanks, Jay

tags: added: needs-spec rfe
Changed in ironic:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
Nisha Agarwal (agarwalnisha1980) wrote :

I did some experiments on how nova uses pci-passthrough filter and found that we dont need to add a new field to ironic node object. We can rather simply update the ironic node.properties with the pci devices count. This will help nova scheduler to pick up ironic node for deploy based on PciPassthroughFilter.
I am planning to change the bug status as "Invalid". Please let me know the opinion.

Revision history for this message
Ruby Loo (rloo) wrote :

@Nisha, Yes, I think updating node.properties with pci devices & count is the way to do it. This is what the inspector does with its 'pci_devices' plugin (https://docs.openstack.org/ironic-inspector/latest/user/usage.html#plugins).

Changed in ironic:
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers