[03:28] nrezinorn, but those files are already installed by setup.py. [03:28] In this case, the spec file should just trust setup.py. [03:28] 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. === mfisch is now known as Guest16197 [07:55] hm. [07:55] larsks, did you specifically have a problem ? [07:56] 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. === shardy is now known as shardy_mtg [12:38] 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] what does that diff look like ? [12:38] I like keeping the config files clean, but either solution works. [12:38] i suspect its pretty small isn't it ? [12:38] Yeah. === Guest16197 is now known as mfisch === mfisch is now known as Guest73037 === Guest73037 is now known as mfisch === mfisch is now known as Guest13281 === Guest13281 is now known as mfisch === mfisch is now known as Guest40664 === Guest40664 is now known as mfisch [16:46] smoser, speaking of service dependencies....cloud-init-local.service is Before=NetworkManager.service. Should it also be Before=networking.service (on ubuntu)? [17:06] larsks, i am pretty sure ubuntu's are right. [17:06] Okay. But why a dependency on NetworkManager but not on networking.service? I am just trying to understand the dependencies. [17:17] 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] right. [17:18] larsks, before=network-pre [17:18] is sufficent actually [17:18] and *should* be sufficient for NetworkManager [17:18] but netowrk manager has bug [17:18] i think there is an upstream bug for that [17:21] yea, so NetworkManager should have [17:21] After=network-pre.target [17:21] its buggy that it does not [17:23] larsks, https://bugzilla.gnome.org/show_bug.cgi?id=761001 [18:52] smoser, awesome, thanks!