Skip to content

Comments

Rdkb 63584: srv_unregister remove DHCPManager Distro#234

Merged
hpatel006c merged 3 commits intodevelopfrom
RDKB-63584
Feb 20, 2026
Merged

Rdkb 63584: srv_unregister remove DHCPManager Distro#234
hpatel006c merged 3 commits intodevelopfrom
RDKB-63584

Conversation

@aadhithan01
Copy link
Contributor

@aadhithan01 aadhithan01 commented Feb 20, 2026

Rdkb 63584: srv_unregister remove DHCPManager Distro

This PR removes the DHCP Manager distro-specific compile guard in the DHCP server registration helper so that srv_unregister() always unregisters dhcp_server from pmon, aligning unregister behavior with the existing unconditional srv_register() behavior.

Changes:

Remove the #if !defined(FEATURE_RDKB_DHCP_MANAGER) guard around pmon.sh unregister dhcp_server.
Drop the associated comment about the DHCP Manager self-heal mechanism.

Copilot AI review requested due to automatic review settings February 20, 2026 05:11
@aadhithan01 aadhithan01 requested review from a team as code owners February 20, 2026 05:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the DHCP Manager distro-specific compile guard in the DHCP server registration helper so that srv_unregister() always unregisters dhcp_server from pmon, aligning unregister behavior with the existing unconditional srv_register() behavior.

Changes:

  • Remove the #if !defined(FEATURE_RDKB_DHCP_MANAGER) guard around pmon.sh unregister dhcp_server.
  • Drop the associated comment about the DHCP Manager self-heal mechanism.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hpatel006c hpatel006c merged commit d366f37 into develop Feb 20, 2026
9 of 11 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants