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

test_neighbor_mac_noptf failing in multi asic devices #3671

Closed
arlakshm opened this issue Jun 15, 2021 · 0 comments · Fixed by #3672
Closed

test_neighbor_mac_noptf failing in multi asic devices #3671

arlakshm opened this issue Jun 15, 2021 · 0 comments · Fixed by #3672
Assignees
Labels

Comments

@arlakshm
Copy link
Contributor

Description

The test in file arp/test_neighbor_mac_noptf are failing on multi asic platforms. The tests fail with following error

self = <tests.arp.test_neighbor_mac_noptf.TestNeighborMacNoPtf instance at 0x7f11261d6780>
duthosts = <tests.common.devices.duthosts.DutHosts object at 0x7f1125516250>
enum_rand_one_per_hwsku_frontend_hostname = 'svcstr-nsonic-acs-1'

    @pytest.fixture(scope="module", autouse=True)
    def setupDutConfig(self, duthosts, enum_rand_one_per_hwsku_frontend_hostname):
        """
            Disabled BGP to reduce load on switch and restores DUT configuration after test completes
    
            Args:
                duthost (AnsibleHost): Device Under Test (DUT)
    
            Returns:
                None
        """
        duthost = duthosts[enum_rand_one_per_hwsku_frontend_hostname]
        duthost.command("sudo config bgp shutdown all")
        if not wait_until(120, 2.0, self._check_no_bgp_routes, duthost):
>           pytest.fail('BGP Shutdown Timeout: BGP route removal exceeded 120 seconds.')
E           Failed: BGP Shutdown Timeout: BGP route removal exceeded 120 seconds.

duthost    = <MultiAsicSonicHost> svcstr-nsonic-acs-1
duthosts   = <tests.common.devices.duthosts.DutHosts object at 0x7f1125516250>
enum_rand_one_per_hwsku_frontend_hostname = 'svcstr-nsonic-acs-1'
self       = <tests.arp.test_neighbor_mac_noptf.TestNeighborMacNoPtf instance at 0x7f11261d6780>

The PR #3369 introduced the change to count the routes. This change doesn't work on multi asic platform.

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

Successfully merging a pull request may close this issue.

1 participant