=== cpaelzer__ is now known as cpaelzer
[08:05] <Erikjan> So anyone here ever had issues where you have nfs entry's that are not yet ready on startup, but say 20 seconds later, where cloud-init would just hang? Have this on rhel images only though, ubuntu works fine. Any idea on how to go about this?
[08:14] <meena> which versions of those two OSes, Erikjan ?
[08:17] <meena> well, my first thought is that something in the init scripts is blocking, but on second thought, i have no idea
[08:19] <Erikjan> latest rhel 7.7 up to 8.1 (have not tested 8.2) and ubuntu the latest LTS
[08:19] <Erikjan> on a cold boot in the cloud it takes some time before hte netapp is attached, it needs to wait, which it doesn't do correctly
[13:33] <Odd_Bloke> smoser: Given that you were reviewing the previous iteration of it (thank you again!), do you also want to review https://github.com/canonical/cloud-init/pull/441?
[16:05] <Odd_Bloke> If I could get a committer's +1 on https://github.com/canonical/cloud-init/pull/440 (paride has reviewed), that'd be much appreciated.
[16:06] <Odd_Bloke> blackboxsw: ^ if you have a minute?
[16:06] <blackboxsw> Odd_Bloke: reading
[16:06] <Odd_Bloke> Thanks!
[16:06] <Odd_Bloke> falcojr: Scott did approve that PR, so you're off the hook. ;)
[16:06] <falcojr> heh, gtk
[16:08] <blackboxsw> approved: Odd_Bloke awaiting CI
[16:10] <Odd_Bloke> Thanks!
[16:16] <Odd_Bloke> smoser: https://github.com/canonical/cloud-init/pull/444 <-- that's the fix for the `instance`-is-a-directory issue (which presumably you saw coming given the earlier PR ;)
[16:17] <falcojr> what are cloud-initramfs-copymods and cloud-initramfs-dyn-netconf ?
[16:17] <falcojr> just noticed them when I did a dpkg -l | grep cloud-init
[16:17] <falcojr> are they related or just have an unfortunate similar name?
[16:19] <falcojr> some kind of cloud specific initramfs thing or something?
[16:39] <meena> falcojr: what's the "Source"? of those packages? apt show should tell you
[16:44] <powersj> https://packages.ubuntu.com/source/focal/cloud-initramfs-tools
[16:58] <Odd_Bloke> OK, take two on that earlier PR: https://github.com/canonical/cloud-init/pull/445
[17:15] <blackboxsw> Odd_Bloke: if you get a chance today, I think https://github.com/cloud-init/ubuntu-sru/pull/113 is ready
[18:06] <Odd_Bloke> blackboxsw: I'll consider it if you send me the bug creation automation you promised earlier. ;)
[18:06] <blackboxsw> hah a trade. I love to barter
[18:06] <blackboxsw> good pt. getting that now
[18:10] <blackboxsw> Odd_Bloke: here's what I did for the schema bugs so I think
[18:10] <blackboxsw> s/so I think/ https://paste.ubuntu.com/p/Dc3dbkDTSs/
[18:10] <blackboxsw> lp-create-bug is in uss-tableflip
[18:11] <blackboxsw> I think something comparable would work for the net refactor
[18:15] <Odd_Bloke> blackboxsw: Nice, thanks!
[19:01] <Odd_Bloke> blackboxsw: Still one outstanding issue on that ubuntu-sru PR. :)
[19:24] <blackboxsw> Odd_Bloke: thanks and pushed https://github.com/cloud-init/ubuntu-sru/pull/113
[20:06] <Odd_Bloke> blackboxsw: I think we now have the opposite problem: the remaining _templates_ will no longer be rendered per-SRU.
[20:09] <blackboxsw> Odd_Bloke: I'm hoping with the next PR to place all the remaining scripts under sru-scripts/manual.
[20:09] <blackboxsw> at the moment only azure-sru.*.txt would be created
[20:10] <blackboxsw> I have a followup to move all remaining scripts into sru-scripts and out of sru-templates
[20:10] <blackboxsw> which will then remedy that gap
[20:10] <Odd_Bloke> blackboxsw: OK, cool.
[20:10] <blackboxsw> I'll post that PR next (and can also post the azure and openstack SRU verification results as well)
[20:11] <Odd_Bloke> blackboxsw: +1, will leave you to merge. :)
[20:11] <blackboxsw> thanks Odd_Bloke
[20:11] <blackboxsw> thanks will merge, and the azure and openstack prs will move those two sru-templates/manual/* scripts under sru-scripts.
[20:18] <lucasmoura> hi everyone, is there a way to manually test this outside a oracle instance ? https://git.launchpad.net/cloud-init/commit/?id=2bedc440
[20:19] <lucasmoura> Reading through the code it seems that only the oracle datasource use this to set network config, but I may be missing something
[20:46] <blackboxsw> lucasmoura: https://github.com/cloud-init/ubuntu-sru/pull/126/files for azure sru verification results review
[20:46] <blackboxsw> I'll check that commit
[20:46] <lucasmoura> blackboxsw, okay
[20:53] <blackboxsw> lucasmoura: I think that commit is already tested as well as you can w/ the unit test that seeds the PROTO='none' @ https://github.com/canonical/cloud-init/blob/master/tests/unittests/test_net.py#L84-L108
[20:53] <blackboxsw> which confirms that a 'none' value gets converted to 'static'
[20:53] <blackboxsw> so I think  we don't need integration test coverage of that
[20:57] <lucasmoura> Okay, I will drop it from the list
[21:02] <lucasmoura> Also, regarding this commit as well https://git.launchpad.net/cloud-init/commit/?id=d5dbbd1c, should we cover it too ? And if yes, does anyone have a good idea for an url we can use to test this more easily ?
[21:22] <blackboxsw> openstack sru verification results and refactor up for review https://github.com/cloud-init/ubuntu-sru/pull/127