Skip to content

[dhcp_server] Fix dhcp_server mk file mistake #19847

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

Merged
merged 3 commits into from
Aug 8, 2024

Conversation

yaqiangz
Copy link
Contributor

@yaqiangz yaqiangz commented Aug 7, 2024

Why I did it

Startup dependency is missing for dhcp_server, which would cause that it startup before swss.

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

How I did it

Fix the startup missing issue

How to verify it

Build image and install to check

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@yaqiangz yaqiangz changed the title [dhcp_server] Fix dhcp_serverer mk file mistake [dhcp_server] Fix dhcp_server mk file mistake Aug 7, 2024
@yaqiangz yaqiangz marked this pull request as ready for review August 7, 2024 10:54
@yaqiangz
Copy link
Contributor Author

yaqiangz commented Aug 8, 2024

Hi @yxieca could you please help to merge this PR?

@yxieca yxieca merged commit 2ab00c7 into sonic-net:master Aug 8, 2024
22 checks passed
@yaqiangz
Copy link
Contributor Author

yaqiangz commented Aug 9, 2024

Hi @bingwang-ms could you please help to approve the backport request?

1 similar comment
@yaqiangz
Copy link
Contributor Author

Hi @bingwang-ms could you please help to approve the backport request?

matiAlfaro pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Aug 21, 2024
* [dhcp_server] Fix dhcp_serverer mk file mistake

* Set include_dhcp_server to y

* Revert "Set include_dhcp_server to y"

This reverts commit 88f0356.
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Aug 23, 2024
* [dhcp_server] Fix dhcp_serverer mk file mistake

* Set include_dhcp_server to y

* Revert "Set include_dhcp_server to y"

This reverts commit 88f0356.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #20004

mssonicbld pushed a commit that referenced this pull request Aug 26, 2024
* [dhcp_server] Fix dhcp_serverer mk file mistake

* Set include_dhcp_server to y

* Revert "Set include_dhcp_server to y"

This reverts commit 88f0356.
yanjundeng pushed a commit to yanjundeng/sonic-buildimage that referenced this pull request Apr 23, 2025
* [dhcp_server] Fix dhcp_serverer mk file mistake

* Set include_dhcp_server to y

* Revert "Set include_dhcp_server to y"

This reverts commit 88f0356.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants