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

IPinIP test case support for Standalone TH2 (7260CX3) to modify config DB DEVICE_METADATA localhost subtype #3256

Closed
gechiang opened this issue Mar 31, 2021 · 1 comment · Fixed by #3110
Assignees
Labels
Dual ToR Orchagent Test Dev ⭕ Action item for orchagent-specific tests for dual ToR Enhancement

Comments

@gechiang
Copy link
Contributor

Description

Need testcase support for IPinIP testing on Standalone TH2 SKU (7260CX3).
BRCM has implemented new IPinIP tunnel support that is needed for DualToR. The support requires new SOC properties enabled at boot time so that the new IPinIP implementation would be used. Currently this SOC properties are only applied if the DUT is configured for DualToR in the config DB. But if 7260CX3 is a standalone DUT, This configuration would never be set.
Since we like to be able to test the new IPinIP functionality with standalone 7260CX3 SKUs, it requires the testcase be changed to first modify the config DB of the standalone 7260CX3 to add "subtype": "DualToR" under the DEVICE_METADATA: "localhost" section:

DEVICE_METADATA": {
        "localhost": {
            "bgp_asn": "4200065100",
            "buffer_model": "traditional",
            "cloudtype": "None",
            "default_bgp_status": "up",
            "default_pfcwd_status": "enable",
            "deployment_id": "1",
            "docker_routing_config_mode": "separated",
            "hostname": "str-7260cx3-acs-2",
            "hwsku": "Arista-7260CX3-D108C8",
            "mac": "74:83:ef:09:33:1c",
            "platform": "x86_64-arista_7260cx3_64",
            "region": "None",
            "synchronous_mode": "enable",
            "subtype": "DualToR",
            "type": "ToRRouter”
        }
    },

Once this is added, reload the SWSS and SYNCD dockers so that the ASIC will be booted up with proper config.bcm properties.
Run the IPinIP tests
At the end o f the test case modify the config DB to remove the "DualToR" setting.

Please be sur if the "DualToR" setting was ever modified in configDB as part of the testcase, it must always remove it no matter what is the result of the testcase.

This testcase change is pending the following PR:
sonic-net/sonic-buildimage#7168
Please make sure that the testcase change is only applicable for master image or 202012 based image (when the above PR is also part of 202012).

@bingwang-ms
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dual ToR Orchagent Test Dev ⭕ Action item for orchagent-specific tests for dual ToR Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants