[minigraph] Support parse IPv6 in device_desc.xml#11095
Merged
Blueve merged 9 commits intosonic-net:masterfrom Jun 28, 2022
Merged
[minigraph] Support parse IPv6 in device_desc.xml#11095Blueve merged 9 commits intosonic-net:masterfrom
Blueve merged 9 commits intosonic-net:masterfrom
Conversation
|
This pull request fixes 1 alert when merging e50c789 into 0b1ae9c - view on LGTM.com fixed alerts:
|
Blueve
added
Request for 201911 Branch 
Request for 202006 Branch
Request for 202012 Branch
Request for 202106 Branch
Request for 202111 Branch
6 tasks
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
|
This pull request fixes 1 alert when merging 3dd201f into c1d0d2e - view on LGTM.com fixed alerts:
|
|
This pull request fixes 1 alert when merging 1feee4d into c1d0d2e - view on LGTM.com fixed alerts:
|
qiluo-msft
reviewed
Jun 23, 2022
|
This pull request fixes 1 alert when merging ecdefad into 4afe209 - view on LGTM.com fixed alerts:
|
ganglyu
approved these changes
Jun 24, 2022
Blueve
added a commit
to Blueve/sonic-buildimage
that referenced
this pull request
Jun 28, 2022
Signed-off-by: Jing Kan jika@microsoft.com
7 tasks
Blueve
added a commit
to Blueve/sonic-buildimage
that referenced
this pull request
Jun 28, 2022
Signed-off-by: Jing Kan jika@microsoft.com
7 tasks
yxieca
pushed a commit
that referenced
this pull request
Jun 28, 2022
Signed-off-by: Jing Kan jika@microsoft.com
Blueve
added a commit
that referenced
this pull request
Jul 1, 2022
Blueve
added a commit
to Blueve/sonic-buildimage
that referenced
this pull request
Jul 5, 2022
sonic-net#11095) (sonic-net#11273) * [minigraph] Support parse IPv6 in device_desc.xml (sonic-net#11095) Signed-off-by: Jing Kan jika@microsoft.com
7 tasks
skbarista
pushed a commit
to skbarista/sonic-buildimage
that referenced
this pull request
Aug 17, 2022
Signed-off-by: Jing Kan jika@microsoft.com
qiluo-msft
reviewed
Jan 26, 2024
| <ManagementAddress xmlns:a="Microsoft.Search.Autopilot.NetMux"> | ||
| <a:IPPrefix>0.0.0.0/0</a:IPPrefix> | ||
| </ManagementAddress> | ||
| <ManagementAddressV6 xmlns:a="Microsoft.Search.Autopilot.NetMux"> |
Collaborator
qiluo-msft
reviewed
Jan 26, 2024
| <Hostname>switch-t0</Hostname> | ||
| <HwSku>Force10-S6000</HwSku> | ||
| <ClusterName>AAA00PrdStr00</ClusterName> | ||
| <ManagementAddress xmlns:a="Microsoft.Search.Autopilot.NetMux"> |
Collaborator
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Why I did it
minigraph parser can parse initial configuration from a fragment of minigraph (device_desc.xml), however it can only get IPv4 address from it and due to some reason, if the device only has IPv6 management IP address, then the IPv4 address will be
0.0.0.0/0which will causeconfig load_mgmt_configcommand failHow I did it
0.0.0.0/0to mgmt configHow to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)