system node installs don't inherit console setting from sysinv

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

Bug Description

Brief Description
-----------------
System node install grub menus' command line should inherit the 'console' setting from system inventory.

Severity
--------
Major: Affects some servers with a different console port number being used.

Steps to Reproduce
------------------
Install a system node

Expected Behavior
------------------
Sysinv console setting is added to command line

Actual Behavior
----------------
Sysinv console setting is not added to command line

Reproducibility
---------------
100%

System Configuration
--------------------
system node installs

Branch/Pull Time/Commit
-----------------------
All Debian loads to date

Last Pass
---------
CentOS

Timestamp/Logs
--------------
N/A

Test Activity
-------------
System Install

Workaround
----------
modify mac based system install menu manually.

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

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/config/+/868353

Changed in starlingx:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to metal (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/metal/+/868356

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to config (master)

Reviewed: https://review.opendev.org/c/starlingx/config/+/868353
Committed: https://opendev.org/starlingx/config/commit/b8ebec36804fd18e6a7c0f8266f14eceeb34e9b5
Submitter: "Zuul (22348)"
Branch: master

commit b8ebec36804fd18e6a7c0f8266f14eceeb34e9b5
Author: Eric MacDonald <email address hidden>
Date: Wed Dec 21 10:50:26 2022 -0500

    Move console setting so that it applies to Debian as well

    This update adds setting of the console port for both
    CentOS and Debian rather than just CentOS.

    Test Plan:

    PASS: Verify all system node install grub menus get 'ttySx*'
          value from system inventory. (dm config)
    PASS: Verify all system node install grub menus default
          to "ttyS0,115200" if missing for a node's provisioning
          in system inventory. (vbox config)

    Partial-Bug: 2000093
    Signed-off-by: Eric MacDonald <email address hidden>
    Change-Id: Ie251308c59883b1708d51b40c514c67d3e0e0d37

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to metal (master)

Reviewed: https://review.opendev.org/c/starlingx/metal/+/868356
Committed: https://opendev.org/starlingx/metal/commit/3fee973f9ee3a5ef47f0976aa68969c9f612e4e8
Submitter: "Zuul (22348)"
Branch: master

commit 3fee973f9ee3a5ef47f0976aa68969c9f612e4e8
Author: Eric MacDonald <email address hidden>
Date: Wed Dec 21 10:54:31 2022 -0500

    Remove console=ttyS0,115200 from system node install menus

    This update removes the 'console=ttyS0,115200' grub command
    line arguement from the debian system node install menus.

    Then allow system inventory to add a customer specified
    console setting at node provisioning time by way of
    xxxAPPEND_OPTIONSxxx variable replacement.

    Test Plan:

    PASS: Verify all system node install grub menus get 'ttySx*'
          value from system inventory. (dm config)
    PASS: Verify all system node install grub menus default
          to "ttyS0,115200" if missing for a node's provisioning
          in system inventory. (vbox config)

    Partial Bug: 2000093
    Depends-On: https://review.opendev.org/c/starlingx/config/+/868353

    Signed-off-by: Eric MacDonald <email address hidden>
    Change-Id: I866f29233d2f2e637725a98b445ac6d24333ea30

Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Medium
assignee: nobody → Eric MacDonald (rocksolidmtce)
Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
tags: added: stx.8.0 stx.metal
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.