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

Fix dhcpcd arguments for static IP config #4659

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

milan-zededa
Copy link
Contributor

@milan-zededa milan-zededa commented Mar 4, 2025

Commit bcb650f (pillar: enhance NTP server handling) introduced a bug in DhcpcdConfigurator (EVE version 14.0.0), where certain arguments for dhcpcd may be unintentionally omitted in static IP configurations.

Specifically, the configuration file option (-f), background execution (-b), and infinite timeout (-t 0) may be missing. As a result, if dhcpcd starts while the network cable is unplugged, it will timeout after 30 seconds (default timeout) and exit. Consequently, even after the cable is reconnected, dhcpcd will no longer be running to apply the static IP.

Commit bcb650f (pillar: enhance NTP server handling) introduced
a bug in DhcpcdConfigurator (EVE version 14.0.0), where certain
arguments for dhcpcd may be unintentionally omitted in static IP
configurations.

Specifically, the configuration file option (-f), background
execution (-b), and infinite timeout (-t 0) may be missing.
As a result, if dhcpcd starts while the network cable is unplugged,
it will time out after 30 seconds and exit. Consequently, even
after the cable is reconnected, dhcpcd will no longer be running
to apply the static IP.

Signed-off-by: Milan Lenco <[email protected]>
@milan-zededa milan-zededa added the bug Something isn't working label Mar 4, 2025
@milan-zededa milan-zededa self-assigned this Mar 4, 2025
Copy link
Contributor

@rene rene left a comment

Choose a reason for hiding this comment

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

LGTM

@eriknordmark eriknordmark merged commit 057995a into lf-edge:master Mar 5, 2025
66 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants