<div dir="ltr">Will do!<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 28, 2015 at 9:41 PM, Lennart Poettering <span dir="ltr"><<a href="mailto:lennart@poettering.net" target="_blank">lennart@poettering.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, 11.01.15 18:18, Charles Devereaux (<a href="mailto:systemd@guylhem.net">systemd@guylhem.net</a>) wrote:<br>
<br>
> What about another option in networkd-wait-online-link.c to pass an<br>
> "ignore" parameter, along with a list of interfaces names, so that<br>
> link_relevant does: something like:<br>
><br>
>         if (l->flags & IFF_LOOPBACK)<br>
>                 return false;<br>
><br>
>         if (STR_IN_SET(l->ifname, arg_ignore)) {<br>
>                 return false;<br>
><br>
> At the moment, my dirty workaround is to have br0 hardcoded  in<br>
> manager_all_configured from src/network/networkd-wait-online-manager.c:<br>
><br>
> if (!link_relevant(l) || STR_IN_SET(l->ifname, "br0")) {<br>
<br>
</span>I think such a patch that introduces --ignore-interface= or so would<br>
make sense. If you prep a good patch we can merge this.<br>
<br>
I'd probably even go one step further with this, and make this a glob,<br>
so that once can easily ignore whole sets of interfaces...<br>
<span class="HOEnZb"><font color="#888888"><br>
Lennart<br>
<br>
--<br>
Lennart Poettering, Red Hat<br>
</font></span></blockquote></div><br></div>