Skip to content

Commit

Permalink
wait-online: by default not all interface need to be online
Browse files Browse the repository at this point in the history
Fixes an issue caused by ab3aed4 (v253).

By default, all managed interface need to be configured, and at least
one interface need to be online. Hence, offline interface should be ignored.

Fixes #29506.
  • Loading branch information
yuwata authored and bluca committed Feb 6, 2024
1 parent 0119370 commit 6f412c0
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/network/wait-online/manager.c
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,9 @@ bool manager_configured(Manager *m) {
}

r = manager_link_is_online(m, l, /* state_range = */ NULL);
if (r < 0 && !m->any) /* Unlike the above loop, unmanaged interfaces are ignored here. */
/* Unlike the above loop, unmanaged interfaces are ignored here. Also, Configured but offline
* interfaces are ignored. See issue #29506. */
if (r < 0 && r != -EADDRNOTAVAIL && !m->any)
return false;
if (r > 0) {
if (m->any)
Expand Down

0 comments on commit 6f412c0

Please sign in to comment.