=== cpaelzer_ is now known as cpaelzer
=== gschanuel7 is now known as gschanuel
[13:52] <holmanb> ryland: good find, yes I think you're right
[14:11] <holmanb> Guest18: my first instinct is to ask whether user/group creation succeeded for the user/group you're trying to write for
[14:13] <holmanb> Guest18: got a cloud-config to share? I can try reproducing locally - this sounds like a module-speicific thing, so I doubt it matters that you're on azure
[14:13] <holmanb> s/speicific/specific/
[16:04] <minimal> Though this might be of interest: https://github.com/irsl/gcp-dhcp-takeover-code-exec
[16:04] <minimal> s/Though/Thought/
[16:07] <minimal> both the references to dhclient issues and also recommendations regarding metadata servers (use IP not DNS name, use TLS)
=== gschanuel9 is now known as gschanuel
[17:29] <holmanb> minimal: thanks for sharing
[17:30] <holmanb> That's really valuable info. There are several active conversations around the future of dhclient in general since it's no longer maintained :(
[18:20] <minimal> holmanb: ever thought of dhcpc?
[18:48] <holmanb> minimal: nope, I'll take a look
[18:50] <minimal> holmanb: at one point in the past I was thinking of getting cloud-init to use either of (Busybox) udhcpc, dhclient, or dhcpc but then realised that there's so many places in cloud-init that assume dhclient
[18:51] <minimal> https://roy.marples.name/projects/dhcpcd/
[18:54] <holmanb> The problem with a replacement  (in my mind at least) is that we'd need something that works for the bsds and the linux distros and is packaged for both.
[18:57] <minimal> I guess either (a) as you say use/support only a single client across all OSes, or (b) support more than 1 client
[18:57] <minimal> a bit like how NTP is handled - supports chrony, ntpd, etc
[18:58] <minimal> I had also intended to work on getting dropbear & tinysshd supported in cloud-init but again the support/assumptions of openssh appear heavily embedded (i.e. config settings naming)
[19:01] <holmanb> minimal: true, a multiple client approach could work as well
[19:03] <minimal> I hadn't thought about dhclient's lack of updates - just checked and Kea is server only (I guess I'd assumed it also provided a client too)
[19:03] <holmanb> we probably need to have some mailing list discussions at some point to make sure this gets enough external eyes
[19:03] <minimal> so is dhclient officially unmaintained? or just that there appears to be no active work on it?
[19:04] <holmanb> from the website: "The client and relay portions of ISC DHCP are no longer maintained."
[19:04] <holmanb> https://www.isc.org/dhcp/
[19:04] <holmanb> there was a recent ubuntu mailing list discussion as well about what to do regarding support
[19:06] <minimal> ah, didn't spot that on the webpage
[19:06] <minimal> Busybox's udhcpc is pretty basic so not sure if its suitable in general.
[19:15] <holmanb> good to know
[21:21] <blackboxsw> https://github.com/canonical/cloud-init/pull/1472 merge which drops Ubuntu Xenial and python < 3.6 support. thanks aciba for the PR.
[21:21] <ubottu> Pull 1472 in canonical/cloud-init "Remove xenial references" [Merged]
[21:27] <blackboxsw> d'oh commit msg typo: drop unused python <= 2.7 logic.  not 3.7 or 3.6.
[21:38] <blackboxsw> holmanb: if you get a chance can you confirm your latest commitish on holmanb/reuse-library-code ? github is showing a visual diff that doesnt't match what I see at all 
[21:38] <holmanb> sure
[21:39] <blackboxsw> thx. for instance I don't see the `convert_to_address` param in my local copy of your branch
[21:40] <blackboxsw> 2db36f128bddf0cd023eaad2e2fa0294b18a5b78 on my side
[21:41] <holmanb> 3 commits come after 2db3
[21:41] <holmanb> 92c40088d93f688e78e04d9339efaddadc2862ec, 5ad12f95a7bdc1c7f8d4d5cfa1d814969580285f, a63d1fa73d35f8b97e464f76c4835aedf68403f5
[21:42] <blackboxsw> something funky is caching my local copy of your remote. ok will sort it thx.
[21:42] <holmanb> No problem
[21:42] <blackboxsw> ahh holmanb/holmanb/reuse-library-code vs holmanb/reuse-library-code
[21:43] <holmanb> that'll do it :)
[21:43] <blackboxsw> ok I have git remote-itis :) that is colliding w/ your branch naming strategy. PEBCAK
[21:43] <falcojr> heh, the double holmanb has duped me before too
[21:43] <holmanb> gotta get that holmanb/holmanb
[21:44] <holmanb> sorry for excessive naming, might need to rethink it a little
[21:44] <blackboxsw> all good. it's my own config problem. everyone's entitled to their naming conventions
[21:45] <blackboxsw> I think somehow I had a branch resolution up at holmanb/ and holmanb/holmanb/* so git defaulted to the former
[21:45] <blackboxsw> which turns out was stale
[21:53] <holmanb> +1
[21:54] <falcojr> this is a fun failure https://github.com/canonical/cloud-init/runs/6802535842?check_suite_focus=true
[21:55] <falcojr> it's complaining about pylint compatibility with simplestreams...but simplestreams doesn't depend on pylint?
[21:56] <falcojr> or I'm just reading the error wrong :P
[22:00] <holmanb> looks to me like a launchpad issue higher up led to that
[22:00] <holmanb> is that what you meant by reading the error wrong?
[22:03] <falcojr> yes, that's exactly what I mean :D