Skip to content

aii-ks: Only wait for 120 seconds in wait_for_network#360

Merged
wpoely86 merged 1 commit intoquattor:mainfrom
jrha:fix_wait_for_network
Jan 14, 2026
Merged

aii-ks: Only wait for 120 seconds in wait_for_network#360
wpoely86 merged 1 commit intoquattor:mainfrom
jrha:fix_wait_for_network

Conversation

@jrha
Copy link
Member

@jrha jrha commented Jan 13, 2026

The previous code assumed that nslookup failed instantly, however the default behaviour makes three requests, each with a timeout of 5 seconds, so a host with completely broken networking will take 120 * (15 + 1) seconds (32 minutes) to error out.

Fixes #354.

@jrha jrha added this to the 26.1 milestone Jan 13, 2026
@jrha jrha added the bug label Jan 13, 2026
The previous code assumed that nslookup failed instantly, however the default behaviour makes three requests, each with a timeout of 5 seconds, so a host with completely broken networking will take 120 * (15 + 1) seconds (32 minutes) to error out.
@jrha jrha force-pushed the fix_wait_for_network branch from 987f888 to 333e0d7 Compare January 13, 2026 15:02
@wpoely86 wpoely86 merged commit 942f805 into quattor:main Jan 14, 2026
5 of 6 checks passed
@jrha jrha deleted the fix_wait_for_network branch January 14, 2026 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

wait_for_network waits longer than two minutes

2 participants