Skip to content

add csonic testbed support#1720

Draft
lguohan wants to merge 2 commits intosonic-net:masterfrom
lguohan:csonic
Draft

add csonic testbed support#1720
lguohan wants to merge 2 commits intosonic-net:masterfrom
lguohan:csonic

Conversation

@lguohan
Copy link
Copy Markdown
Contributor

@lguohan lguohan commented Jun 2, 2020

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Approach

How did you do it?

add csonic as sonic neighbor device

depends on sonic-net/sonic-buildimage#4689

How did you verify/test it?

./testbed-cli.sh -t vtestbed.csv -m veos.vtb -k csonic add-topo vms-kvm-t0-2 password.txt

Any platform specific information?

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

Documentation

network_mode: container:net_{{ vm_set_name }}_{{ inventory_hostname }}
detach: True
capabilities:
- net_admin
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to support ipv6


[eos:children]
#[eos:children]
#vms_1
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to uncomment these

@@ -0,0 +1,41 @@
{% set host = configuration[hostname] %}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to support backplane interface

{% if iface['lacp'] is defined %}
"PORTCHANNEL_MEMBER": {
{% set index = name | replace("Ethernet", "") | int - 1 %}
"PortChannel{{ iface['lacp'] }}|Ethernet{{ index * 4}}": {}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to bring up the Ethernet interface

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 2, 2020

This pull request introduces 4 alerts and fixes 4 when merging 9ebab9c into 031dd2a - view on LGTM.com

new alerts:

  • 2 for Unused import
  • 1 for Module is imported more than once
  • 1 for 'import *' may pollute namespace

fixed alerts:

  • 2 for Unused import
  • 1 for Module is imported more than once
  • 1 for 'import *' may pollute namespace

kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…tically (#24805)

#### Why I did it
src/sonic-sairedis
```
* 65c8a8fc - (HEAD -> 202505, origin/202505) [202505][ci] Use 1ES agent pool to provide more disk when building swss. (sonic-net#1722) (34 hours ago) [prabhataravind]
* b4e0e9fe - Use sonic build pool for building sairedis (sonic-net#1720) (2 days ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

1 participant