=== shardy is now known as shardy_afk === shardy_afk is now known as shardy === shardy is now known as shardy_lunch === Hazelesque_ is now known as Hazelesque === shardy_lunch is now known as shardy === rangerpbzzzz is now known as rangerpb [16:43] harlowja, around ? [16:43] looking at cloudinit/distros/rhel.py i realize i think we have multiple paths doing the same thing now. [16:55] actually, i think _write_network is probably never called now. [16:56] except for very explicit dsmode=pass on openstack config drive. [17:26] larsks, ping [17:26] https://code.launchpad.net/~bregeer-ctl/cloud-init/+git/cloud-init/+merge/305058 [17:26] i'd appreciate your eyes on that. [17:27] and rangerpb also [17:33] smoser: taking a look... [17:34] smoser: This better? :P [17:35] x58, sure. and x58 i just asked larsks to review your mp also [17:35] Cheers :-) [17:40] smoser what up [17:40] yes, we have multiple paths for networking :-/ [17:40] i think the one there is dead now. [17:40] sweet [17:41] dead code removal ftw [17:41] as in never used. [17:41] as in deaded? [17:41] not alive? [17:41] dead as a doorknob [17:41] except for oepnstack case where user feeds 'dsmode=pass' [17:41] which actually doesnt work that well anyway [17:41] ya, i bet, never tried that way [18:38] Sorry gents, thanks for the review, but I can't in good conscience allow Canonical to take my code and potentially relicense it under a different license. [19:26] rharper, https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/305137 [19:26] if you want to read that quick... seems simple. probably necessary for curtin too [19:27] actually not necessary for curtin [19:27] * smoser updates bug === rangerpb is now known as rangerpbzzzz