=== cpaelzer__ is now known as cpaelzer
ErikjanSo 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:05
meenawhich versions of those two OSes, Erikjan ?08:14
meenawell, my first thought is that something in the init scripts is blocking, but on second thought, i have no idea08:17
Erikjanlatest rhel 7.7 up to 8.1 (have not tested 8.2) and ubuntu the latest LTS08:19
Erikjanon a cold boot in the cloud it takes some time before hte netapp is attached, it needs to wait, which it doesn't do correctly08:19
Odd_Blokesmoser: 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?13:33
Odd_BlokeIf 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:05
Odd_Blokeblackboxsw: ^ if you have a minute?16:06
blackboxswOdd_Bloke: reading16:06
Odd_Blokefalcojr: Scott did approve that PR, so you're off the hook. ;)16:06
falcojrheh, gtk16:06
blackboxswapproved: Odd_Bloke awaiting CI16:08
Odd_Blokesmoser: 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:16
falcojrwhat are cloud-initramfs-copymods and cloud-initramfs-dyn-netconf ?16:17
falcojrjust noticed them when I did a dpkg -l | grep cloud-init16:17
falcojrare they related or just have an unfortunate similar name?16:17
falcojrsome kind of cloud specific initramfs thing or something?16:19
meenafalcojr: what's the "Source"? of those packages? apt show should tell you16:39
Odd_BlokeOK, take two on that earlier PR: https://github.com/canonical/cloud-init/pull/44516:58
blackboxswOdd_Bloke: if you get a chance today, I think https://github.com/cloud-init/ubuntu-sru/pull/113 is ready17:15
Odd_Blokeblackboxsw: I'll consider it if you send me the bug creation automation you promised earlier. ;)18:06
blackboxswhah a trade. I love to barter18:06
blackboxswgood pt. getting that now18:06
blackboxswOdd_Bloke: here's what I did for the schema bugs so I think18:10
blackboxsws/so I think/ https://paste.ubuntu.com/p/Dc3dbkDTSs/18:10
blackboxswlp-create-bug is in uss-tableflip18:10
blackboxswI think something comparable would work for the net refactor18:11
Odd_Blokeblackboxsw: Nice, thanks!18:15
Odd_Blokeblackboxsw: Still one outstanding issue on that ubuntu-sru PR. :)19:01
blackboxswOdd_Bloke: thanks and pushed https://github.com/cloud-init/ubuntu-sru/pull/11319:24
Odd_Blokeblackboxsw: I think we now have the opposite problem: the remaining _templates_ will no longer be rendered per-SRU.20:06
blackboxswOdd_Bloke: I'm hoping with the next PR to place all the remaining scripts under sru-scripts/manual.20:09
blackboxswat the moment only azure-sru.*.txt would be created20:09
blackboxswI have a followup to move all remaining scripts into sru-scripts and out of sru-templates20:10
blackboxswwhich will then remedy that gap20:10
Odd_Blokeblackboxsw: OK, cool.20:10
blackboxswI'll post that PR next (and can also post the azure and openstack SRU verification results as well)20:10
Odd_Blokeblackboxsw: +1, will leave you to merge. :)20:11
blackboxswthanks Odd_Bloke20:11
blackboxswthanks will merge, and the azure and openstack prs will move those two sru-templates/manual/* scripts under sru-scripts.20:11
lucasmourahi everyone, is there a way to manually test this outside a oracle instance ? https://git.launchpad.net/cloud-init/commit/?id=2bedc44020:18
lucasmouraReading through the code it seems that only the oracle datasource use this to set network config, but I may be missing something20:19
blackboxswlucasmoura: https://github.com/cloud-init/ubuntu-sru/pull/126/files for azure sru verification results review20:46
blackboxswI'll check that commit20:46
lucasmourablackboxsw, okay20:46
blackboxswlucasmoura: 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-L10820:53
blackboxswwhich confirms that a 'none' value gets converted to 'static'20:53
blackboxswso I think  we don't need integration test coverage of that20:53
lucasmouraOkay, I will drop it from the list20:57
lucasmouraAlso, 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:02
blackboxswopenstack sru verification results and refactor up for review https://github.com/cloud-init/ubuntu-sru/pull/12721:22

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!