=== cpaelzer__ is now known as cpaelzer | ||
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:05 |
---|---|---|
meena | which versions of those two OSes, Erikjan ? | 08:14 |
meena | well, my first thought is that something in the init scripts is blocking, but on second thought, i have no idea | 08:17 |
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 | 08:19 |
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? | 13:33 |
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:05 |
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:06 |
blackboxsw | approved: Odd_Bloke awaiting CI | 16:08 |
Odd_Bloke | Thanks! | 16:10 |
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:16 |
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:17 |
falcojr | some kind of cloud specific initramfs thing or something? | 16:19 |
meena | falcojr: what's the "Source"? of those packages? apt show should tell you | 16:39 |
powersj | https://packages.ubuntu.com/source/focal/cloud-initramfs-tools | 16:44 |
Odd_Bloke | OK, take two on that earlier PR: https://github.com/canonical/cloud-init/pull/445 | 16:58 |
blackboxsw | Odd_Bloke: if you get a chance today, I think https://github.com/cloud-init/ubuntu-sru/pull/113 is ready | 17:15 |
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:06 |
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:10 |
blackboxsw | I think something comparable would work for the net refactor | 18:11 |
Odd_Bloke | blackboxsw: Nice, thanks! | 18:15 |
Odd_Bloke | blackboxsw: Still one outstanding issue on that ubuntu-sru PR. :) | 19:01 |
blackboxsw | Odd_Bloke: thanks and pushed https://github.com/cloud-init/ubuntu-sru/pull/113 | 19:24 |
Odd_Bloke | blackboxsw: I think we now have the opposite problem: the remaining _templates_ will no longer be rendered per-SRU. | 20:06 |
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:09 |
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:10 |
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:11 |
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:18 |
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:19 |
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:46 |
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:53 |
lucasmoura | Okay, I will drop it from the list | 20:57 |
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:02 |
blackboxsw | openstack sru verification results and refactor up for review https://github.com/cloud-init/ubuntu-sru/pull/127 | 21:22 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!