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

[VS/DPB] Introducing port_breakout_config_db.json #6034

Merged
merged 1 commit into from
Nov 25, 2020

Conversation

vasant17
Copy link
Contributor

Introducing port_breakout_config_db.json, which will be used when you…breakout port with -l option

Signed-off-by: Vasant [email protected]

- Why I did it
To provide default config for new created ports as part of breakout
- How I did it
port_breakout_config_db.json will have configuration of ports that will be loaded on creating a port as part of dynamic port breakout. To make use of this feature you have run the breakout CLI command with -l option.
Example: SONiC$ config interface breakout Ethernet0 4x25G[10G] -l -y

In the above example, Ethernet0 will be deleted and Ethernet0, Ethernet1, Ethernet2, and Ethernet3 will be craeted. If there is any configuration related these four ports is present in port_breakout_config_db.json, it'll be automatically configured. For example, adding port to VLAN, ACL or configuring an interface IP.

The file that we are checking in here is the one required for DPB VS test cases to exercise/test -l option.
- How to verify it
By running VS test cases
- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@vasant17 vasant17 changed the title port_breakout_config_db.json [VS/DPB] Introducing port_breakout_config_db.json Nov 25, 2020
@daall daall merged commit 8f0452d into sonic-net:master Nov 25, 2020
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 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