Comment 2 for bug 1980381

Reviewed: https://review.opendev.org/c/starlingx/metal/+/849383
Committed: https://opendev.org/starlingx/metal/commit/081e7fdd6a2f559bf94d46379391fa758b344d43
Submitter: "Zuul (22348)"
Branch: master

commit 081e7fdd6a2f559bf94d46379391fa758b344d43
Author: Eric MacDonald <email address hidden>
Date: Mon Jul 11 12:54:55 2022 +0000

    Debian: Get http port from insturl for system node installs only

    The Debian kickstart is not defaulting the http_port to 8080
    in /etc/platform/platform.conf when no port is specified in
    the insturl environment variable. This is seen to occur on
    initial controller-0 install because the http port is typically
    not specified on the command line in that case.

    The CentOS kickstart only ever tried to parse the http port value
    from the command line for 'system node' installs.
    See stx/metal/bsp-files/kickstarts/post_net_common.cfg

    With the unified kickstart, in Debian, an extra check needs to
    be added to only parse the insturl string for the http port
    for 'system node' installs.

    This update will continue to default the http port to 8080 unless
    its a 'system node' install which is identified by 'pxecontroller'
    in the insturl environment variable. In which case the http port
    should be present and will be extracted and added to platform.conf
    as done in CentOS.

    Test Plan:

    PASS: Debian build and install
    PASS: Verify platform.conf:http_port defaults to 8080 for
          controller-0 pxeboot or USB install cases.
    PASS: Verify platform.conf:http_port is updated with port number
          specified in insturl for the 'system node' install case.

    Closes-Bug: 1980381
    Change-Id: Ibfa30a45fcf435eebb183d12c2380e06074fd157