Skip to content

[v2] baremetal: add vendor/model/serial_number to FirmwareComponent - #3970

Merged
winiciusallan merged 1 commit into
v2from
bp-v2-72cdeba
Aug 20, 2026
Merged

winiciusallan merged 1 commit into
v2from
bp-v2-72cdeba

Conversation

@gophercloud-backport-bot

Copy link
Copy Markdown

Backport: #3849

Summary

  • Add Vendor, Model, SerialNumber pointer fields to FirmwareComponent struct
  • Prepares gophercloud for Ironic microversion 1.114 which adds hardware identity fields to GET /v1/nodes/{ident}/firmware
  • Fields use *string per gophercloud convention: nil when API does not return them (microversion < 1.114)

Reference

Test plan

  • Unit tests updated with new fields in fixture

Assisted-By: Claude [email protected]

@github-actions github-actions Bot added v2 This PR targets v2 edit:baremetal This PR updates baremetal code semver:minor Backwards-compatible change labels Aug 20, 2026
@coveralls

coveralls commented Aug 20, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 63.933%. remained the same — bp-v2-72cdeba into v2

Prepares gophercloud for Ironic microversion 1.114 which adds hardware
identity fields to GET /v1/nodes/{ident}/firmware (spec:
https://review.opendev.org/c/openstack/ironic-specs/+/985800).

Operators currently cannot correlate a firmware record with inventory
entries. Ironic will populate vendor/model/serial_number from Redfish
resources already fetched during firmware caching -- no extra BMC calls.

Fields use pointer types (*string) per gophercloud convention: nil when
the API does not return them (microversion < 1.114), non-nil otherwise.

Assisted-By: Claude <[email protected]>
@winiciusallan
winiciusallan merged commit 3100215 into v2 Aug 20, 2026
19 checks passed
@winiciusallan
winiciusallan deleted the bp-v2-72cdeba branch August 20, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edit:baremetal This PR updates baremetal code semver:minor Backwards-compatible change v2 This PR targets v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants