=== shardy is now known as shardy_lunch
=== shardy_lunch is now known as shardy
[15:53] <smoser> blackboxsw: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/345630
[15:53] <smoser> i think thiat is ready
[15:54] <smoser> and https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/346747 is simple enough.
[15:54] <blackboxsw> smoser: will grab both.
[16:21] <smoser> robjo: https://bugs.launchpad.net/cloud-init/+bug/1772961
[16:21] <ubot5> Ubuntu bug 1772961 in cloud-init "suse (packages/suse/cloud-init.spec.in) spec does not build" [Medium,Confirmed]
[16:21] <smoser> is that somethign you could look at? if necessary i could/can provide you with a system that you can './run-container' on.
[16:42] <robjo> smoser: I'll take a look, probably Friday, the next couple of days will be dedicated to dealing with Specter v4 fall out
[19:21] <smoser> blackboxsw: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/345630 that could use an eyeball. seems mostly non-contentious
[19:23] <blackboxsw> ... will wrap it up and land it.
[19:39] <smoser> do we have a bug on https://jenkins.ubuntu.com/server/view/cloud-init,%20curtin,%20streams/job/cloud-init-integration-lxd-a/357/artifact/cloud-init/results/
[19:40] <smoser> or https://jenkins.ubuntu.com/server/view/cloud-init,%20curtin,%20streams/job/cloud-init-integration-lxd-b/
[19:40] <smoser> hm.. why did that just now go red
[19:41] <powersj> smoser: I changed the integration tests yesterday
[19:41] <powersj> they now use the ppa...
[19:41] <powersj> but, that means they use upstream tests
[19:41] <smoser> hm.. but ppa is up to date.
[19:41] <powersj> yeah so I didn't think we would have a gap
[19:41] <powersj> but that is what changed
[19:42] <blackboxsw> lxd-b salt_minion error we have a bug
[19:42] <blackboxsw> lemme see here
[19:42] <smoser> well, in -a it was chrony
[19:42] <smoser> install of chrony failed
[19:42] <smoser> how do you know what failed ?
[19:42] <smoser> we really need a nic elist of which tests failed.
[19:42] <blackboxsw> weird... /me thinks we have a salt_minion bug...
[19:45] <powersj> salt_minion did fail
[19:45] <blackboxsw> https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1769754
[19:45] <ubot5> Ubuntu bug 1769754 in cloud-init (Ubuntu) "salt-minion: public/private keys not preserved in /etc/salt/pki" [Undecided,New]
[19:45] <blackboxsw> I filed against the package, should have been against cloud-init project
[19:46] <blackboxsw> I can grab that so we see pretty green if folks want
[19:46] <blackboxsw> I can also try to consolidate the integration test failures to a human readable output at the end
[19:46] <smoser> hm...
[19:47] <smoser> blackboxsw: first make c-i like os-local
[19:47] <smoser> and we can land that
[19:53] <smoser> rharper: if you could look at https://code.launchpad.net/~mgerdts/cloud-init/+git/cloud-init/+merge/344168
[19:53] <smoser> i think that is close
[19:53] <rharper> sure
[20:03] <smoser> blackboxsw: ... i think maybe something push overwrote ?
[20:03] <smoser> https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/345630 says merged.
[20:03] <blackboxsw> smoser: I think you and I collided twice
[20:03] <smoser> i had just pushed https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/346415
[20:03] <smoser> yeah...
[20:03] <smoser> but the tools hould fail
[20:03] <blackboxsw> I tryied running review-mps on your branch
[20:03]  * blackboxsw is hands off.
[20:03] <blackboxsw> I'll let you land them
[20:03] <smoser> hm..
[20:04] <smoser> i'd have thought it would fail though.  the push should fail if its not ff-only
[20:05] <blackboxsw> on my side this is what I saw
[20:05] <blackboxsw> https://pastebin.ubuntu.com/p/GHWBX4bw4J/
[20:05] <blackboxsw> so the merge failed, but before the push, the script made changes to the branch in Launchpad
[20:05] <smoser> right.
[20:05] <smoser> ok. that makes seense.
[20:05] <blackboxsw> so we should do those changes only after a successful push
[20:06] <smoser> so in that case it should not change the bugs.
[20:06] <smoser> yeah.
[20:06] <blackboxsw> +1  I'll change that now
[20:06] <blackboxsw> and I'll stop running review-mps
[20:06] <smoser> also i was just thinking ... since i am re-using a '--local-repo-dir'
[20:06] <smoser> we should rm -Rf .tox
[20:06] <blackboxsw> +1 will add it
[20:07] <smoser> so that it gets re-built with any un-intentionally not pinned versions
[20:07] <smoser> but first make c-i like your os-local branch
[20:28] <blackboxsw> ok pushed fix for os-local. awaiting https://jenkins.ubuntu.com/server/job/cloud-init-ci/35/
[20:29] <blackboxsw> moving around review-mps operations to avoid touching bugs/MPs until publish has occurred
[20:29] <blackboxsw> s/publish/git push/
[21:12] <blackboxsw> oops had to rebase for the httpretty changes. running again
[21:42] <blackboxsw> I'm going to put up an upload for C as os-local is landing
[22:20] <blackboxsw> ok cosmic upload proposed
[22:20] <blackboxsw> https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/346780