/srv/irclogs.ubuntu.com/2016/11/03/#juju-dev.txt

alexisbanastasiamac, I will be right there00:05
anastasiamacalexisb: me 200:05
perrito666alexisb: ill grab a bite and be back is that ok?00:05
alexisbyes perrito66600:05
babbageclunkanyone have a script for quickly opening a mongo shell on a controller?02:16
veebersbabbageclunk: I think menn0 had something that I cribbed off a while ago02:16
menn0babbageclunk: yes, I do... give me a sec02:17
wgranthttps://github.com/juju/juju/wiki/Login-into-MongoDB is what I normally use02:17
* anastasiamac cheers for ppl using wiki \o/02:18
wgrantSo I see that various Juju 1.25.7 bugs are now Fix Released, but I can't see the release anywhere. Is it mid-release, so we can expect to be able to upgrade environments in the next few days?02:18
menn0babbageclunk: http://paste.ubuntu.com/23418892/02:18
* menn0 updates the wiki02:18
anastasiamacwgrant: mid-release. should go into proposed soon. just finalising release notes :D02:19
wgrantanastasiamac: Marvellous, thanks.02:19
menn0wgrant, babbageclunk, anastasiamac: i've updated and reorganised that wiki page02:29
babbageclunkmenn0: Thanks!02:29
anastasiamacmenn0: \o/02:30
menn0babbageclunk: also remember that "juju dump-db" often negates the need for using the mongo shell02:30
babbageclunkCan anyone tell me why sometimes when I build juju the version number gets increased and sometimes it doesn't? Or rather: I think it used to (so rebuild/upgrade-juju would push out the new version), and now it appears not to.02:43
menn0babbageclunk: if you're deploying a local build the build number gets incremented02:51
menn0babbageclunk: and every time you run upgrade-juju with a local build the number gets incremented again02:52
babbageclunkmenn0: ugh, the reason my redirection wasn't happening was because my bootstrapped controllers were using binaries from the stream instead of my local ones, until I bumped the version locally and upgrade-juju'd.02:53
babbageclunkmenn0: but based on your description I don't understand why.02:53
menn0babbageclunk: if you want to be sure your local build is used, pass --build-agent to bootstrap / upgrade-juju02:54
menn0babbageclunk: i'm guessing you haven't rebased lately so your tree still has the version as 2.0.102:54
menn0babbageclunk: which means the client will use the recently released 2.0.1 instead of your own build02:55
menn0it's bitten me enough that I always use --build-agent now02:55
babbageclunkmenn0: yeah, that'll be it. I'll do that too.02:55
babbageclunkmenn0: I think I'm not in the habit because it's *usually* ahead, but not in my tree at the moment.02:56
babbageclunkmenn0: Hmm - this tree is rebased though. Although I think it might be based on the wrong branch?02:58
menn0babbageclunk: I suspect that's true for all of us02:58
menn0babbageclunk: we're usually ahead, but sometimes not02:58
menn0babbageclunk: and it often takes a while to figure out what's going on02:58
babbageclunkmenn0: It's branched off staging at the moment.02:59
menn0that's it then02:59
babbageclunkWhat should I be branching off? develop?03:00
menn0babbageclunk: staging where possible03:00
menn0babbageclunk: sticking with --build-agent should avoid the issue03:01
babbageclunkmenn0: ok, cool - thankks03:01
menn0wallyworld or axw: https://github.com/juju/juju/pull/653403:21
menn0it's big and dull (sorry)03:21
wallyworld:-(03:21
wallyworld:-)03:21
menn0wallyworld: very mechanical, you can probably skim a lot of it03:22
wallyworldwill do, didn't mean the :-(03:22
axwwallyworld: I'm thinking I'll add an option to nova.Client to disable API version discovery, and then update juju 2.0 branch to set it. sound OK to you?03:23
axwwallyworld: we've got another bug: https://bugs.launchpad.net/juju/+bug/163870403:23
mupBug #1638704: openstack provider: if use-floating-ip=true, uses incorrect compute API endpoint to determine available floating IPs <juju:Triaged by alexis-bruemmer> <https://launchpad.net/bugs/1638704>03:23
wallyworldaxw: when would we enable it again?03:23
axwwallyworld: when we use non-deprecated APIs03:24
wallyworldso all the neutron ones support the versioning?03:24
axwwallyworld: we might need to get a little bit smarter about selecting microversions when too03:24
wallyworldbut the nova network ones don't?03:24
axwwallyworld: the nova API has versioning, but we don't specify which microversion to use. so you get the latest one03:25
axwwallyworld: and the latest one has the old network API endpoints removed03:25
wallyworldah03:25
wallyworldmaybe we need to consider specifying micro version for 2.1 as well then? assuming we disable for 2.0.x as you say03:26
axwwallyworld: yes, probably a good idea to do that. maybe not immediately, but certainly need to keep it in mind03:26
wallyworldyup03:26
babbageclunkmenn0: The client side of the redirection isn't working because it doesn't pass any credentials.03:27
babbageclunkmenn0: https://github.com/juju/juju/blob/staging/juju/api.go#L7903:27
babbageclunkmenn0: rogpeppe doesn't seem to be around (presumably at the sprint)03:28
babbageclunkOh duh, he's no longer in the same timezone as me!03:28
babbageclunkmenn0: I don't know enough about macaroons to understand "we'll use macaroon authentication directly without sending account details"03:30
babbageclunkmenn0: I can see in the redirected login request it doesn't send any macaroons03:36
menn0babbageclunk: i've got to pick up a kid... will help when i'm back03:38
babbageclunkmenn0: ok03:38
axwwallyworld: https://github.com/go-goose/goose/pull/29 <- adds method so we can revert to old behaviour in juju 2.0.x03:50
wallyworldok03:50
axwwallyworld: https://github.com/go-goose/goose/pull/28 <- another fix related to volume attachments03:50
wallyworldaxw: they both look good, thanks03:54
axwwallyworld: thanks03:54
wallyworldmenn0: oh and i reviewed yours too, had a question but +103:56
menn0wallyworld: tyvm04:21
menn0babbageclunk: so getting back to this redirect issue04:25
menn0babbageclunk: a user account can either use username/password or macaroons04:25
menn0it looks like the client code doesn't accomodate that04:26
babbageclunkmenn0: I mean, I think it's right that it can't know what the user should be on the destination controller, isn't it?04:27
menn0the auth details will in fact be the same for the migration case04:27
babbageclunkmenn0: ah, ok04:27
menn0actually... they probably will be04:27
menn0not guaranteed04:27
* menn0 ponders04:29
babbageclunkso how can I find what macaroons should be attached to the login request? At the moment nothing gets passed.04:29
menn0babbageclunk: macroons will already get attached if there are some04:30
menn0babbageclunk: but the admin user won't have any, it'll be using username/password04:30
babbageclunkmenn0: How would I have gotten macaroons? If it'd been registered instead of being admin? I can check that.04:31
menn0babbageclunk: yes, when juju register is used you get a macaroon04:32
babbageclunkok, I'll try that.04:32
menn0babbageclunk: there's also a way to switch the admin user to macaroons, via "juju login" I think04:32
babbageclunkI mean, I guess we need to solve the other problem too, but it'd be good to see it working with macaroons04:33
menn0babbageclunk: yeah and i'm not sure what the right answer is for the user/password case04:33
menn0babbageclunk: we don't necessarily want to sending off passwords to arbitrary controllers04:36
menn0maybe it's ok...04:37
axwbabbageclunk: also if you use "juju change-user-password", your initial password is cleared and you get a macaroon04:39
menn0axw: thanks. I was trying to remember the trick you've mentioned before04:40
axwanastasiamac: teeny weeny review please: https://github.com/juju/juju/pull/653504:40
axwfixes 3 openstack bugs04:40
menn0babbageclunk: the more I think about it, the more I think using the username and password, if they were provided originally is probably ok04:41
babbageclunkaxw: thanks - annoyingly the sequence of things I tried seems to have left me in an unrecoverable state, other than blowing away the lxcs for my dest controller.04:41
menn0babbageclunk: not ideal but ok04:41
menn0babbageclunk: I just saw this: // TODO(rog) update cached model addresses.04:41
menn0babbageclunk: that'll need to be taken care of now04:41
babbageclunkmenn0: yes - was about to ping him about that.04:42
menn0babbageclunk: that would involve reporting back to the caller somehow that the redirect has happened, and some access to the new addresses will be required so that the caller can save them04:43
menn0babbageclunk: I wonder if it would be saner to have NewAPIConnection return the RedirectError instead of handling it all transparently04:44
menn0babbageclunk: then the caller can take care of updating the cached addresses04:44
menn0babbageclunk: and the caller can decide if it's ok to retry with username and password in place04:45
menn0babbageclunk: changing that will involve coordination with rog though04:45
anastasiamacaxw: when asked so amazingly smoothly, of course m looking \o/04:51
axwveebers: any ideas about this? http://juju-ci.vapour.ws:8080/job/github-merge-juju/9607/artifact/artifacts/trusty-err.log/*view*/05:08
=== rmcall_ is now known as rmcall
axwmgz: when you're around, can you please see if any of the CI ec2 instances can be cleaned up? merge jobs are failing like in http://juju-ci.vapour.ws:8080/job/github-merge-juju/9608/artifact/artifacts/trusty-err.log/*view*/06:20
veebersaxw: ugh sorry went EOD without disconnecting and I"m just heading out the door to an appointment now. I can try take a look when I get back in. I'm pretty sure there are scripts that clean up stale machines so it might rectify itself in a little06:56
axwveebers: np, figured you were EOD06:56
=== frankban|afk is now known as frankban
mgzmorning all09:16
mgzaxw: taking a look09:16
mgzaxw: well, that's a little scary, we really shouldn't be close to quota on aws09:21
mgzaxw: but mjs' branch has landed since, so we should be back under, I'll retrigger merge on yours09:21
axwmgz: gracias09:22
mgzaxw: the pre-check on pr #6536 also failed for that reason, can !!build!! to retrigger09:25
axwmgz: ok, thanks09:26
axwmgz: !!build!! never seems to work for me. I just added it, and there's nothing in http://juju-ci.vapour.ws/job/github-check-merge-juju/09:27
axwnothing running09:27
mgzaxw: fun. it didn't work for me either, but does for some people. I'll see if I can find out why.09:28
axwmgz: thanks09:28
mgzwell, mystery09:49
mgzconfig still doesn't have user restrictions set at all, the match is rebuild on `!!.*!!` pattern09:50
mgzand the log just states PR ... not changed09:50
mgzdocs for the jenkins plugin not very useful09:51
mgzrick_h___: are you around or busy with sessions?12:31
rick_h___mgz: bit of both12:32
rick_h___On my phone, what's up?12:33
mgzrick_h___: 1:1 - nothing urgent from me though, so I'm fine leaving it till next week12:33
mgzonly news is I've done a bunch of CI stuff this morning as we'd leaked lots of machines in us-east-1 which was causing job failures12:34
mgzback to windows this afternoon12:34
rick_h___mgz: ah sorry, did I not update the calendar?12:36
rick_h___mgz: thanks for the heads up in ci stuff12:37
voidspacemgz: care to rubber stamp this?13:00
voidspacemgz: https://github.com/juju/juju/pull/653713:00
voidspacemgz: reviewed and QA'd for develop (and still in the merge queue I think)13:00
mgzvoidspace: sure thing13:01
mgzvoidspace: lgtm13:02
voidspacemgz: tyvm13:03
rick_h___frobware: room opereta in 10min13:20
frobwarerick_h___: heading there now13:21
natefinchredir: thanks for the review of my add-cloud stuff!13:35
dooferladvoidspace, katco, natefinch, mgz: standup?14:01
katcodooferlad: oh wow... time warp. brt14:01
alexisbkatco, ping14:44
katcoalexisb: pong14:44
alexisbgood morning14:44
katcoalexisb: howdy14:44
alexisbkatco, are there folks on the rteam that would be able to looks at a regression on devel:14:45
alexisbhttps://bugs.launchpad.net/juju/+bug/163894414:45
mupBug #1638944: HA failed: timeout waiting for controller response <ci> <ha> <jujuqa> <juju:Triaged> <https://launchpad.net/bugs/1638944>14:45
katcoalexisb: nate is the only one between things; however, i just emailed torsten. it looks like issues with the CI environment are actually obfuscating blesses14:46
alexisbkatco, lovely, did you include me on that email?14:46
katcoalexisb: no, i will forward to you and rick14:46
alexisbthanks14:46
katcoalexisb: there you are14:47
katcoalexisb: let me look at the bug14:47
alexisbkatco, thanks, let me know if you guys can pick it up14:48
katcomgz: do you think this bug ^^^ is legit?14:48
katcoalexisb: also, do we know this is a regression? with HA might be more an intermittent test failure? although doesn't look like it's occurred since june?14:49
alexisbhmm, if tha tis the case then we need to push back14:50
alexisbby putting a comment in the bug and marking it invalid14:50
katcoabentley: ping14:52
abentleykatco: pong14:52
mgzkatco: the recovery test failures do seem to have an actual regression recently,14:52
katcoabentley: hey, how are you14:52
mgzbut it's somewhat obscured by being split across several generic issues14:52
abentleykatco: Not bad.  How're you doing?14:52
katcoabentley: doing ok. a bit mad that it's fall here and 85F14:53
katcoabentley: not sure how it is up there14:53
mgzbug 1626573 seems to be the most actionable one14:53
mupBug #1626573: Restore-backup cannot initiate replica set <ci> <intermittent-failure> <restore-backup> <juju:Triaged> <https://launchpad.net/bugs/1626573>14:53
abentleykatco: 11C, and it's been closer to 3 in the past week.14:53
katcoabentley: lucky :)14:53
katcomgz: ok, you think looking at this would solve more issues including the regression?14:54
mgzI think we do need to look at the ha-recovery issue, though we don't have a clean window14:55
mgzdue to all the build failures end of last week14:56
katcomgz: ok ta. it sounds like alexisb is saying this is lower priority than the crit bugs we have on our board anyway, so maybe a moot point14:56
katcoalexisb: fwiw mick is out and frobware is at the sprint. so it's just voidspace natefinch mgz and i14:57
alexisbkatco, I will try to get someone on it today but we are all pretty booked atm and most of NZ/AUS is out today given they are getting on a plane14:58
katcoalexisb: yep14:58
deanmanShould proxy settings be forwarded in all users of a newly spawned LXD?15:07
dooferladvoidspace, katco, mgz... three tiny PRs to improve your karma: https://github.com/juju/juju/pull/6538 https://github.com/juju/juju/pull/6539 https://github.com/juju/juju/pull/654015:26
* dooferlad goes for tea.15:26
mgzdooferlad: a couple of those are small bug scary :)15:28
katcooh cool a 264 line test!16:02
=== jamespag` is now known as jamespage
redirnatefinch: np16:47
voidspacemgz: care to apply another rubber stamp?17:03
voidspacemgz: https://github.com/juju/juju/pull/654117:03
mgzvoidspace: a-looking17:05
mgzvoidspace: lgtm17:06
voidspacemgz: ta :-)17:06
=== frankban is now known as frankban|afk
mgzdooferlad: your prs are being guinea pigs, don't mind me18:24
* menn0 has 2 blocks of jelly-tip chocolate for wallyworld 20:49
wallyworldyay :-D20:50
* redir steps out for a few21:17
babbageclunkmenn0: didn't get a response from rogpeppe - do you think I should just start moving the redirect handling up?21:18
babbageclunkmenn0: Or I can pause on this and start doing the status change until I/we can discuss with him.21:18
menn0babbageclunk: that might be better21:19
babbageclunkmenn0: ok cool21:19
menn0babbageclunk: i'm worried about breaking the API21:19
menn0i've already managed to annoy them once recently by breaking the API of the "api" pacakge21:19
babbageclunkmenn0: yeah, would much rather talk to him about it first - he's likely to have a good idea about how we could handle it.21:32
menn0babbageclunk: agreed21:47
redirback21:52
=== blahdeblah_ is now known as blahdeblah

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!