mtce system type scan is system_mode case sensitive

Bug #1827904 reported by Eric MacDonald
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Eric MacDonald

Bug Description

Mtce is looking for 'Standard' in system_type and will default to AIO if the 'S' in 'standard' is not uppercase.

Severity
--------
Minor

Steps to Reproduce
------------------
Provision system_type in platform.conf as 'standard' in Ansible rather than 'Standard'

Expected Behavior
------------------
Mtce daemons learn Normal System type for standard systems.

Actual Behavior
----------------
Defaults to AIO DX

Reproducibility
---------------
 100% reproducible if the corresponding Ansible change is made ; which it is not presently.

System Configuration
--------------------
Normal System

Branch/Pull Time/Commit:N/A
Last Pass: N/A
Timestamp/Logs:N/A

Revision history for this message
Ghada Khalil (gkhalil) wrote :

Marking as low priority / not gating as there is no issue given the current Ansible implementation. would-be--nice to fix if there is time.

tags: added: stx.metal
Changed in starlingx:
importance: Undecided → Low
status: New → Triaged
assignee: nobody → Eric MacDonald (rocksolidmtce)
Changed in starlingx:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to metal (master)

Reviewed: https://review.opendev.org/657408
Committed: https://git.openstack.org/cgit/starlingx/metal/commit/?id=b13750d5f0d354ee83fed79ff03224b89dcad23b
Submitter: Zuul
Branch: master

commit b13750d5f0d354ee83fed79ff03224b89dcad23b
Author: Eric MacDonald <email address hidden>
Date: Mon May 6 13:06:31 2019 -0400

    Make Mtce system mode scan case in-sensitive

    Mtce is looking for 'Standard' in system_type and will
    default to AIO if the 'S' in 'standard' is not uppercase.

    This update makes the mtce system_type driver handle
    system_type and system_mode readings case in-sensitive.

    Tested on-system case variations for both type and mode
    in platform.conf.

    Closes-Bug: 1827904
    Change-Id: I5e33097e1b13e5b5d385929dd13e7912ae89ead8
    Signed-off-by: Eric MacDonald <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Adding the stx.2.0 release label as the fix was merged for that release

tags: added: stx.2.0
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.