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

[ansible/MCLAG]: add t0-mclag and t0-mclag-16 typologies support #1251

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

William-zx
Copy link
Contributor

@William-zx William-zx commented Dec 2, 2019

Description of PR

Summary:
Submit deploy topology t0-mclag and t0-mclag-16 according to mclag test plan Azure/SONiC/pull/422.

It works together with below PRs:

• MCLAG feature for SONIC Azure/sonic-buildimage/pull/2514

• [tests/mclag] Submit mclag testcases according to mclag test plan Azure/sonic-mgmt/pull/1239

Type of change

  • [] Bug fix
  • Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

Approach

How did you do it?

  1. Add t0-mclag-16 or t0-mclag topologies.
    ./testbed-cli.sh add-topo vms-t0-mclag-16 password.txt
    add_topo_mclag-16.txt

  2. The default configuration is deployed separately on each device, using the additional parameter dut_no.
    ansible-playbook -i lab config_sonic_basedon_testbed.yml -l sonic-dut-01 -e vm_base="VM0300" -e topo=t0-mclag-16 -e deploy=True -e save=True -e save=True -e testbed_name=vms-t0-mclag-16 -e dut_no=1
    generate_configuration_for_dut1.txt
    ansible-playbook -i lab config_sonic_basedon_testbed.yml -l sonic-dut-02 -e vm_base="VM0300" -e topo=t0-mclag-16 -e deploy=True -e save=True -e testbed_name=vms-t0-mclag-16-dut2 -e dut_no=2

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

t0-mclag and t0-mclag-16

Documentation

https://github.com/shine4chen/SONiC/blob/mclag-test-case/doc/mclag_hld/MCLAG-ansible-test-plan.md

@msftclas
Copy link

msftclas commented Dec 2, 2019

CLA assistant check
All CLA requirements met.

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

Successfully merging this pull request may close these issues.

3 participants