Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: srv6/test_srv6_basic_sanity.py failed due to KeyError #15561

Open
ZhaohuiS opened this issue Nov 14, 2024 · 1 comment
Open

[Bug]: srv6/test_srv6_basic_sanity.py failed due to KeyError #15561

ZhaohuiS opened this issue Nov 14, 2024 · 1 comment

Comments

@ZhaohuiS
Copy link
Contributor

ZhaohuiS commented Nov 14, 2024

Issue Description

srv6/test_srv6_basic_sanity.py failed due to KeyError

Results you see

=================================== FAILURES ===================================
_________________________ test_interface_on_each_node __________________________

duthosts = [<MultiAsicSonicHost bjw2-can-8102-6>]
rand_one_dut_hostname = 'bjw2-can-8102-6'
nbrhosts = {'ARISTA01T1': <EosHost VM85115>, 'ARISTA02T1': <EosHost VM85116>, 'ARISTA03T1': <EosHost VM85117>, 'ARISTA04T1': <EosHost VM85118>}

    def test_interface_on_each_node(duthosts, rand_one_dut_hostname, nbrhosts):
        for vm_name in test_vm_names:
>           nbrhost = nbrhosts[vm_name]['host']
E           KeyError: 'PE1'

duthosts   = [<MultiAsicSonicHost bjw2-can-8102-6>]
nbrhosts   = {'ARISTA01T1': <EosHost VM85115>, 'ARISTA02T1': <EosHost VM85116>, 'ARISTA03T1': <EosHost VM85117>, 'ARISTA04T1': <EosHost VM85118>}
rand_one_dut_hostname = 'bjw2-can-8102-6'
vm_name    = 'PE1'

srv6/test_srv6_basic_sanity.py:81: KeyError

Results you expected to see

This case should be passed

Is it platform specific

generic

Relevant log output

srv6.test_srv6_basic_sanity.test_interface_on_each_node | failure | KeyError: 'PE1'
srv6.test_srv6_basic_sanity.test_check_bgp_neighbors | failure | KeyError: 'PE3'

Output of show version

No response

Attach files (if any)

No response

@ZhaohuiS
Copy link
Contributor Author

#15564

To skip it for non cisco vs topologies.
This script should not run on any topologies.

pytest.mark.topology("any"),

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant