mwhudson | menn0, waigani_ (or anyone else): if i've bootstrapped and env and then hacked some stuff, how do i "upgrade" the env to the new jujud? | 02:58 |
---|---|---|
mwhudson | or is it easier to just destroy and bootstrap again? | 02:58 |
davecheney | mwhudson: there is juju upgrade-juju --devel | 03:01 |
davecheney | honestly I jsut guessed thatb | 03:01 |
davecheney | but there is a flag that builds new tools locally with a minor version one more than what's been deployed | 03:01 |
mwhudson | seems the flag is --upload-tools | 03:02 |
waigani_ | mwhudson: yep --upload-tools is what you want | 03:02 |
menn0 | mwhudson: remember to "go install" first :) | 03:03 |
mwhudson | heh yes | 03:04 |
mwhudson | heh so http://paste.ubuntu.com/14466151/ is too simplistic | 03:08 |
menn0 | mwhudson: what happens? | 03:09 |
mwhudson | agent-state-info: 'failed to retrieve the template to clone: lxc container creation | 03:10 |
mwhudson | failed: juju-xenial-lxc-template' | 03:10 |
davecheney | maybe there is no lxc template for xenial yet | 03:12 |
mwhudson | there isn't, i guess i need to make one | 03:23 |
mwhudson | oh | 03:23 |
mwhudson | no, i think it's probably more that i hacked too hard | 03:23 |
* mwhudson afk for now | 03:36 | |
mup | Bug #1532670 opened: Suggest next major version upgrade <2.0> <upgrade-juju> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1532670> | 04:44 |
mup | Bug #1532670 changed: Suggest next major version upgrade <2.0> <upgrade-juju> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1532670> | 04:47 |
=== ses is now known as Guest41660 | ||
mup | Bug #1532670 opened: Suggest next major version upgrade <2.0> <upgrade-juju> <juju-core:Triaged> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1532670> | 05:02 |
cherylj | If anyone's around, could I get a review? http://reviews.vapour.ws/r/3490/ | 05:33 |
anastasiamac | cherylj: i'll look... | 05:34 |
cherylj | thanks, anastasiamac ! | 05:34 |
cherylj | and now, off to bed. so tired... | 05:35 |
=== ashipika1 is now known as ashipika | ||
=== akhavr1 is now known as akhavr | ||
dooferlad | voidspace, frobware: standup? | 10:03 |
voidspace | dooferlad: omw | 10:03 |
voidspace | dooferlad: frobware is ill though | 10:03 |
=== akhavr1 is now known as akhavr | ||
=== gberginc_ is now known as gberginc | ||
mgz | master is blocked at present, new unit test fails. | 12:01 |
mup | Bug #1532777 opened: undertakerSuite.TestEnvironConfig fails <blocker> <ci> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1532777> | 12:15 |
frobware | dooferlad, that gomaasapi bug we/I looked at late on Friday was because the test didn't enable static-ipaddresses as part of the capability set. | 13:37 |
frobware | dooferlad, I take that back. | 13:47 |
dooferlad | frobware: :-| | 13:48 |
mup | Bug #1532831 opened: upgrade in progress - Juju functionality is limited after bootstrap <bootstrap> <ci> <intermittent-failure> <regression> <juju-core:Incomplete> <juju-core controller-rename:Triaged> <https://launchpad.net/bugs/1532831> | 15:32 |
natefinch | ericsnow: btw there's a couple blank review comments in the review I lefty... just ignore those. I had intended to delete them, and failed somehow. | 15:51 |
ericsnow | natefinch: np | 15:51 |
mup | Bug #1532841 opened: cannot add charm to storage <charm> <ci> <juju-core:Incomplete> <juju-core structured-metadata:Triaged> <https://launchpad.net/bugs/1532841> | 16:02 |
mup | Bug #1532849 opened: precise-amd64 and trusty-ppc64el unittests do not complete <ci> <ppc64el> <precise> <regression> <unit-tests> <juju-core:Incomplete> <juju-core structured-metadata:Triaged> <https://launchpad.net/bugs/1532849> | 16:02 |
mup | Bug #1532841 changed: cannot add charm to storage <charm> <ci> <juju-core:Incomplete> <juju-core structured-metadata:Triaged> <https://launchpad.net/bugs/1532841> | 16:05 |
mup | Bug #1532849 changed: precise-amd64 and trusty-ppc64el unittests do not complete <ci> <ppc64el> <precise> <regression> <unit-tests> <juju-core:Incomplete> <juju-core structured-metadata:Triaged> <https://launchpad.net/bugs/1532849> | 16:05 |
katco | natefinch: let's touch base after lunch on how the demo's looking | 16:05 |
mup | Bug #1532841 opened: cannot add charm to storage <charm> <ci> <juju-core:Incomplete> <juju-core structured-metadata:Triaged> <https://launchpad.net/bugs/1532841> | 16:11 |
mup | Bug #1532849 opened: precise-amd64 and trusty-ppc64el unittests do not complete <ci> <ppc64el> <precise> <regression> <unit-tests> <juju-core:Incomplete> <juju-core structured-metadata:Triaged> <https://launchpad.net/bugs/1532849> | 16:11 |
natefinch | katco: roger | 16:18 |
cherylj | frobware: if we get a bless on your revert branch, is it all reviewed to be merged? (https://github.com/juju/juju/tree/1.25-revert-maas-devices) | 16:49 |
natefinch | ericsnow, katco: reading the HTTP registry stuff kinda gives me the willies... it's a heck of a lot of code that I can't really determine if it's doing the right thing or not. Maybe it's just my unfamiliarity with the HTTP code we have. | 16:50 |
natefinch | ericsnow, katco: Gonna give it a ship it, but not sure I won't have missed some subtleties. | 16:50 |
katco | natefinch: i agree and left a comment to that effect | 16:51 |
katco | natefinch: i'd rather not ship that until we have empirically tested code that relies on it | 16:51 |
katco | natefinch: i was also unaware that we were taking that on for the demo; it's not represented in the card. | 16:51 |
katco | natefinch: ericsnow: i would have preferred to work on that later. | 16:51 |
natefinch | katco, ericsnow: gah, I left comments for the http registration on the upload api patch.... this is what you get for using dependent patches, eric :) | 16:58 |
frobware | cherylj, I have a "Ship It!" from dooferlad. | 16:58 |
katco | natefinch: ericsnow: yeah it would be nice to do a diff against that other PR to get just the important bits | 16:58 |
frobware | voidspace, want to take a quick look? http://reviews.vapour.ws/r/3474/ | 16:59 |
ericsnow | katco: ah, I must have forgotten to rbt post :( | 16:59 |
frobware | dooferlad, ping | 17:16 |
voidspace | frobware: quick look !? that's a 1500 line diff... | 17:16 |
frobware | voidspace, ah, yes, well. | 17:17 |
voidspace | maybe tomorrow morning :-) | 17:17 |
natefinch | ericsnow, katco: finished my reviews of eric's code btw | 17:37 |
perrito666 | ericsnow: ptal to http://reviews.vapour.ws/r/3392/ whenever you have a moment | 17:38 |
ericsnow | perrito666: k | 17:39 |
ericsnow | perrito666: might be a little while | 17:39 |
perrito666 | np | 17:39 |
natefinch | ericsnow: bah, zero value fingerprint is not valid | 18:58 |
ericsnow | natefinch: yeah, ran into that earlier | 18:58 |
natefinch | ericsnow: I think zero value fingerprint is valid for zero bytes of data | 18:59 |
ericsnow | natefinch: it's because it isn't initialized; GenerateFingerprint(nil) worked as an alternative | 19:00 |
ericsnow | natefinch: but I agree the zero value should be okay | 19:00 |
natefinch | ericsnow: gah, your external tests make using the tests for debugging impossible :/ | 19:03 |
natefinch | ericsnow: I guess not impossible... but more roundabout than I'd like | 19:04 |
natefinch | ericsnow: I assumed the hash for no bytes would be empty as well, but I guess it makes sense that the hash is constant length, so you have to define some hash for zero bytes. | 19:06 |
ericsnow | natefinch: right | 19:07 |
ericsnow | natefinch: however, as zero-value fingerprint should be the same a hash of an empty string | 19:07 |
ericsnow | natefinch: we'll fix that later | 19:07 |
ericsnow | natefinch: or at least discuss if that is meaningful :) | 19:08 |
natefinch | ericsnow: yeah, I can do the generate(nil) for the demo... but I would like to make the zero value valid. Shouldn't be that hard | 19:09 |
ericsnow | natefinch: agreed | 19:09 |
ericsnow | katco, natefinch: I've refactored that API patch to not need the registration (for now) | 19:17 |
ericsnow | katco, natefinch: also, I found a small bug: http://reviews.vapour.ws/r/3495/ | 19:18 |
katco | ericsnow: cool, ship it | 19:20 |
mup | Bug #1514857 changed: cannot use version.Current (type version.Number) as type version.Binary <blocker> <ci> <regression> <test-failure> <juju-core:Fix Released by natefinch> <juju-core lxd-provider:Fix Released by natefinch> <https://launchpad.net/bugs/1514857> | 19:27 |
natefinch | ericsnow: ship it | 19:29 |
natefinch | ericsnow: also, updated the demo data to use generateFingerprint(nil) | 19:29 |
ericsnow | natefinch: k | 19:29 |
katco | natefinch: confused... why even set it at all? | 19:30 |
natefinch | katco: because the default value is invalid and fails validate checks later one | 19:33 |
natefinch | on | 19:33 |
natefinch | katco: we were just talking about how we should fix that so the default value is valid for zero data | 19:33 |
katco | natefinch: yeah | 19:33 |
katco | ericsnow: ty for the refactor too | 19:35 |
ericsnow | katco: :) | 19:35 |
cherylj | Is there someone who can do a review for me? http://reviews.vapour.ws/r/3490/ | 19:42 |
katco | cherylj: lgtm. looks much more robust, kudos :) | 19:59 |
cherylj | katco: thanks!! | 20:04 |
mup | Bug #1532932 opened: Unable to bootstrap the lxd provider on vivid <adoption> <juju-core:New> <https://launchpad.net/bugs/1532932> | 20:15 |
katco | ericsnow: natefinch: fyi i'm collecting lxd bugs here: https://blueprints.launchpad.net/juju-core/+spec/charmer-experience-lxd-provider | 20:16 |
katco | ericsnow: natefinch: the one mbruzek just opened we already knew about, but weren't tracking anywhere | 20:16 |
mbruzek | katco: How do I add the bug there. | 20:17 |
katco | mbruzek: there's a "link a bug report" link; not sure if you have to own the blueprint for it to show up | 20:18 |
katco | mbruzek: i linked it for you | 20:18 |
mbruzek | katco: OK | 20:18 |
mbruzek | thanks for your help | 20:18 |
katco | mbruzek: ty for trying it out | 20:19 |
mup | Bug #1532932 changed: Unable to bootstrap the lxd provider on vivid <adoption> <juju-core:New> <https://launchpad.net/bugs/1532932> | 20:24 |
natefinch | ericsnow: oops, wrong channel... | 20:31 |
natefinch | ericsnow: anyway... looks like tests aren't hooking up the resources code | 20:31 |
ericsnow | natefinch: you have to do registration (see state/resources_test.go) | 20:32 |
mup | Bug #1532932 opened: Unable to bootstrap the lxd provider on vivid <adoption> <juju-core:New> <https://launchpad.net/bugs/1532932> | 20:33 |
katco | natefinch: have you been able to run through the demo yet? | 20:36 |
natefinch | katco: sorry, no, I was waiting for my stuff to land and then I am working on fixing the tests in that one branch... once that's done I can merge eric's branch and do an end to end test. | 20:40 |
katco | natefinch: np. demo > tests right now though | 20:41 |
natefinch | katco: right, but I can't land if the tests fail | 20:41 |
katco | natefinch: we can create a binary from a branch that's not in github.com/juju | 20:42 |
natefinch | katco: right... ok, will do that right now. | 20:42 |
katco | natefinch: cool, hope it goes smooth the first time :) | 20:42 |
natefinch | katco: it would work better if ericsnow's code compiled ;) | 20:44 |
katco | lol | 20:45 |
ericsnow | lol | 20:45 |
natefinch | 3 silly little fixes... one left | 20:48 |
natefinch | ericsnow: state.State doesn't implement GetResource? | 20:49 |
ericsnow | natefinch: correct | 20:50 |
natefinch | ericsnow: fixed it... | 20:51 |
ericsnow | natefinch: fixed what? | 20:52 |
natefinch | ericsnow: newResourceHandler needed a couple tweaks to compile | 20:52 |
ericsnow | natefinch: yeah, I fixed those | 20:52 |
natefinch | ericsnow: well then, push ;) | 20:52 |
natefinch | ericsnow: nevermind, just re-fetched... only one problem now with cannot use *"github.com/juju/juju/resource/api/client".Client as type "github.com/juju/juju/resource/cmd".UploadClient | 20:54 |
ericsnow | natefinch: yeah, the client stuff is not right yet | 20:55 |
natefinch | reader vs. readseeker | 20:55 |
natefinch | ericsnow: is there anything I can do to help? | 21:00 |
ericsnow | natefinch: not really | 21:00 |
katco | ericsnow: demo > tests if that helps | 21:02 |
ericsnow | katco: yep | 21:02 |
ericsnow | natefinch: should be ready (sans *any* test coverage on the client) | 21:44 |
natefinch | ericsnow: cool | 21:44 |
frobware | cherylj, ping | 21:45 |
cherylj | frobware: what up | 21:47 |
frobware | cherylj, are we doing some additional runs with the backed-out PRs for 1.25.2? | 21:50 |
frobware | cherylj, I was just trying to understand where we are with this before I EOD. | 21:50 |
cherylj | frobware: yes, it was supposed to get re-run today. | 21:51 |
frobware | cherylj, any news? | 21:51 |
cherylj | jog, sinzui is the revert-branch currently being tested? | 21:51 |
natefinch | katco, ericsnow: I have a branch, resource-demo (note: singular resource, sorry) it has all the code from everyone. I also have the charm at github.com/natefinch/starsay | 21:51 |
* frobware wonders if no new is good news... | 21:51 | |
katco | natefinch: nice | 21:51 |
katco | natefinch: have you tried running through the demo script yet? | 21:52 |
sinzui | cherylj: No, in the next hour it will be | 21:52 |
natefinch | katco, ericsnow: just got it building... doing so now | 21:52 |
katco | natefinch: woot woot | 21:52 |
ericsnow | natefinch: cool | 21:52 |
cherylj | frobware: while I have you here, did you get the chance to figure out what the AWS Robustness for Spaces was referring to? | 21:53 |
cherylj | :) | 21:53 |
natefinch | katco, ericsnow: doh. ERROR failed to upload resource "store-resource": PUT https://localhost:17070/environment/2a00757a-9329-401d-8ee5-4140647864c1/environment/2a00757a-9329-401d-8ee5-4140647864c1/services/starsay/resources/store-resource: request body supplied unexpectedly | 21:53 |
jog | cherylj, machine-dep-engine is still running | 21:54 |
ericsnow | natefinch: so awesome! | 21:54 |
natefinch | katco, ericsnow: that's after deploy, running juju upload | 21:54 |
ericsnow | natefinch: fixing now | 21:54 |
natefinch | ericsnow: notably, environment/UUID in there twice, too | 21:55 |
=== Guest3473 is now known as Makyo | ||
ericsnow | natefinch: indeed | 21:55 |
natefinch | katco, ericsnow: btw, I updated my demo data at deploy branch so the tests will pass: http://reviews.vapour.ws/r/3484/ | 21:56 |
frobware | cherylj, I sent some email dimiter - but we got caught up with demo stuff. he's back on Weds and will follow-up then. apologies for the delay. | 21:57 |
cherylj | frobware: okay, I'll move it to alpha2 until we can get it figured out :) | 21:57 |
natefinch | katco, ericsnow: it injects the resource code into a registry now, which is probably more like how we'd want to do it (though I imagine we'll actually want to inject DB ops to keep it transactional)... but it means that if you don't have resources registered, we won't try to use resources. | 21:57 |
natefinch | ericsnow, katco: I gotta run for a few hours for dinner and bedtime for the kids, but will be back after. | 21:58 |
katco | natefinch: kk ty | 21:59 |
katco | ericsnow: can you pick up where he left off> | 21:59 |
katco | ? | 21:59 |
natefinch | oh and to potentially save you a few minutes figuring it out, use 'juju deploy ~/src/github.com/natefinch/starsay --series trusty' to deploy starsay locally (so you don't have to put it in a charm repo or special folder) | 22:00 |
natefinch | (obviously replacing that path with the path where you clone the repo) | 22:00 |
natefinch | ok, afk now | 22:01 |
=== natefinch is now known as natefinch-afk | ||
ericsnow | katco: k | 22:02 |
mup | Bug #1527068 changed: MAAS retains child devices' IP addresses when a parent node is released <ci> <destroy-environment> <maas-provider> <network> <regression> <juju-core:Invalid> <juju-core 1.25:Triaged> <MAAS:Fix Released by mpontillo> <MAAS 1.8:Won't Fix> <https://launchpad.net/bugs/1527068> | 22:03 |
mup | Bug #1527068 opened: MAAS retains child devices' IP addresses when a parent node is released <ci> <destroy-environment> <maas-provider> <network> <regression> <juju-core:Invalid> <juju-core 1.25:Triaged> <MAAS:Fix Released by mpontillo> <MAAS 1.8:Won't Fix> <https://launchpad.net/bugs/1527068> | 22:06 |
frobware | cherylj, bug #1525280 is now for alpha2, so not a cut-off for this weds? | 22:07 |
mup | Bug #1525280: 1.25.1 with maas 1.8: devices dns allocation uses non-unique hostname <maas-provider> <network> <regression> <juju-core:Triaged by frobware> <juju-core 1.25:Fix Committed by dimitern> <https://launchpad.net/bugs/1525280> | 22:08 |
cherylj | frobware: isn't that bug no longer relevant when we back out bug #1483879? | 22:08 |
mup | Bug #1483879: MAAS provider: terminate-machine --force or destroy-environment don't DHCP release container IPs <bug-squad> <destroy-machine> <landscape> <maas-provider> | 22:08 |
mup | <sts> <juju-core:Fix Released by dimitern> <juju-core 1.24:Won't Fix> <juju-core 1.25:Fix Released by dimitern> <https://launchpad.net/bugs/1483879> | 22:08 |
cherylj | oh wait | 22:08 |
frobware | cherylj, on master? | 22:08 |
cherylj | you're talking about master | 22:09 |
cherylj | yeah | 22:09 |
cherylj | I just kinda assumed that you guys weren't going to get to it before then | 22:09 |
frobware | trying... | 22:09 |
cherylj | frobware: if you think you can get it in in the next couple days, then we can move it back | 22:09 |
cherylj | frobware: but really, there are some branches that are trying to land that are holding up alpha1 | 22:09 |
frobware | cherylj, ok. I'll do some manual (yuck!) testing tomorrow. | 22:09 |
cherylj | frobware: if you need any help, let me know | 22:10 |
frobware | cherylj, I'm beginning to like the (over the wall) testing that gets done by a feature branch. | 22:11 |
cherylj | frobware: yeah it's nice. We just need to get more hardware for the QA team since the number of branches we have to test has gone up so much! | 22:11 |
frobware | cherylj, arguably, to me at least, seems the most expedient thing we could do. | 22:11 |
cherylj | frobware: there is hardware that hasn't been set up by IS yet | 22:12 |
cherylj | I'm trying to find a name of someone who can help hurry that along | 22:12 |
mup | Bug #1527068 changed: MAAS retains child devices' IP addresses when a parent node is released <ci> <destroy-environment> <maas-provider> <network> <regression> <juju-core:Invalid> <juju-core 1.25:Triaged> <MAAS:Fix Released by mpontillo> <MAAS 1.8:Won't Fix> <https://launchpad.net/bugs/1527068> | 22:12 |
cherylj | hey natefinch-afk - is your fix for bug 1486553 in master? | 22:14 |
mup | Bug #1486553: i/o timeout errors can cause non-atomic service deploys <cisco> <landscape> <juju-core:In Progress by natefinch> <juju-core 1.24:Fix Released by natefinch> <juju-core 1.25:Fix Released by natefinch> <https://launchpad.net/bugs/1486553> | 22:14 |
cherylj | natefinch-afk: hmm, it's old. Looks like it's there, though | 22:15 |
davechen1y | cherylj: hello, how can I help ? | 23:00 |
cherylj | hey davechen1y welcome back! | 23:00 |
cherylj | davechen1y: we're seeing seemingly random test failures on wily and xenial | 23:00 |
cherylj | davechen1y: sinzui is working on compiling a list of the failures for you to look at | 23:01 |
davechen1y | cherylj: ok, i'll get on it | 23:01 |
cherylj | thanks davechen1y! | 23:01 |
davechen1y | can you paste links to the failing builds ? | 23:02 |
cherylj | sinzui: do you have examples you can give davechen1y to look at while you're compiling the list? | 23:02 |
sinzui | cherylj: Still pasting. | 23:06 |
sinzui | davechen1y: cherylj: This is what we know, which is not much :( https://pastebin.canonical.com/147376/ | 23:30 |
perrito666 | sinzui: I know this is probably a stupid question but, do you have a moment? | 23:31 |
sinzui | perrito666: I do have a moment | 23:45 |
perrito666 | sinzui: trying something else so I dont waste my time when I ask you the question | 23:49 |
perrito666 | s/my/your | 23:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!