=== sambetts|afk is now known as sambetts === shardy is now known as shardy_lunch === shardy_lunch is now known as shardy === rangerpbzzzz is now known as rangerpb [13:37] smoser: I reverted all the modifications you did yesterday, and created manually the file /run/net-eth0.conf as it seems to be expected according to your unittests and various searches on google (resulting to https://pastebin.com/uwu7e1aJ) [13:38] however, running cloud-init init raises because: [13:38] File "/root/cloud-init-scaleway/cloudinit/net/network_state.py", line 296, in handle_physical [13:38] if ':' in subnet['address']: [13:38] KeyError: 'address' [13:38] shoudln't we add 'ADDRESS' here? https://github.com/brmzkw/cloud-init-scaleway/blob/wip/cloudinit/net/cmdline.py#L104 [13:41] niluje, it would appear so. [13:41] and the unit test is just busted. [13:41] you can fix the STATIC_EXPECTED_1 [13:41] as it does not have an 'address' which it clearly should [13:41] (this path we've not tested much as everyhthing maas used was dhcp) [13:41] you are doing dhcp though, right? [13:42] yes [13:42] yeah, so that is another path (although we should fix this) [13:42] but I configured the interface as static for testing purpose [13:42] and actually, yeah, its busted in dhcp also [13:43] well, in dhcp, there isn't really a sense in adding the static address [13:43] yes [13:43] but since nothing is done if the value doesn't exist in the net-xx.conf file, I still can add "ADDRESS" in the list and fix the unittests accordingly. Correct? [13:50] 'ADDR'* not 'ADDRESS' [14:12] yeah, please fix the unit tests. [14:13] niluje, and add ADDRESS as you suggested. you diagnosed this correctly. thanks. === jgrimm is now known as jgrimm-away [15:14] next sru in progress https://public.etherpad-mozilla.org/p/cloud-init-sru-info-20170427 [15:43] ok, I'm heading down the list to associate cloud-init sru bugs w/ the proper series [15:44] smoser, why is this bug in the list again https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1676460 just because we need it in zesty now too? [15:44] Ubuntu bug 1676460 in cloud-init (Ubuntu Yakkety) "dpkg-reconfigure does not list Bigstep" [Medium,Fix released] [15:45] blackboxsw, smoser: #ubuntu-devel says its fine to leave them as just nominated [15:45] sounds good thx dpb1 [15:45] when the sru is requested, they will do the accepting. [15:46] blackboxsw, hm.. [15:46] that shoudlnt be there. [15:46] removing [15:47] ok can you kill my nominations too smoser [15:47] (that was tehre just because i left it there to make the new sections look like it) [15:47] as an example [15:47] then forgot to remove it [15:49] ahh makes sense [17:03] smoser, dpb1 how do I switch my lp view to allow nominating for series on the distro project instead of the main project? https://bugs.launchpad.net/cloud-init/+bug/1684869 [17:03] Ubuntu bug 1684869 in cloud-init "growing root partition does not always work with root=PARTUUID=" [Medium,Confirmed] [17:04] right now on th above bug I have a highlighted cloud-init project so I can't select nominate for series on the distro project [17:04] the selections I get when nominating for series are trunk/2.0 instead of xenial/yak/zesty..etc [17:04] blackboxsw: https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1684869 [17:04] Ubuntu bug 1684869 in cloud-init "growing root partition does not always work with root=PARTUUID=" [Medium,Confirmed] [17:05] blackboxsw: is that what you mean? [17:05] blackboxsw, edit the url :) [17:05] not kidding. [17:05] yeah nacc that is highlighting the appropriate project for me [17:05] ahhh [17:05] blackboxsw: yeah, so i hovered over the cloud-init (ubuntu) task [17:05] thats why my template has 2 urls [17:05] and that showed the other task's direct url base [17:06] heh I hadn't noticed the url path difference [17:06] ok [17:06] thanks [17:06] blackboxsw: yeah, only ubuntu/ urls can have ubuntu release tasks [17:11] smoser, do I also add needs-verification tag now or do we wait until the SRU template is written [17:12] blackboxsw: the sru team adds those tags once they upload [17:13] *approve [17:13] ahh thanks nacc === sambetts is now known as sambetts|afk [18:47] smoser, how's this sru template for #1682160 http://paste.ubuntu.com/24474635/? [18:48] it took me a while to just validate the breakage/script [18:49] * blackboxsw has to head out for a late lunch back within an hour [18:52] blackboxsw, install would p[robably trigger it to [18:52] but sure. that looks sane. [18:53] if you just launch an image on an openstack (or otherwise utilze the cloud image) then grub-legacy-ec2 will be there already. [19:42] back === rangerpb is now known as rangerpbzzzz