Comment 2 for bug 1973071

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

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

commit 24ecd22ae104842443acabf26aa17797cb67378c
Author: John Kung <email address hidden>
Date: Thu May 12 10:29:10 2022 -0400

    Update sysinv-api to perform case-insensitive mgmt_mac check

    Orchestrators or admin may attempt to perform host-update to a host
    provisioned with an upper case mgmt_mac.
    This was being rejected by the api as it was not considering
    a match as the mgmt_mac is only stored as lower case.

    This commit updates the api to match on a case-insensitive
    basis the mgmt_mac.

    Align host api _patch() to safer method to get passed
    in patch value.

    Remove a null-effect loop in updated host api _patch() method
    that was left from host hardware profile removal
    story https://storyboard.openstack.org/#!/story/2009163

    Closes-Bug: 1973071

    Test Plan:
    PASSED added tox unit tests for mgmt_mac case
    PASSED host-add with upper case mgmt_mac accepted
    PASSED host-update with upper case mgmt_mac accepted
    PASSED host-update with lower case mgmt_mac accepted
    PASSED host-update with upper case matching lower case mgmt_mac accepted
    PASSED host-ptp-instance-assign verifying get of value

    Signed-off-by: John Kung <email address hidden>
    Change-Id: Ibcd2efbb41e42cf140c5d602585aa34fa30cf4e1