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