[00:39] <rharper> ok, systemd_timesyncd and chrony done;
[00:39] <rharper> now I'm done, tomorrow fixing up py26 unittests
[00:39] <rharper> *sigh*
[01:38] <smoser> blackboxsw: i would probably not add the new depends into x, a
[01:39] <smoser> it probably *is* strictly required.
[01:39] <smoser> but because isc-dhcp-client is part of ubuntu-minimal
[01:40] <smoser> and you're really expected to create a ubuntu without 'ubuntu-minimal'
[01:40] <smoser> it is not likely that cloud-init is to be installed in such a place.
[01:43] <smoser> so.. i'd just leave it be.  additionally, adding depends or recommends like that to a stable can be problematic
[01:43] <smoser> see someone complaining about a similar add
[01:43] <smoser>  https://bugs.launchpad.net/ubuntu/+source/initramfs-tools/+bug/1633643
[01:44] <smoser> and then also the squashfuse bug (bug 1628289)
[01:53] <blackboxsw> +1 on dropping isc-dhcp-client depends
[01:53] <blackboxsw> will repost the MPs without that
[02:01] <blackboxsw> smoser: thanks for the reference to https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1756173
[02:03] <blackboxsw> I've added a card https://trello.com/c/OTYKbFEu/720-drop-squashfuse-install-logic-from-ccsnap-when-https-bugslaunchpadnet-ubuntu-source-snapd-bug-1756173-lands
[02:29] <blackboxsw> pushed ubuntu/xenial and ubuntu/artful without isc-dhcp-client explicit dependency
[13:53] <rharper> smoser: ./tools/run-centos is super handy =)
[13:55] <smoser> yeah. we need to make it work for suse too
[13:56] <smoser> and actually.. i've even wnated it to work for ubuntu
[13:56] <smoser> when there was some bug trhat didnt reproduce on my desktoip, but c-i saw.
[14:05] <rharper> interesting
[17:01] <smoser> blackboxsw: ok... can you give qa-scripts/new-upstream-snapshot a try ?
[17:01] <smoser> you can see the last commit message there.
[17:02] <smoser> i think i have addressed most of the pain in it.
[17:02] <blackboxsw> checking/updating now
[17:02] <blackboxsw> and thanks!
[17:03] <blackboxsw> smoser: we'll drop new-upstream-snapshot script after the SRU or as part of the SRU?
[17:03] <blackboxsw> in the ubuntu/xenial|artful branches
[17:03] <smoser> i think no hurry to drop it.
[17:03] <smoser> so i'd say after.
[17:03] <smoser> but if you want to drop it we can.
[17:06]  * blackboxsw checks out the refresh patches logic. that was a surprise
[17:07] <smoser> thats why your xenial didnt work
[17:07] <smoser> because the patch would only apply with fuzz
[17:07] <smoser> as we'd changed ds-identify aroudn the xenial patches
[17:09] <blackboxsw> gotcha, yeah I just didn't expect to see a separate commit message for it, and changelog entry. Both make sense as separate items, I just wanted to understand the quilt magic
[17:13] <blackboxsw> smoser: pushed just ubuntu/xenial branch again
[17:13] <blackboxsw> shall I push artful again?
[17:14] <blackboxsw> new-upstream-snapshot works like a charm
[17:15] <smoser> blackboxsw: i uploaded artful all ready.
[17:15] <blackboxsw> ahh hadn't checked, thx
[17:15] <smoser> i don tthink any reason to re-do it.
[17:16] <blackboxsw> changelog didn't end up changing anything, so good there on artful anyway
[17:26] <ihre> I'd like to test some changes in user-data after applying some changes to user-data. So far, I've mostly used cloud-init w/ qemu and genisoimage for testing, but I think there should be an easier way. I've tested w/ manual changes to cloud-config.txt/user-data.txt/user-data.txt.1 in /var/lib/cloud/instances/$instance-id/, and by reattaching the cdrom through the qemu monitor, yet changes aren't picked up.
[17:26] <ihre> Which file on disk should I modify to be able to rerun cloud-init without having to go through the qemu/genisoimage steps?
[17:26] <blackboxsw> ihre: there are several semaphores that block re-running specific cloud-init modules across reboots
[17:27] <blackboxsw> you can clear out those semaphores by running "sudo cloud-init clean --logs --reboot"
[17:27] <blackboxsw> it'll remove all cloud-init former run logs and reboot the machine so cloud-init can re-run "fresh" with any user data you've provided
[17:28] <blackboxsw> I generally iterate using lxc on ubuntu
[17:28] <blackboxsw> via lxc launch ubuntu-daily:bionic myb1
[17:29] <blackboxsw> lxc exec myb1 bash   # then manipulate user-data in /var/lib/cloud/seed/nocloud-net/user-data
[17:30] <blackboxsw> in lxcs, the  no-cloud/seed directory is kept across reboots.    I'll check how the qemu image detected  re-run to better answer your question
[17:31] <blackboxsw> hrm, I actually have to run on an errand, will get back to that in ~45 mins
[17:32] <ihre> blackboxsw: Thanks, didnt think of using a container to test it, that seems a nice alternative as well.
[17:32] <ihre> Previously, I just removed a/the semaphore(s) by hand and reran cloud-init modules --mode final.
[17:33] <blackboxsw> it's at least the fastest iterative method I've found for dev-test cycles.
[17:33]  * blackboxsw heads out 
[17:33] <ihre> No rush on my end, the current way works fine though, it just feels a little cumbersome at times, anyways, thanks for taking the time to look into it :)
[17:39] <smoser> blackboxsw: your quilt refrsh differs in behavior from mine.
[17:40] <nacc> blackboxsw: smoser: it's sensitive to ~/.quiltrc
[17:40] <nacc> use --quiltrc -
[17:40] <nacc> (just a guess)
[17:42] <smoser> yeah.
[17:44] <nacc> we have to do that in git-ubuntu
[17:44] <nacc> smoser: i'm guessing this is about reproducibility?
[17:49] <smoser> blackboxsw: grab tip of qa-scripts and try again, would you ?
[17:50] <smoser> nacc: well, yeah.
[17:50] <smoser>  
[17:50] <smoser> http://paste.ubuntu.com/p/3BwC5n3v4P/
[17:50] <nacc> smoser: yeah
[17:50] <smoser> we run 'new-upstream-snapshot' to create a new upstream snapshot of cloud-init.
[17:50] <smoser> and that is a run here versus on chad's system.
[17:50] <nacc> yeah, you want your snapshot to be the same as chad
[17:51] <nacc> similar idea for us, we want our same-argument builds to be contentfully the same
[18:33] <blackboxsw> smoser: just pushed https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+ref/ubuntu/xenial
[18:36] <smoser> blackboxsw: woot.
[18:36] <smoser> mine to yours : http://paste.ubuntu.com/p/spBd2CCrKg/
[18:36] <smoser> new feature there to strip bug numbers on sru
[18:36] <smoser> and make you give an SRU bug
[18:53] <blackboxsw> ahh right I totally forgot to strip bug#'s on artful
[18:56] <blackboxsw> smoser: what should I do w.r.t. devel/arful
[18:56] <blackboxsw> smoser: what should I do w.r.t. devel/artful. should we strip bug#'s
[18:58] <smoser> blackboxsw: oh. shoot. yeah i guess we should. right ?
[18:58] <smoser> we are supposed to.
[18:58] <smoser> we can just re-upload
[18:58] <blackboxsw> yeah, I think so, yeah I'll repropose and do that right.
[18:58] <blackboxsw> forgot that part of the process obviously
[18:59] <smoser> well now the script does the right thing
[18:59] <smoser> you forgot, i forgot too.
[19:11] <blackboxsw> https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/342329 just --forced a new-upstream-release for artful
[19:12] <blackboxsw> merge conflicts against existing ubuntu/artful.
[19:13] <blackboxsw> for all the bug comment drops
[19:13] <smoser> blackboxsw: how bout this..
[19:16] <smoser> git checkout ubuntu/artful
[19:16] <smoser> git reset --hard ubuntu/17.2-35-gf576b2a2-0ubuntu1_17.10.2
[19:16] <smoser> new-upstream-snapshot
[19:16] <smoser> dch --release --distribution=artful
[19:16] <smoser> git push chad.smith HEAD --force
[19:16] <smoser> basically,  lets just re-do it.
[19:16] <blackboxsw> yeah I did basically that.
[19:16] <blackboxsw>  git checkout -b ubuntu/artful a2ce41af52c619c13da66f84ee2a789b9df8f3a2
[19:16] <blackboxsw> and re-ran everything
[19:16] <blackboxsw> I'll use your git-fu here and try again
[19:17] <smoser> yours is probably ok now.
[19:17] <smoser> let me see. i thought you were saying it wasnt
[19:17] <blackboxsw> In the UI I see merge conflicts
[19:17] <blackboxsw> in debian/changelog
[19:17] <blackboxsw> but all the other content is good
[19:19] <smoser> yours is good.
[19:19] <smoser> k. i'll just build and push and push over upstream
[19:21] <blackboxsw> excellent. and the moral if this story is: Chad don't push branches late in the evening
[19:22] <smoser> i'm quite happy with those changes to that script.
[19:22] <smoser> i think we have covered most of the pain poitns of it.
[19:23] <blackboxsw> smoser: I love it. though I throw eggs at any shell script.
[19:23]  * blackboxsw will become better versed in shell someday.
[19:26] <smoser> blackboxsw: fair enough. it is lots of variables and such in there. definitely "shell script"
[19:26] <blackboxsw> smoser: right so although you pushed the upload, nothing showed in artful-proposed  yet so we're safe to re-push right?
[19:27] <blackboxsw> I'm looking for the proposed  build queue in launchpad again
[19:27] <blackboxsw> https://launchpad.net/ubuntu/xenial/+queue?queue_state=1&queue_text=cloud-init
[19:27] <blackboxsw> or https://launchpad.net/ubuntu/artful/+queue?queue_state=1&queue_text=cloud-init rather
[19:28] <smoser> i just uploaded artful
[19:29] <blackboxsw> ok great so that list tells me you just re-submitted same unapproved upload 30 seconds ago
[19:29] <smoser> yep
[19:29] <blackboxsw> now do we ping in ubuntu-release for attention on artful/xenial?
[19:30] <smoser> we can, yeah.
[19:30] <blackboxsw> will do.
[19:43] <blackboxsw> ok smoser I'm going to get to work on updating out manual cloud test scripts for this SRU https://hackmd.io/jlq3C4qbSgurZ_DZ5GTiuw?both
[20:25] <blackboxsw> smoser: I hadn't tested review-mps yet on non-master branches. I didn't want to automatically botch the merges
[20:26] <blackboxsw> all 3 branches are approved
[20:26] <blackboxsw> I can use review-mps to land them later if you don't land them now
[20:58] <ihre> thanks for the tip re: lxc/lxd blackboxsw, that is definitely an easier workflow than using qemu :)
[20:59] <blackboxsw> good to hear ihre, I'll probably write up a blog post about it (and use of other cloud-init CLI commands=)
[21:03] <ihre> i'll keep an eye out for it!
[21:51] <blackboxsw> ok smoser landed your ubuntu/(artful|xenial) mps and fixed up qa-scripts:review-mps tool to properly handle --upstream-branch origin/ubuntu/*
[22:09] <blackboxsw> thanks for the cloud-init release email  BTW https://lists.launchpad.net/cloud-init/msg00145.html