[03:28] <larsks> nrezinorn, but those files are already installed by setup.py.
[03:28] <larsks> In this case, the spec file should just trust setup.py.
[03:28] <larsks> harlowja, smoser: https://code.launchpad.net/~larsks/cloud-init/+git/cloud-init/+merge/312519 is my proposal for moving distribution-dependent service names out of the systemd unit files, as I was discussing with harlowja earlier today.
[07:55] <smoser> hm.
[07:55] <smoser> larsks, did you specifically have a problem ?
[07:56] <smoser> while there is noise in those files for sure (such as Before=network.service). it doesn't cause any issues that i'm aware of.
[12:38] <larsks> smoser, if you prefer to just list all the dependencies in the existing file (so Before=network.service networking.service NetworkManager.service), I'll submit that patch instead.
[12:38] <smoser> what does that diff look like ?
[12:38] <larsks> I like keeping the config files clean, but either solution works.
[12:38] <smoser> i suspect its pretty small isn't it ?
[12:38] <larsks> Yeah.
[16:46] <larsks> smoser, speaking of service dependencies....cloud-init-local.service is Before=NetworkManager.service.  Should it also be Before=networking.service (on ubuntu)?
[17:06] <smoser> larsks, i am pretty sure ubuntu's are right.
[17:06] <larsks> Okay.  But why a dependency on NetworkManager but not on networking.service?  I am just trying to understand the dependencies.
[17:17] <larsks> smoser, because I would like to make the deps under RHEL match as much as possible, but I don't understand the current model w/r/t networkmanager vs. networking.
[17:17]  * larsks heads out for a bit; back later.
[17:17] <smoser> right.
[17:18] <smoser> larsks, before=network-pre
[17:18] <smoser> is sufficent actually
[17:18] <smoser> and *should* be sufficient for NetworkManager
[17:18] <smoser> but netowrk manager has  bug
[17:18] <smoser> i think there is an upstream bug for that
[17:21] <smoser> yea, so NetworkManager should have
[17:21] <smoser>  After=network-pre.target
[17:21] <smoser> its buggy that it does not
[17:23] <smoser> larsks, https://bugzilla.gnome.org/show_bug.cgi?id=761001
[18:52] <larsks> smoser, awesome, thanks!