sonic-buildimage icon indicating copy to clipboard operation
sonic-buildimage copied to clipboard

[action] [PR:19514] Fix no standalone lo_v6 entry which break YANG validation (#19514)

Open mssonicbld opened this issue 1 year ago • 1 comments

Why I did it

The load_mgmt_config generated lo_v6 is missing standalone entry which fail YANG validation

Work item tracking
  • Microsoft ADO (number only):28665800

How I did it

Add the standalone entry

How to verify it

Manual test in DUT

Before change:

admin@str-msn2700a1-03:~$ sonic-cfggen -M /etc/sonic/device_desc.xml --print-data 
{
    "DEVICE_METADATA": {
        "localhost": {
            "hostname": "CPQ21-0101-0509-04T0",
            "hwsku": "Arista-7260CX3-D108C8"
        }
    },
    "LOOPBACK_INTERFACE": {
        "lo_v6|2603:10d0:e:5eb::/128": {}         <==== missing standalone entry which fail YANG
    },
    "MGMT_INTERFACE": {
        "eth0|100.84.101.139/26": {
            "gwaddr": "100.84.101.129"
        },
        "eth0|2603:10e2:f0:7478::b/64": {
            "gwaddr": "2603:10e2:f0:7478::1"
        }
    }
}

After change:

admin@str-msn2700a1-03:~$ sonic-cfggen -M /etc/sonic/device_desc.xml --print-data 
{
    "DEVICE_METADATA": {
        "localhost": {
            "hostname": "CPQ21-0101-0509-04T0",
            "hwsku": "Arista-7260CX3-D108C8"
        }
    },
    "LOOPBACK_INTERFACE": {
        "lo_v6": {},                                          <=== added standalone entry
        "lo_v6|2603:10d0:e:5eb::/128": {}
    },
    "MGMT_INTERFACE": {
        "eth0|100.84.101.139/26": {
            "gwaddr": "100.84.101.129"
        },
        "eth0|2603:10e2:f0:7478::b/64": {
            "gwaddr": "2603:10e2:f0:7478::1"
        }
    }
}

mssonicbld avatar Jul 12 '24 01:07 mssonicbld

Original PR: https://github.com/sonic-net/sonic-buildimage/pull/19514

mssonicbld avatar Jul 12 '24 01:07 mssonicbld

/azp run

bingwang-ms avatar Aug 02 '24 22:08 bingwang-ms

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Aug 02 '24 22:08 azure-pipelines[bot]