[02:06] blackboxsw: ah. yeah, that we shoudl get there... as you can see it just called util.json_dumps [06:35] Hello cloud-init! If I drop a script under /var/lib/scripts/per-boot ... is it right to assume that by the time cloud-init picks up on it the instance/machine I'm running on will have networking up? [06:35] It's not entirely clear to me when reading https://cloudinit.readthedocs.io/en/latest/topics/boot.html :-S [07:15] @powersj, great blogposts about cloud-init, btw! ;) [15:20] agreed, good work powersj. [15:21] thanks :) === r-daneel_ is now known as r-daneel [18:38] blackboxsw: i'm going to set 'approved' on azure-network-per-boot [18:38] ok? === r-daneel_ is now known as r-daneel [19:12] blackboxsw: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/352921 ... review and approve there would be grand. [19:12] will get that now [20:01] powersj: is https://jenkins.ubuntu.com/server/job/admin-lp-git-autoland/ something we need to kick today (instead of automated trigger) [20:01] smoser: and I both have cloud-init branches that are queued approved [20:02] blackboxsw, I disabled testing while deej and I were playing with the network settings [20:02] give me 20 to re-test things so we don't get silly failures [20:02] should be setup now for the weekend [20:02] ok. i was just about to ask that too [20:02] gotcha sounds good [20:02] blackboxsw: i'd like a upload to cosmic today. [20:02] should I unset approve on the cloud-init branches for 20? [20:02] smoser: +1 me too [20:02] nah nothing will run [20:02] ok leaving branches in approved state [20:03] i'm tempted ... really tempted to merge manually. [20:03] will put up a cosmic MP 5 mins after they land [20:03] smoser: your week is almost over. [20:03] maybe it's best for these two so you don't have to hang around waiting on me [20:03] ok. ... we think review-mps should work, right ? [20:03] we'll have more branches to autoland monday (removal of deprecated snap MPs, and jinja template hopefully) [20:04] smoser: I think it should I'll do one test locally against my own repo [20:11] smoser: looks good locally shall I [20:11] ? [20:11] review-mps rather [20:14] blackboxsw, I enabled testing [20:14] watching jobs notifications... [20:15] you beat the manual response. L( [20:15] :) [20:15] so I won't touch review-mps then [20:15] yeah looks like there are 2 getting tested currently [20:15] hopefully serially :) [20:15] otherwise we risk a merge collision I suppose [20:16] test runs concurrently, lander is serially [20:17] ahh I misunderstood. ok you are running lander tests on non-cloud-init public reviews [20:17] ok [20:18] non-cloud-init? [20:18] ok looks like those are only ~5 mins anyway [20:18] ok [20:18] * powersj choose a bad week to make this change [20:18] next week would be a bad week [20:19] yeah :\ [20:19] :) [20:21] ok, just so smoser doesn't have to stay late I'll run review-mps on one branch now [20:21] will that land them? [20:22] blackboxsw, ^ [20:22] that'll land it in master [20:22] well... there are 2 branches almost done [20:22] let me see what is under test [20:22] cancelled, the branch names are yours looked like [20:22] I thought you were running 1-off tests [20:23] looks like scaleway? [20:23] 51f49dc1da0c045d01ddc977874c9bed70cb510f [20:24] not a valid approved review [20:24] we should be looking at just oracle or azure related branches [20:24] so I thought you were doing a little test branch run [20:24] " blackboxsw, I enabled testing" [20:25] DEBUG: Merging feature/azure-network-per-boot to master [20:25] that just landed [20:25] yeah "testing" was vague in my interpretation... testing meant your own test branch runs, "enabled autolander" meant ci was going [20:25] ahh [20:25] ah [20:25] DEBUG: Merging feature/oracle-datasource to master [20:25] that just landed [20:25] ahh interesting, so the description on the given test run [20:26] is tip of master, not the commit of the branch under test=... [20:26] ok [20:26] good, I had cancelled my local review-mps on the azure branch [20:26] ah yeah! that's because we checkout master and merge in the review [20:26] that is misleading :\ [20:26] yeah makes sense then [20:26] ok so cloud-init ci is good [20:26] now gotta go look at curtin [20:26] excellent [20:27] also have an integration test running [20:27] now that I have ppa access again [20:27] thanks, again powersj . ok I'm cutting a cosmic release branch [20:27] for review [20:32] https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/353353 for cosmic release [20:33] smoser: ^ [20:33] * blackboxsw is running through a quick upgrade check on azure [20:54] azure upgrade path good on cosmic MP [20:56] minor pre-existing bug found with cloud-init status if someone uses the unsupported CLI path of 'sudo cloud-init init; cloud-init status --wait'. When running any of the stages on the cmdline directly after-boot, we delete /run/cloud-init/result.json unless the stage run is modules-final... this break cloud-init status --wait into lying that cloud-init is "running" [21:10] minor bug we don't really have to address https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1787657 [21:10] Ubuntu bug 1787657 in cloud-init (Ubuntu) "cloud-init status reports 'running' after 'cloud-init init --local' run on commandline" [Low,New] [21:12] ok CI approved cosmic branch [21:12] will dput once there is a 2nd review on changes in https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/353353 [21:28] blackboxsw: sorry, went into a python2.6 hole [21:29] I believe it [21:29] 'tis fine. I'm just working on the render subcommand (and a couple other tuneups for CLI) [21:29] blackboxsw, have a merge for cii fixes incoming [21:30] * blackboxsw ducks [21:30] heh [21:30] well it's that or lots of emails this weekend ;) [21:30] I guess those are easy to deal with [21:30] blackboxsw: i just approved that. [21:30] thanks for landin the others [21:30] thanks auto-powers [21:31] blackboxsw: shoot [21:32] smoser: listening? [21:33] http://paste.ubuntu.com/p/gy2vWgQ8Mx/ [21:33] that needs doing (and smoke testing) on the ubuntu/devel branch [21:33] best ifdone before you uplaod [21:34] smoser: ahh right, pkg update [21:36] h [21:36] ok. i have to run [21:37] will d [21:37] do [21:40] have a good one [22:14] blackboxsw, https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/353355 if you aren't still trying to get cosmic out [22:31] powersj: just finished cosmic upload [22:31] sweet [22:31] will look after curtin upload [22:31] :) [22:31] thank you! [22:31] * blackboxsw adds another tab... this time to firefox [22:32] I'm separating my abuse of browser tabs to avoid chrome memory throttling