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

sources/azure: retry dhcp for failed processes #1401

Merged
merged 1 commit into from
Apr 25, 2022

Conversation

cjp256
Copy link
Contributor

@cjp256 cjp256 commented Apr 25, 2022

Commands may fail in the process of setting up DHCP, e.g.:
udevadm settle, ip link set dev eth0 up, etc.

Report these failures and retry until timeout.

Signed-off-by: Chris Patterson cpatterson@microsoft.com

Commands may fail in the process of setting up DHCP, e.g.:
udevadm settle, ip link set dev eth0 up, etc.

Report these failures and retry until timeout.

Signed-off-by: Chris Patterson <cpatterson@microsoft.com>
@cjp256 cjp256 force-pushed the azure-retry-dhcp-setup-failures branch from 0ea2973 to 40c971c Compare April 25, 2022 14:27
@cjp256 cjp256 changed the title sources/azure: retry dhcp setup for failed processes sources/azure: retry dhcp for failed processes Apr 25, 2022
Copy link

@vmarcella vmarcella left a comment

Choose a reason for hiding this comment

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

These changes look good to me.

Copy link
Member

@holmanb holmanb left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@holmanb holmanb merged commit 43c1cf4 into canonical:main Apr 25, 2022
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