Host with host name longer than 32 bytes does not go online

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

Bug Description

Host with host name longer than 32 bytes does not go online.

Severity: minor

Steps to Reproduce: create hosts with hostnames longer than 32 characters

Expected Behavior: host goes online after installation

Actual Behavior: host stays offline

Reproducibility: Reproducible 100%

System Configuration: Any

Branch/Pull Time/Commit: Any

Last Pass: Never

Timestamp/Logs: N/A

Test Activity: Any

Ghada Khalil (gkhalil)
tags: added: stx.metal
Austin Sun (sunausti)
Changed in starlingx:
assignee: nobody → Austin Sun (sunausti)
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Assigning to Eric as he is already familiar with this issue and is working on a fix.

Changed in starlingx:
assignee: Austin Sun (sunausti) → Eric MacDonald (rocksolidmtce)
tags: added: stx.2.0
Changed in starlingx:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Ghada Khalil (gkhalil) wrote :

Marking as release gating; as this is a would-be-nice to fix. Not very high priority.

Revision history for this message
Bart Wensley (bartwensley) wrote :

When we increase the maximum supported host length from 32 to something larger, we also need to update sysinv to prevent the addition/modification of hostnames longer than what we support. The user should not be able to add a host with a hostname so long that it causes the system to malfunction.

Revision history for this message
Eric MacDonald (rocksolidmtce) wrote :

I have not seen the system malfunction as a result of adding an extra long hostname since sysinv supports 255 byte names. Such extra long names longer than 63 bytes just won't resolve in the kernel and the host won't come online.

I agree that the main stx stream should prevent provisioning hostnames longer than the system supports.

Changed in starlingx:
status: Triaged → 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/665969

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to metal (master)
Download full text (3.6 KiB)

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

commit a4238c2a355899aa4d6d9bcd429a7f8614041416
Author: Eric MacDonald <email address hidden>
Date: Tue Jun 18 08:33:28 2019 -0400

    Add 50 byte hostname support to maintenance

    Hosts with hostnames longer than 31 characters do not
    go online (locked-disabled-online) after installation.

    This update enhances maintenance messaging to support
    up to 50 byte/character hostnames.

    System Install:
    ---------------
    PASS: Verify system install
    PASS: Verify AIO system install (regression)
    PASS: Verify system install with long hostnames, deployment-config.yaml
    PASS: Verify mtcAgent process startup/restart logs
    PASS: Verify hbsAgent process startup/restart logs (active controller)
    PASS: Verify hbsAgent process startup/restart logs (standby controller)
    PASS: Verify hwmond process startup/restart logs
    PASS: Verify guestAgent process startup/restart logs
    PASS: Verify all common maintenance daemons startup/restart logs
    PASS: Verify patch applies and removes cleanly

    PASS: Verify long hostname Add ; inventory distribution
    PASS: Verify short hostname Add ; inventory distribution

    Long Hostname Handling:
    -----------------------
    PASS: Verify host name support for up to 50 and 51 byte hostnames

    Heartbeat Monitoring:
    ---------------------
    PASS: Verify cluster-host interface link down handling.
    PASS: Verify graceful recovery from host reboot.
    PASS: Verify pmond process failure and recovery cycle.

    Maintenance Actions:
    --------------------
    PASS: Verify host install with 50 byte hostname
    PASS: Verify host lock
    PASS: Verify host unlock
    PASS: Verify host reboot
    PASS: Verify host reinstall
    PASS: Verify host delete (no core dump / all daemon logs)
    PASS: Verify host power-off
    PASS: Verify host power-on
    PASS: Verify BMC State Info
    PASS: Verify lock and unlock storage node
    PASS: Controller Swact over and Back
    PASS: Verify thresholded heartbeat failure handling
    PASS: Verify node locked flag file
    PASS: Verify no core dumps during testiong

    Hardware Monitor:
    -----------------
    PASS: Verify BMC Provisioning/Reprovisioning/Deprovisioning
    PASS: Verify Inventory Add/Delete/Modify
    PASS: Verify Sensor Model and Monitoring
    PASS: Verify Sensor Model Relearn
    PASS: Verify Alarming and Logs
    PASS: Verify Sensor Action, Interval modification
    PASS: Verify Critical Sensor Action handling (ignore, log, alarm, reset, power cycle)

    Guest Agent:
    ------------
    PASS: Verify inventory add and delete

    Process Monitor:
    ----------------
    PASS: Verify process monitor logs
    PASS: Verify process monitor events into mtcAgent
    PASS: Verify process monitor failure alarming and recovery clear.
    PASS: Verify process monitor regression script (test-pmon.sh -c restart)
    PASS: Verify process monitor regression script (test-pmon.sh -c kill)
 ...

Read more...

Changed in starlingx:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.