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

[l2-preset]: Add all dual ToR required fields #7318

Merged
merged 4 commits into from
Apr 15, 2021

Conversation

theasianpianist
Copy link
Contributor

@theasianpianist theasianpianist commented Apr 14, 2021

Signed-off-by: Lawrence Lee lawlee@microsoft.com

Why I did it

How I did it

  • Add peer_switch field to DEVICE_METADATA table
  • In PORT table:
    • Set fec value for uplinks
    • Set used ports to admin status up
    • Set mux_cable to true for downlinks in use
  • In MUX_CABLE table:
    • Only add entry if the downlink is in use

How to verify it

Run test_j2files.py

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
* start server IP offset at 1
* remove FEC from port table

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@theasianpianist theasianpianist requested a review from prsunny April 14, 2021 20:47
@lguohan lguohan merged commit 39a027b into sonic-net:master Apr 15, 2021
lguohan pushed a commit that referenced this pull request Apr 15, 2021
- Add peer_switch field to DEVICE_METADATA table
- In PORT table:
   - Set used ports to admin status up
   - Set mux_cable to true for downlinks in use
- In MUX_CABLE table:
   - Only add entry if the downlink is in use

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@theasianpianist theasianpianist deleted the l2-fix branch May 13, 2021 23:11
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
- Add peer_switch field to DEVICE_METADATA table
- In PORT table:
   - Set used ports to admin status up
   - Set mux_cable to true for downlinks in use
- In MUX_CABLE table:
   - Only add entry if the downlink is in use

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
- Add peer_switch field to DEVICE_METADATA table
- In PORT table:
   - Set used ports to admin status up
   - Set mux_cable to true for downlinks in use
- In MUX_CABLE table:
   - Only add entry if the downlink is in use

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
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.

4 participants