Skip to content

[multi-asic][vs]: Use the right KVM platform string for multi-asic vs#4335

Merged
SuvarnaMeenakshi merged 1 commit intosonic-net:masterfrom
SuvarnaMeenakshi:masic_platform
Sep 24, 2021
Merged

[multi-asic][vs]: Use the right KVM platform string for multi-asic vs#4335
SuvarnaMeenakshi merged 1 commit intosonic-net:masterfrom
SuvarnaMeenakshi:masic_platform

Conversation

@SuvarnaMeenakshi
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911

Approach

What is the motivation for this PR?

Error is seen when doing add-topo to bring up a multi-asic testbed.

./testbed-cli.sh -t vtestbed.csv -m veos_vtb add-topo vms-kvm-four-asic-t1-lag password.txt
..
TASK [find interface name mapping and individual interface speed if defined from dut] ***
Wednesday 22 September 2021  00:05:06 +0000 (0:00:00.937)       0:00:05.518 *** 
[WARNING]: Platform linux on host vlab-08 is using the discovered Python
interpreter at /usr/bin/python, but future installation of another Python
interpreter could change this. See https://docs.ansible.com/ansible/2.8/referen
ce_appendices/interpreter_discovery.html for more information.
fatal: [vlab-08]: FAILED! => {"ansible_facts": {"discovered_interpreter_python": "/usr/bin/python"}, "changed": false, "msg": "failed to find the correct port config for msft_four_asic_vslist index out of range"

Reason:
sonic-net/sonic-buildimage#8269 added new multi-asic KVM platforms, for 4-asic and 6-asic VS.
When add-topo is done, port_alias.py runs on localhost to get the right front-panel interfaces so that VS can be brought up with the right number of front-panel interfaces. With the above PR, new platform string was added due to which port-config.ini file was under a new KVM platform directory, due to which the port config file was not being retrieved correctly.

How did you do it?

Based on number of asics, use the right KVM platform string so that the right port-config file can be retrieved when bringing up VS testbed.

How did you verify/test it?

Was able to bring up VS testbed without the above error.

Any platform specific information?

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

Documentation

vs.  New multi-asic KVM platform strings were added. This string
is required to get the right port-config file when bringing up
virtual switch.

Signed-off-by: Suvarna Meenakshi <sumeenak@microsoft.com>
@SuvarnaMeenakshi SuvarnaMeenakshi merged commit 8457ef9 into sonic-net:master Sep 24, 2021
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

Successfully merging this pull request may close these issues.

2 participants