tkluck / pac4cli

Proxy-auto-discovery for command-line applications
25 stars 9 forks source link

systemd config: wait for NetworkManager #46

Closed tkluck closed 6 years ago

tkluck commented 6 years ago

Before this commit, we would start more-or-less simultaneously with NetworkManager, as we were also part of the network target. This leads to race conditions, and we'll terminate with an error if we cannot talk to NetworkManager's dbus interface.

This was an issue for me at the reboot immediately following the upgrade to Ubuntu 18.10, but I'm not sure it's related to that.

In any case, it's probably better not to start unless NetworkManager is up.

@kdehairy I haven't been able to test this in the archlinux container yet, but I'll try that.

tkluck commented 6 years ago

Hmmmm.... after inspecting my logs more closely, I'm not so sure this config here is related to the issue I had at all. Going to close this pull request so as not to take your time.