Skip to content

Add routed port support to dhcp_relay#102

Open
devesh-nexthop wants to merge 1 commit into
sonic-net:masterfrom
nexthop-ai:dhcp-relay-routed-port
Open

Add routed port support to dhcp_relay#102
devesh-nexthop wants to merge 1 commit into
sonic-net:masterfrom
nexthop-ai:dhcp-relay-routed-port

Conversation

@devesh-nexthop

Copy link
Copy Markdown

Adding support for routed ports.

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@devesh-nexthop devesh-nexthop marked this pull request as ready for review May 5, 2026 05:02
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@devesh-nexthop devesh-nexthop force-pushed the dhcp-relay-routed-port branch from 8aac3d4 to 6e4c51b Compare May 31, 2026 06:53
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

Signed-off-by: Devesh Pathak <250206677+devesh-nexthop@users.noreply.github.com>
@devesh-nexthop devesh-nexthop force-pushed the dhcp-relay-routed-port branch from 6e4c51b to 4ebead6 Compare June 1, 2026 06:35
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

} else {
circuit_id = m_config.hostname + ":" + intf_alias + ":" + config->vlan;
/* No format specified, use default format */
if (feature_dhcp_server_enabled) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code can be optimized to reduce redundancy

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.

3 participants