/azp run

/azp run

/azp run

Hello,

This test case fails if it gonna be enabled for multi-asic.

The test case is suppossed to change the Loopback interface under /localhost namespace.
Patch that is applied successfully:
[{"op": "remove", "path": "/LOOPBACK_INTERFACE/Loopback0|10.1.0.1~132"}, {"op": "remove", "path": "/LOOPBACK_INTERFACE/Loopback0|FC00:10::1~1128"}, {"op": "add", "path": "/LOOPBACK_INTERFACE/Loopback0|10.1.0.210~132", "value": {}}, {"op": "add", "path": "/LOOPBACK_INTERFACE/Loopback0|FC00:1::210~1128", "value": {}}]

After that change running database has these values:

$ sonic-cfggen -d -v LOOPBACK_INTERFACE
{'Loopback0': {}, ('Loopback0', '10.1.0.210/32'): {}, ('Loopback0', 'FC00:1::210/128'): {}}

$ sonic-cfggen -n asic0 -d -v LOOPBACK_INTERFACE
{'Loopback0': {}, ('Loopback0', '10.1.0.1/32'): {}, ('Loopback0', 'FC00:10::1/128'): {}, 'Loopback4096': {}, ('Loopback4096', '2603:10e2:400::1/128'): {}, ('Loopback4096', '8.0.0.1/32'): {}}

$ sonic-cfggen -n asic1 -d -v LOOPBACK_INTERFACE
{'Loopback0': {}, ('Loopback0', '10.1.0.1/32'): {}, ('Loopback0', 'FC00:10::1/128'): {}, 'Loopback4096': {}, ('Loopback4096', '2603:10e2:400::2/128'): {}, ('Loopback4096', '8.0.0.2/32'): {}}

As you can see there is an incosistency for Loopback0 between /localhost and asic namespaces.

The test verifies based on "show ip int" command and doesnt' find the updated expected value and fails.

$ show ip int
Interface       Master    IPv4 address/mask    Admin/Oper    BGP Neighbor    Neighbor IP
--------------  --------  -------------------  ------------  --------------  -------------
Loopback0                 10.1.0.1/32          up/up         N/A             N/A

Which means the cli command "show ip int" fetches values based on namespaces cfg?

I am not sure what is the expected here.

We can modify the patch to apply change in localhost as well as in all asic namespaces in case of multiasci duthost and that will pass the case,
but my question is , is the output of "show ip int" expected in above scenario?

Thanks,
Olympia

/azp run

/azp run

@okaravasi there is a predependency check which is failing can you check

There is a cross-feature dependence. File: tests/generic_config_updater/test_lo_interface.py, import module: tests.bgp.bgp_helpers
There are cross-feature dependencies, which is not allowed in our repo
To resolve this issue, please move the shared function to common place, such as 'tests/common'

/azp run