Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[dhcp_server] Add dhcp_server container #14031

Merged
merged 17 commits into from
Sep 11, 2023

Conversation

yaqiangz
Copy link
Contributor

@yaqiangz yaqiangz commented Mar 1, 2023

Why I did it

Add dhcp_server ipv4 feature to SONiC.
HLD: sonic-net/SONiC#1282

Work item tracking
  • Microsoft ADO (number only): 24978865

How I did it

To be clarify: This container is disabled by INCLUDE_DHCP_SERVER = n for now, which would cause container not build.

  1. Add INCLUDE_DHCP_SERVER to indicate whether to build dhcp_server container
  2. Add docker file for dhcp_server, build and install kea-dhcp4 inside container
  3. Add template file for dhcp_server container services.
  4. Add entry for dhcp_server to FEATURE table in config_db.

How to verify it

Build image with INCLUDE_DHCP_SERVER = y to verify:

  1. Image can be install successfully without crush.
  2. By config feature state dhcp_server enabled to enable dhcp_server.
  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@yaqiangz yaqiangz closed this Mar 1, 2023
@yaqiangz yaqiangz reopened this Aug 4, 2023
@yaqiangz yaqiangz force-pushed the master_check_submodule branch 10 times, most recently from c6961fa to 48cd751 Compare August 10, 2023 14:42
@yaqiangz yaqiangz force-pushed the master_check_submodule branch 4 times, most recently from 88a37b9 to 8ef454f Compare August 15, 2023 08:56
@yaqiangz yaqiangz force-pushed the master_check_submodule branch 2 times, most recently from eece0d2 to 4472b5d Compare August 17, 2023 13:29
@yaqiangz yaqiangz changed the title [test] Just want to build [dhcp_server] Add dhcp_server container Aug 18, 2023
@yaqiangz yaqiangz marked this pull request as ready for review August 31, 2023 02:34
@yxieca yxieca merged commit 76b7cb8 into sonic-net:master Sep 11, 2023
17 checks passed
yaqiangz added a commit to yaqiangz/sonic-buildimage that referenced this pull request Sep 12, 2023
Why I did it
Add dhcp_server ipv4 feature to SONiC.
HLD: sonic-net/SONiC#1282

How I did it
To be clarify: This container is disabled by INCLUDE_DHCP_SERVER = n for now, which would cause container not build.

Add INCLUDE_DHCP_SERVER to indicate whether to build dhcp_server container
Add docker file for dhcp_server, build and install kea-dhcp4 inside container
Add template file for dhcp_server container services.
Add entry for dhcp_server to FEATURE table in config_db.
How to verify it
Build image with INCLUDE_DHCP_SERVER = y to verify:

Image can be install successfully without crush.
By config feature state dhcp_server enabled to enable dhcp_server.
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
Why I did it
Add dhcp_server ipv4 feature to SONiC.
HLD: sonic-net/SONiC#1282

How I did it
To be clarify: This container is disabled by INCLUDE_DHCP_SERVER = n for now, which would cause container not build.

Add INCLUDE_DHCP_SERVER to indicate whether to build dhcp_server container
Add docker file for dhcp_server, build and install kea-dhcp4 inside container
Add template file for dhcp_server container services.
Add entry for dhcp_server to FEATURE table in config_db.
How to verify it
Build image with INCLUDE_DHCP_SERVER = y to verify:

Image can be install successfully without crush.
By config feature state dhcp_server enabled to enable dhcp_server.
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