Skip to content

[202012] Automatically enable tunnel_qos_remap on T1 and T0 in DualToR deployment#11056

Merged
bingwang-ms merged 2 commits intosonic-net:202012from
bingwang-ms:auto_generate_system_default
Jun 9, 2022
Merged

[202012] Automatically enable tunnel_qos_remap on T1 and T0 in DualToR deployment#11056
bingwang-ms merged 2 commits intosonic-net:202012from
bingwang-ms:auto_generate_system_default

Conversation

@bingwang-ms
Copy link
Contributor

Signed-off-by: bingwang wang.bing@microsoft.com

Why I did it

This PR is to enable tunnel_qos_remap on T1 and T0 in DualToR deployment.
On T1, we check the property DownstreamRedundancyTypes. On T0, we check the property RedundancyType.
tunnel_qos_remap is set to enabled if gemini is in DownstreamRedundancyTypes (on T1) or RedundancyType (on T0).

How I did it

The change is implemented in minigraph.py.

How to verify it

Verified by test_minigraph_case.py and 'test_j2files.py`.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Automatically enable tunnel_qos_remap on T1 and T0 in DualToR deployment.

Link to config_db schema for YANG module changes

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

Signed-off-by: bingwang <wang.bing@microsoft.com>
@neethajohn neethajohn requested a review from prsunny June 7, 2022 19:58
@bingwang-ms
Copy link
Contributor Author

PR test will be unblocked by this PR sonic-net/sonic-mgmt#5786

@neethajohn
Copy link
Contributor

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms bingwang-ms merged commit adfe20b into sonic-net:202012 Jun 9, 2022
@stephenxs
Copy link
Collaborator

@bingwang-ms do we have this change in master?

neethajohn pushed a commit that referenced this pull request Jul 25, 2022
…ent (#11508)

Why I did it
This PR is to backport PR #11056 and PR #11045 into master branch.
This PR is to enable tunnel_qos_remap on T1 and T0 in DualToR deployment.
On T1, we check the property DownstreamRedundancyTypes. On T0, we check the property RedundancyType.
tunnel_qos_remap is set to enabled if gemini is in DownstreamRedundancyTypes (on T1) or RedundancyType (on T0).

How I did it
The change is implemented in minigraph.py.

How to verify it
Verified by test_minigraph_case.py and 'test_j2files.py`.
yxieca pushed a commit that referenced this pull request Jul 28, 2022
…ent (#11508)

Why I did it
This PR is to backport PR #11056 and PR #11045 into master branch.
This PR is to enable tunnel_qos_remap on T1 and T0 in DualToR deployment.
On T1, we check the property DownstreamRedundancyTypes. On T0, we check the property RedundancyType.
tunnel_qos_remap is set to enabled if gemini is in DownstreamRedundancyTypes (on T1) or RedundancyType (on T0).

How I did it
The change is implemented in minigraph.py.

How to verify it
Verified by test_minigraph_case.py and 'test_j2files.py`.
skbarista pushed a commit to skbarista/sonic-buildimage that referenced this pull request Aug 17, 2022
…ent (sonic-net#11508)

Why I did it
This PR is to backport PR sonic-net#11056 and PR sonic-net#11045 into master branch.
This PR is to enable tunnel_qos_remap on T1 and T0 in DualToR deployment.
On T1, we check the property DownstreamRedundancyTypes. On T0, we check the property RedundancyType.
tunnel_qos_remap is set to enabled if gemini is in DownstreamRedundancyTypes (on T1) or RedundancyType (on T0).

How I did it
The change is implemented in minigraph.py.

How to verify it
Verified by test_minigraph_case.py and 'test_j2files.py`.
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.

5 participants