[201811] Port changes for 7050QX-32S-S4Q31 into 201811 branch #3570
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…onic-net#3527) * minigraph: Add the ability to set a per-port speed in the minigraph In the case of devices with mixed port speeds, the speed for each port has to be individually specified in the minigraph, and a single speed for the entire device cannot be used. Instead, read the connection graph, which contains the speed for each individual link, and use the speed specified there for each port. The speed specified in the connection graph is used only if port_config.ini doesn't have a speed specified there. If there is a speed specified there, then that is used. If no speed is specified in port_config.ini, and there's no speed specified for the port in the connection graph, then the speed in the inventory yaml file is used. Signed-off-by: Saikrishna Arcot <saiarcot895@gmail.com> (cherry picked from commit cef1f77)
That entry needs to be specified as referring to a variable. Signed-off-by: Saikrishna Arcot <saiarcot895@gmail.com>
Signed-off-by: Saikrishna Arcot <saiarcot895@gmail.com> Adds a topology definition for 31 40G ports and 4 10G/1G ports, where those 10G ports are not used for automated tests. What is the motivation for this PR? This is for use with Arista 7050QX-32S-S4Q31. How did you verify/test it? Deployed a minigraph with this topology and checked to make sure that the VLAN assignments and the port channel assignments were correct. No automated tests were run. (cherry picked from commit 6d1720b)
saiarcot895
changed the title
neethajohn
approved these changes
Jun 2, 2021
neethajohn
changed the title
yxieca
approved these changes
Jun 2, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary:
Port changes made for bringing up 7050QX-32S-S4Q31 from the master branch to the 201811 branch.
This PR includes the minigraph generation changes and the new topology.
Type of change
Testing
Verified that at least one of the current setups successfully generated the same minigraph.