=== shardy is now known as shardy_lunch | ||
=== shardy_lunch is now known as shardy | ||
smoser | blackboxsw: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/345630 | 15:53 |
---|---|---|
smoser | i think thiat is ready | 15:53 |
smoser | and https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/346747 is simple enough. | 15:54 |
blackboxsw | smoser: will grab both. | 15:54 |
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:21 |
robjo | smoser: I'll take a look, probably Friday, the next couple of days will be dedicated to dealing with Specter v4 fall out | 16:42 |
smoser | blackboxsw: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/345630 that could use an eyeball. seems mostly non-contentious | 19:21 |
blackboxsw | ... will wrap it up and land it. | 19:23 |
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:39 |
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:40 |
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:41 |
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:42 |
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:45 |
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:46 |
smoser | blackboxsw: first make c-i like os-local | 19:47 |
smoser | and we can land that | 19:47 |
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 | 19:53 |
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:03 |
smoser | i'd have thought it would fail though. the push should fail if its not ff-only | 20:04 |
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:05 |
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:06 |
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:07 |
blackboxsw | ok pushed fix for os-local. awaiting https://jenkins.ubuntu.com/server/job/cloud-init-ci/35/ | 20:28 |
blackboxsw | moving around review-mps operations to avoid touching bugs/MPs until publish has occurred | 20:29 |
blackboxsw | s/publish/git push/ | 20:29 |
blackboxsw | oops had to rebase for the httpretty changes. running again | 21:12 |
blackboxsw | I'm going to put up an upload for C as os-local is landing | 21:42 |
blackboxsw | ok cosmic upload proposed | 22:20 |
blackboxsw | https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/346780 | 22:20 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!