[Static DNS]fix static dns issue#11765
Conversation
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
5fd19b2 to
055cb7b
Compare
1. Need to check if the dhclient pid exist or not or the dut host. 2. Save the dhclient pid under the /run folder instead of the /tmp folder, When dhclient runs it drops the permissions to meet the required security level. With the dropped per in the Debian 12, it can’t create a PID file in a /tmp/ directory. Instead /run/ directory should be used.
055cb7b to
abbf97b
Compare
ganglyu
left a comment
There was a problem hiding this comment.
I have verified this PR.
Need to check if the dhclient pid exist or not on the dut host. ## Type of change - [x] Bug fix - [ ] Testbed and Framework(new/improvement) - [ ] Test case(new/improvement)
|
Cherry-pick PR to 202311: #12110 |
Need to check if the dhclient pid exist or not on the dut host. ## Type of change - [x] Bug fix - [ ] Testbed and Framework(new/improvement) - [ ] Test case(new/improvement)
Need to check if the dhclient pid exist or not on the dut host.
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
How did you do it?
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation