Enhance lldpd.conf.j2 for IPv6 support#24468
Conversation
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@arlakshm : this need cherry-pick for msft 202405. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds IPv6 support to the LLDP daemon configuration template by enabling IPv6 management interface configuration alongside the existing IPv4 support.
- Adds IPv6 address extraction from management interfaces
- Implements conditional logic to configure management IP pattern for IPv6-only or IPv4+IPv6 scenarios
- Prioritizes IPv4 over IPv6 when both are present
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/Azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@abdosi is this a test gap? please raise a test issue, thanks. |
|
@abdosi - this is a generic issue applicable to all sonic roles right |
|
/azp run |
|
You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list. |
|
/azp run |
|
You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list. |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
kenneth-arista
left a comment
kenneth-arista
left a comment
There was a problem hiding this comment.
Looks good to me
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
abdosi
dismissed stale reviews from mlok-nokia, kenneth-arista, and anamehra
via
b9d7edf
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Add support for IPv6 management interface configuration. Signed-off-by: abdosi <58047199+abdosi@users.noreply.github.com>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@abdosi do we have sonic-mgmt test? |
|
Cherry-pick PR to msft-202405: Azure/sonic-buildimage-msft#2066 |
|
Cherry-pick PR to 202511: #26179 |
Enhance lldpd.conf.j2 for IPv6 support
mssonicbld
added
Included in 202511 Branch
and removed
Created PR to 202511 Branch
labels
What I did:
Added support for parsing MGMT interface for ipv6 address also.
With this we will first try ipv4 address as system management ip and than fallback to ipv6.
Alias will be used if any of ip address is present.
Why I did:
Without parsing ipv6 address in ipv6 only network mgmt port alias does not geta advertised.
How I verify:
Manual Verification