=== tds1 is now known as tds
=== tds7 is now known as tds
[13:24] <Odd_Bloke> blackboxsw: I've approved and landed the xenial daily branch fix; it did require configuring the xenial recipe build to use the daily branch (which I'm about to do for bionic and eoan too; this is just FYI), but it's building successfully now. \o/
[13:51] <Odd_Bloke> blackboxsw: And now the same for the bionic daily branch fix. \o/
[14:25] <ananke> since we've been using cloud-init to manage our automated AMI build pipelines, I am looking for something similar to it in the windows world. it appears there's cloudbase-init project that attempts to do something similar, do any of you have other recommendations?
[14:48] <Odd_Bloke> blackboxsw: I have a high-level approach Q on https://github.com/cloud-init/ubuntu-sru/pull/113#pullrequestreview-431851557, so I've posted that and will defer further review until we've discussed it. :)
[14:50] <Odd_Bloke> ananke: I don't (but I wouldn't really expect to, I don't use Windows for anything other than gaming ;).
[15:03] <ananke> Odd_Bloke: it's certainly a very new territory for me too, after managing unix/linux systems for 25 years :)
[15:29] <blackboxsw> ananke: cloudbase-init is the only alternative I was aware of, though it's feature support level seems to pale in comparison :/
[16:15] <blackboxsw> Odd_Bloke: responded and push commits for discussion https://github.com/cloud-init/ubuntu-sru/pull/113#discussion_r441665171
[16:15] <blackboxsw> *pushed commits*
[16:15] <Odd_Bloke> blackboxsw: Ack, thanks!
[17:14] <blackboxsw> Odd_Bloke: I missed it at standup but, I see you've already updated https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-bionic and https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-xenial to use daily
[17:14] <blackboxsw> thanks
[17:35] <Odd_Bloke> :)
[17:42] <robjo> blackboxsw: Have a few some time? https://bugs.launchpad.net/cloud-init/+bug/1883907
[17:42] <ubot5> Ubuntu bug 1883907 in cloud-init "EC2 data source does not properly return the instance ID when cached data exists" [Undecided,New]
[18:39] <blackboxsw> robjo: will glance over that today
[19:04] <blackboxsw> Odd_Bloke: if you get a chance today I'd like to merge this iteration of consolidated sru-vars if we can so I can push two cloud verifications up (azure and openstack) which rely on this branch https://github.com/cloud-init/ubuntu-sru/pull/113
[19:11] <Odd_Bloke> blackboxsw: Looking now.  Do those rely on the supplemental test support?
[19:14] <robjo> blackboxsw: OK, thanks, I am a bit confused as to what happens when and why. So based on debug we load the pickled object with the wrong instance data, but then we recover at some point (I have not found the place in the code where that happens) and the data source gets the data fro the new instance. And the user data is executed as expected, in my test
[19:15] <robjo> Is there a possibility that a user data script could access the data source data directly and end up with the wrong data, i.e. the data loaded from the cache?
[19:16] <robjo> I do not have access to the user data script that supposedly hangs in this context
[19:16] <Odd_Bloke> blackboxsw: I have responded.
[19:51] <blackboxsw> Odd_Bloke: thanks I've force pushed https://github.com/cloud-init/ubuntu-sru/pull/113
[19:51] <blackboxsw> dropping the changes and moving azure-sru and sru-vars.template into sru-scripts/
[20:09] <Odd_Bloke> blackboxsw: Thanks!  Reviewed again.
[20:13] <falcojr> in this sru verification, I just discovered a legitimate issue
[20:14] <falcojr> it's not technically a regression, because the old behavior failed too...but it hasn't been fixed either
[20:15] <falcojr> do we try to fix it right away and then get it into the SRU? Or do we pull this particular feature out? Or do we file a bug and backport it anyway since it was already broken?
[20:17] <falcojr> creating a swapfile still doesn't work even with dd on btrfs
[20:17] <falcojr> following these instructions I was able to make it work
[20:17] <falcojr> https://wiki.archlinux.org/index.php/Swap#Swap_file
[20:19] <Odd_Bloke> falcojr: As it isn't a regression, we don't _need_ to restart the process with a fix.  And, given how far through the process we are at this point, I think it makes sense to file a bug for it and move on.
[20:20] <Odd_Bloke> Oh, no blackboxsw, I'd also like his opinion.
=== smoser1 is now known as smoser
[21:22] <blackboxsw> thanks Odd_Bloke renamed necessary files https://github.com/cloud-init/ubuntu-sru/pull/113 and resolved comments