menn0waigani: would you mind looking at: http://reviews.vapour.ws/r/2456/01:22
waiganimenn0: sure01:22
menn0waigani: it's a fix for intermittent allenvwatcher test failures that i've noticed01:22
menn0waigani: there's another problem to fix as well but this one issue is happening more often01:22
waiganimenn0: shipit01:25
menn0waigani: thanks01:36
davecheneyi love the fact that all our upgrade and provisioner tests use "ppc64le" as the "i cannot possibly happen" operating system01:39
anastasiamacdavecheney: isn't it easier to support impossible operating systems? :D01:41
davecheneywe're basically using quantal-ppc64le as our "nope, cannot happen" sentinal02:06
mwhudsondavecheney: good think the ubuntu architecture is called ppc64el then02:37
mwhudsondoh can't brain today02:37
davecheneyaxw_: godeps: cannot get information on "/home/dfc/src/launchpad.net/gwacl": bzr revision-info has unexpected result "244.1.1 andrew.wilkins@canonical.com-20150810080433-4l0x47qsc16ure7c\n"02:49
davecheneymwhudson: oh god02:50
davecheneydon't mention the war02:50
davecheneyor the worst joke in the world that continues to keep on giving02:50
davecheneystate tests now take 450+ seconds to pass03:09
davecheney% go test ./state/...03:09
davecheneyok      github.com/juju/juju/state      438.504s03:09
davecheneygolf clap03:09
davecheneyFAIL    github.com/juju/juju/state/leadership   600.006s03:09
davecheneyfucking fantastic03:09
mupBug #1487947 opened: provider/ec2: localServerSuite.TestNetworkInterfaces  <juju-core:New> <https://launchpad.net/bugs/1487947>03:12
mupBug #1487947 changed: provider/ec2: localServerSuite.TestNetworkInterfaces  <juju-core:New> <https://launchpad.net/bugs/1487947>06:46
dimiternrogpeppe, hey, have you seen https://code.launchpad.net/~dimitern/godeps/fix-bzr-revision-info-parsing/+merge/268223 ?07:16
rogpeppedimitern: nope :)07:16
rogpeppedimitern: only just back from 2 weeks' holiday07:16
dimiternrogpeppe, have a look when you have a moment :) it's just 2 characters change07:17
rogpeppedimitern: looking07:17
dimiterndooferlad, TheMue, hey guys, it seems net-cli is blessed for merging in master07:17
rogpeppedimitern: interesting. out of interest, do you actually want to use non mainline revision numbers as dependencies?07:20
dimiternrogpeppe, I suppose not, but at least when there is such a case now godeps refuses to work anymore07:21
rogpeppedimitern: yeah, i agree it shouldn't fail in that case07:21
dimiternrogpeppe, maybe print a warning when there are dots in the bzr revision and go on?07:21
rogpeppedimitern: i think i'll just go with your fix actually. if you really want a non mainline revision, you can have one07:22
rogpeppedimitern: it should be obvious in the .tsv file anyway07:22
dimiternrogpeppe, sounds good07:22
rogpeppedimitern: reviewed07:26
dimiternrogpeppe, thanks!07:31
dimiternrogpeppe, I think you should manually merge the MP, as there's no bot to pick it up07:32
rogpeppedimitern: merged (ooo, my bzr-fu is getting rusty!)07:38
dimiternrogpeppe, :) mine too07:40
dooferladdimitern: great! Lets do it and go drink some beer!07:57
* dooferlad knows that TheMue would rather drink whisky07:57
TheMuedooferlad: good idea, got time next week monday. wonna meet in London?07:57
TheMuedooferlad: ah, you know me right07:58
dooferladTheMue: I think you meen tuesday. Well, maybe Monday evening...08:00
TheMuedooferlad: yeah, Monday evening08:01
dimiterndooferlad, voidspace, standup?09:01
voidspacedimitern: trying, it's not letting me in :-/09:01
voidspacerestarting the browser09:01
dimiternvoidspace, oh boy :/ monday hangout issues09:01
voidspacedimitern: not having used this machine for six weeks issues too...09:02
voidspace350mb of updates09:02
voidspaceincluding a new firefox - that maybe the issue09:02
voidspacethat looks better09:02
voidspacedimitern: ready10:03
dimiternvoidspace, omw10:03
dimiternfwereade, hey, are you around today?11:33
TheMuedimitern: btw, the provider I meant has been the vsphere one. i has a partial implementation of the networking interface12:49
dimiternTheMue, ok, so for vsphere then SupportsSpaces will be false, but the other method will remain12:53
TheMuedimitern: yep, did it that way12:53
dimiternTheMue, +112:54
redelmannhi, im having some problem with juju-gui13:11
redelmannmachine-0 logs says:  error stopping *state.Multiwatcher resource: unit not found13:11
redelmanneverytime i load juju-gui on browser13:12
redelmannand no icons appears13:12
redelmannthis is a juju bug or gui?13:12
redelmanni open this issue: https://bugs.launchpad.net/juju-gui/+bug/148524913:18
mupBug #1485249: Juju gui is not loading. <juju-gui:New> <https://launchpad.net/bugs/1485249>13:18
rogpeppeanyone know about the openstack provider?13:56
urulamadimitern: ^13:57
mupBug #1486106 changed: TestNetworkInterfaces fails on vivid/wily <ci> <ec2-provider> <intermittent-failure> <test-failure> <juju-core:Fix Released by dooferlad> <https://launchpad.net/bugs/1486106>13:59
rogpeppein particular at this moment I'm interesting in understanding the different roles of the tenant-name (taken from env var OS_REGION_NAME) and region (no env var) attributes13:59
mgzrogpeppe: you have that mixed up, tenant-name is taken from OS_TENANT_NAME14:01
mgzit's a user-specific grouping of resources14:02
rogpeppemgz: so they are - not sure how i read that wrong14:02
mgzwhereas regions are a cloud-level seperation14:02
rogpeppemgz: for some reason i'd copied/pasted the wrong env vars and didn't check back to the source14:03
rogpeppemgz: i did think it was potentially very confusing :)14:03
rogpeppemgz: one weird thing: why does openstack fall back to using the ec2 passwords?14:04
mgzrogpeppe: there's a semi-compatible ec2 api, so many of the rcs set the same vars for ec214:07
rogpeppemgz: it seems like a potential security leak to me14:07
rogpeppemgz: and i don't see any other values that do the same thing14:08
wwitzel3fwereade: ping14:08
fwereadewwitzel3, pong14:08
wwitzel3fwereade: katco isn't feeling well, are you able to move up our discussion to now? so she can take off and get some rest14:09
fwereadewwitzel3, sure, let's14:09
rogpeppemgz: and surely if it was using AWS_SECRET_ACCESS_KEY it ought to use AWS_ACCESS_KEY_ID too?14:09
wwitzel3fwereade: we are in moonstone, https://plus.google.com/hangouts/_/canonical.com/moonstone?authuser=114:09
mgzrogpeppe: the names used are just based on what was looked at in practice by the clients14:12
mgzand there's not really a security issue, the access ley/secret key are not passed along unless you say you're using that authmode14:15
rogpeppemgz: do you think it might be reasonable to drop support for using AWS_SECRET_ACCESS_KEY and EC2_SECRET_KEYS for the openstack password?14:15
mgzso you need to be pretty deliberate about wanting it14:15
rogpeppemgz: isn't userpass the default auth mode?14:16
rogpeppemgz: so ISTM that it'll send that password in the usual case14:17
mgzand it's basically always going over https to somewhere that won't be loging your auth attempts14:17
mgzyou'd need to have and endpoint url manually set to something hostile, or dns poisoned14:18
rogpeppemgz: from the p.o.v. of the user that's not necessarily true - they don't know what's at the other end14:18
mgzand also have accidentally sourced your ec2 creds14:18
mgzI don't think the ec2 provider is robust against that14:18
mgzwe don't check that the ec2 endpoint is really amazon14:19
rogpeppemgz: i think we do, don't we14:19
rogpeppemgz: 'cos we use https14:19
rogpeppemgz: without using InsecureSkipVerify14:19
mgzthat just verifies it has been signed by something in your chain14:19
mgzit doesn't verify what it is.14:20
rogpeppemgz: well, same security as any website :)14:20
rogpeppemgz: and, yes, i'm well aware that https security is a farce14:20
mgzthe ec2 signing mechanism is better though, in not passing along the secret14:21
rogpeppemgz: but we all blithely go on assuming that verisign et al are somewhat reliable14:21
rogpeppemgz: i mean, fair enough i guess if people actually use AWS_SECRET_ACCESS_KEY for their openstack password, we don't wanna break that14:22
rogpeppemgz: it just triggers my Something Wrong Here sense.14:22
mgzI think it's likely not really used any more, all the current rc things will at least include both, and generally only the OS_ versions14:23
mgzjust whether it's worth futzing with14:23
rogpeppemgz: i'm moving the env vars into the openstack provider to do the environschema stuff, and it made me wonder...14:24
rogpeppemgz: i mean, maybe goose.v1/identity should export the env var names14:25
mupBug #1465404 changed: worker/provisioner: fail lxcBrokerSuite.TestStartInstanceLoopMountsDisallowed <intermittent-failure> <unit-tests> <juju-core:Fix Released by dooferlad> <https://launchpad.net/bugs/1465404>14:29
mupBug #1473209 changed: github.com/juju/juju/service/windows undefined: newConn <ci> <intermittent-failure> <test-failure> <unit-tests> <windows> <juju-core:Fix Released by bteleaga> <juju-core trunk:Fix Released by bteleaga> <https://launchpad.net/bugs/1473209>14:29
mupBug #1465404 opened: worker/provisioner: fail lxcBrokerSuite.TestStartInstanceLoopMountsDisallowed <intermittent-failure> <unit-tests> <juju-core:Fix Released by dooferlad> <https://launchpad.net/bugs/1465404>14:35
mupBug #1473209 opened: github.com/juju/juju/service/windows undefined: newConn <ci> <intermittent-failure> <test-failure> <unit-tests> <windows> <juju-core:Fix Released by bteleaga> <juju-core trunk:Fix Released by bteleaga> <https://launchpad.net/bugs/1473209>14:35
=== redelmann is now known as popi_
=== popi_ is now known as redelmann
=== redelmann is now known as s0plete
=== s0plete is now known as redelmann
dimiternfwereade, hey14:39
dimiternfwereade, does this look in line with what we discussed? http://reviews.vapour.ws/r/2463/14:40
rogpeppemgz: this is what i've got: http://paste.ubuntu.com/12184056/14:41
mupBug #1465404 changed: worker/provisioner: fail lxcBrokerSuite.TestStartInstanceLoopMountsDisallowed <intermittent-failure> <unit-tests> <juju-core:Fix Released by dooferlad> <https://launchpad.net/bugs/1465404>14:41
mupBug #1473209 changed: github.com/juju/juju/service/windows undefined: newConn <ci> <intermittent-failure> <test-failure> <unit-tests> <windows> <juju-core:Fix Released by bteleaga> <juju-core trunk:Fix Released by bteleaga> <https://launchpad.net/bugs/1473209>14:41
rogpeppemgz: does that look reasonable to you?14:41
rogpeppemgz: actually, i should avoid copy/pasting those env vars14:42
mgzrogpeppe: you have EC2_SECRET_KEYS ... with an S14:45
rogpeppemgz: that's what goose uses14:46
mgzalso, not really to do here, but we should add tenant-id already14:46
mgzheh, in which case you probably can just delete it, as it's been wrong for a while then14:47
rogpeppemgz: :)14:47
rogpeppemgz: i suspect the same of AWS_SECRET_ACCESS_KEY too14:47
mupBug # changed: 1458717, 1461393, 1463641, 1467372, 1474788, 1478660, 148461714:47
rogpeppemgz: but i'm actually thinking i don't want to risk breaking anything at all14:47
mgzrogpeppe: no, that one is right, the old naming there was just dumb14:48
rogpeppemgz: i know it's right - i just think that using an AWS name for openstack creds is silly14:48
rogpeppemgz: particularly when it doesn't use AWS_ACCESS_KEY_ID for one of the user names14:49
mgzit was the only way to use non-password auth on the old hp cloud14:49
mgzas there aren't OS names for the different values14:49
rogpeppemgz: ah, makes sense14:51
rogpeppemgz: ok, here's a review for you: https://github.com/go-goose/goose/pull/1414:59
mupBug #1459064 changed: worker/leadership: data races in test and code <intermittent-failure> <tech-debt> <unit-tests> <juju-core:Fix Released by dave-cheney> <https://launchpad.net/bugs/1459064>14:59
dimiternvoidspace, dooferlad, TheMue, please have a look at these last 2 PRs we need to unblock merging net-cli into master: http://reviews.vapour.ws/r/2463/ and http://reviews.vapour.ws/r/2464/14:59
mupBug #1460893 changed: many unhandled assigned values <tech-debt> <juju-core:Fix Released by dave-cheney> <https://launchpad.net/bugs/1460893>14:59
* dimitern steps out, bbl15:01
frobwarehello all - new starter on the Juju Sapphire team today.15:02
mgzrogpeppe: looks fine. as I hinted earlier, there are some bigger bits that could do with fixing in the os auth stuff, but nothing that blocks this change.15:02
mgzfrobware: hey there!15:02
rogpeppemgz: thanks15:02
mgzfrobware: another Andrew?15:03
frobwaremgz, tis the way...15:03
ericsnowfwereade: we didn't mean to chase you away! :)15:06
mupBug #1459064 opened: worker/leadership: data races in test and code <intermittent-failure> <tech-debt> <unit-tests> <juju-core:Fix Released by dave-cheney> <https://launchpad.net/bugs/1459064>15:11
mupBug #1460893 opened: many unhandled assigned values <tech-debt> <juju-core:Fix Released by dave-cheney> <https://launchpad.net/bugs/1460893>15:11
rogpeppemgz: could you land my goose branch for me please? i'm not a member of goose collaborators currently15:14
mupBug #1459064 changed: worker/leadership: data races in test and code <intermittent-failure> <tech-debt> <unit-tests> <juju-core:Fix Released by dave-cheney> <https://launchpad.net/bugs/1459064>15:17
mupBug #1460893 changed: many unhandled assigned values <tech-debt> <juju-core:Fix Released by dave-cheney> <https://launchpad.net/bugs/1460893>15:17
redelmannanyone know about juju-gui?15:18
redelmannim trying to debug an issue15:18
natefinchredelmann: probably better to ask in #juju15:18
redelmannthank natefinch15:19
voidspacedooferlad: I see you reviewed the first one, are you looking at the second too?15:19
mgzrogpeppe: erm... can't remember exactly but the bot may actually just still check if you're a juju memeber15:19
dooferladvoidspace: yes15:19
mgzrogpeppe: added the comment anywa15:19
rogpeppemgz: thanks15:19
voidspacedooferlad: ok, I'll leave it to you :-)15:21
dooferladvoidspace: I haven't graduated yet. I suggest you look at the first one first - it seems to have been committed with the second as well.15:21
dooferladvoidspace: then you know which bits to ignore :-)15:21
voidspaceI did think the second had some extraneous stuff in it15:22
voidspacedooferlad: dimitern: first looks good and it tells me which bits of the second I can ignore too15:23
voidspacein which case the second looks trivial15:24
voidspacedooferlad: although I missed the concern you raised, need to read the code agian!15:24
mupBug #1488139 opened: juju should add nodes IPs to no-proxy list <juju-core:New> <https://launchpad.net/bugs/1488139>15:29
mupBug #1488139 changed: juju should add nodes IPs to no-proxy list <juju-core:New> <https://launchpad.net/bugs/1488139>15:35
mupBug #1488139 opened: juju should add nodes IPs to no-proxy list <juju-core:New> <https://launchpad.net/bugs/1488139>15:48
bodie_is the answer to why godeps is used rather than godep a simple one?16:01
alexisbwelcome frobware !16:01
mgzbodie_: godeps is older than godep16:05
bodie_I see.  thanks mgz o/16:07
rogpeppemgz and anyone else: fancy a smallish review of a change to the openstack provider code? http://reviews.vapour.ws/r/2465/16:26
dimiternkatco & moonstone: fwereade texted me about an hour ago his power's out and wanted to convey this apologies16:27
dimiternvoidspace, dooferlad, thanks for the reviews guys!16:27
mgzrogpeppe: looks fine in general, the descriptions are a bit all over the place though, do you want nitpicks?16:30
rogpeppemgz: specific suggestions would be great16:30
rogpeppemgz: i copied the descriptions from the boilerplate16:30
rogpeppemgz: much appreciated16:31
mgzah, so they're now it two places? that's a bit unfortunate.16:31
rogpeppemgz: when i get some extra time, the plan is to generate the boilerplate from the schema16:31
mgzhow much will you hit me over the head if I ask you about generating the boilerplate from the schema?16:31
mgzeheh, you win.16:32
rogpeppemgz: i'm also not liking my decision to go with partial sentences for the descriptions. using go-like full sentences seems like it's a better plan in the end.16:33
rogpeppemgz: but these things are fairly easy to change16:34
mgzrogpeppe: review'd16:38
rogpeppemgz: ta v much16:38
rogpeppemgz: not sure about mentioning that control-bucket will be auto-generated - that's kinda true of every parameter that's optional.16:39
mgzI'm a little antsy on that one as setting it can be actually harmful... but it can also just be required16:41
rogpeppewhen would it be required?16:46
rogpeppemgz: perhaps a comment in the description saying "(do not set this unless you know what you are doing)" might be appropriate16:47
mgzrogpeppe: yeah, but leave it for now, don't need to futz with here.16:47
rogpeppemgz: k16:48
mgzwe need it for ci testing on canonistack, I *think* it's related to being able to cleanup reliably16:48
=== urulama is now known as urulama__
mupBug #1488166 opened: Service with just one unit left which doesn't think it's the leader <landscape> <leadership> <juju-core:New> <https://launchpad.net/bugs/1488166>16:57
perrito666bbl groceries shopping17:21
natefinchericsnow: getting this error when running featuretests: charm not found in "/home/nate/src/github.com/juju/juju/testcharms/charm-repo": local:quantal/workload-actions17:44
ericsnownatefinch: is it actually there (/home/nate/src/github.com/juju/juju/testcharms/charm-repo/quantal/workload-actions)?17:46
ericsnownatefinch: perhaps still named "proc-actions"?17:47
ericsnownatefinch: (which should be fixed)17:47
natefinchericsnow: doh, yep17:49
natefinchI blame wwitzel317:49
ericsnownatefinch: :)17:49
natefinchericsnow: I'll submit a PR to rename them17:51
ericsnownatefinch: sounds good17:51
natefinchI think I got whatever katco has.  I should remember to wear a mask to hangouts from now on.19:28
mupBug #1488245 opened: Recurring lxc issue: failed to retrieve the template to clone  <canonical-bootstack> <juju-core:New> <https://launchpad.net/bugs/1488245>20:54
=== natefinch is now known as natefinch-afk
davechen1yping http://reviews.vapour.ws/r/2458/21:47
perrito666davechen1y: ship it, I am happy that that is being worked :)22:22
perrito666there is only a typo in the description22:22
davechen1yperrito666: thanks23:10

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