hieradata prepends an invalid comment to UUID when SMBIOS is newer then dmidecode supports

Bug #1762460 reported by Alex Schultz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Medium
Alex Schultz

Bug Description

If the SMBIOS of a host is newer than is supported by the installed version of dmidecode, an invalid entry is included before the UUID in a node's heiradata. For example:

---
# SMBIOS implementations newer than version 2.8 are not# fully supported by this version of dmidecode.<UUID>
---

This impacts the pre_deploy per_node.yaml phase of an overcloud deployment because host UUIDs are not valid.

One potential solution could be for dmidecode to be updated prior to running pre_deploy per_node.yaml.

The '#SMBIOS implementations newer than version 2.8 are not# fully supported by this version of dmidecode.' comment is included in the UUID string in hieradata.

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

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

Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/559770
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=dc3781778e1fef0fcb0e969d821d36b0fe03a6da
Submitter: Zuul
Branch: master

commit dc3781778e1fef0fcb0e969d821d36b0fe03a6da
Author: Alex Schultz <email address hidden>
Date: Mon Apr 9 09:35:43 2018 -0600

    Sanitize the uuid string

    dmidecode can return some additional data if SMBIOS is updated. Let's
    ensure that the expected output matches the expected UUID format. The
    expected string from the 'dmidecode --s system-uuid' call should look
    like "79287E4C-2FBF-11B2-A85C-EB9FB9250CBA".

    Change-Id: Ib0d19e64b2f24f4a9229f8868795e8979e267f04
    Closes-Bug: #1762460

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 9.0.0.0b2

This issue was fixed in the openstack/tripleo-heat-templates 9.0.0.0b2 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (stable/rocky)

Related fix proposed to branch: stable/rocky
Review: https://review.openstack.org/636125

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (stable/rocky)

Reviewed: https://review.openstack.org/636125
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=90c456463862fdedbb709966ead479051ed459a7
Submitter: Zuul
Branch: stable/rocky

commit 90c456463862fdedbb709966ead479051ed459a7
Author: Giulio Fidente <email address hidden>
Date: Thu Feb 7 09:01:32 2019 +0100

    Sanitize the uuid string for ceph-ansible

    dmidecode can return some additional data if SMBIOS is updated; this
    ensures output matches the UUID format.

    Change-Id: I5c61c0049bfd16114894cf4db3b79f94b6d9291b
    Related-Bug: 1762460
    (cherry picked from commit ad803ab716448e2dfaa446d732f64c6bd851cb05)

tags: added: in-stable-rocky
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.