/srv/irclogs.ubuntu.com/2016/05/04/#juju-dev.txt

mupBug #1577988 opened: Revert destroy service when machine is off <juju-core:New> <https://launchpad.net/bugs/1577988>00:07
wallyworldredir: ericsnow: meeting?01:04
=== natefinch-afk is now known as natefinch
menn0axw: ping02:28
axwmenn0: pong, in a meeting atm02:28
menn0axw: np... let me know when you're done please02:28
axwmenn0: sorry, back now02:51
thumpereither, we suck horribly02:51
thumperor the last CI test run had issues02:51
* thumper gives each one about 50%02:52
menn0axw: np... I think I've mostly figured it out but i'll confirm with you anyway02:52
menn0axw: you did the debughooks work right?02:52
axwmenn0: yes, long ago02:53
menn0axw: I'm in the final throes of integrating the SSH host key stuff and noticed the debughooks tests don't do what they appear to02:53
menn0axw: https://github.com/juju/juju/blob/master/cmd/juju/commands/debughooks_test.go02:53
menn0axw: in debugHooksTests the proxy and result fields aren't used :)02:53
menn0axw: I guess I just wanted be sure that I wasn't missing something before fixing02:54
axwmenn0: :/  they used to be02:55
axwmenn0: don't think you're missing anything02:55
menn0axw: must have been later edits that broke the tests02:56
menn0axw: I need to rework them anyway. i'm trying to get them to pass again.02:56
menn0(with my changes)02:57
mupBug #1508923 changed: Support for Azure Resource Groups <azure-provider> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1508923>03:13
mupBug #1291201 changed: Azure provider does not use storage service's affinity group <azure-provider> <juju-core:Won't Fix> <https://launchpad.net/bugs/1291201>03:43
axwwallyworld: I was expecting at least an "argh!" response from someone to using sync.Cond :)04:29
wallyworldaxw: i can live with it :-)04:29
wallyworldseems to do what's needed nicely04:30
axwwallyworld: yeah, it's appropriate IMO. people reach for channels because they're easy, but they're not always sufficient04:31
wallyworld+104:31
wallyworldwhen you have a hammer, everything looks like a nail04:31
axwwallyworld: were you looking at the restore-backup bug? ISTR you were looking at something in the area04:31
wallyworldyeah04:31
axwwallyworld: thanks for review, please see reply on BuildToolsTarball PR04:48
wallyworldok04:48
wallyworldaxw: yeah, i was wondering if there were more tests. it seems ok i think04:50
=== frankban_ is now known as frankban
voidspacemgz: ping08:54
voidspacemgz: unping - emailed instead08:56
babbageclunkvoidspace, dimitern: standup? Am I in the wrong one? You guys missed the fire alarm test!09:01
voidspaceomw09:01
dimiternoops sorry omw09:01
voidspacebabbageclunk: eager beaver09:02
voidspacebabbageclunk: keen bean09:02
babbageclunkvoidspace, dimitern: porting tests to MAAS2 http://reviews.vapour.ws/r/4766/09:45
voidspacebabbageclunk: looking09:46
voidspacebabbageclunk: LGTM09:49
babbageclunkvoidspace: thanks#!09:49
babbageclunk-#09:49
dimiternbabbageclunk: still reviewing btw09:51
babbageclunkdimitern: d'oh, sorry09:51
dimiternbabbageclunk: np :) I should've said I did09:51
babbageclunkdimitern: luckily there are blockers anyway09:51
dimiternbabbageclunk: reviewed09:54
babbageclunkdimitern: thanks, fixed. How do I spell jfdi for this again?09:59
dimiternbabbageclunk: $$__JFDI__$$10:00
babbageclunkdimitern: Thanks10:00
dimiternbabbageclunk: thanks for chasing all those CI issues so quickly :)10:01
babbageclunkdimitern: :)10:02
babbageclunkhmm - godef stopped working for stdlib packages after upgrading to xenial. Any hints for how to make it work again?10:29
dimiternbabbageclunk: what error are you getting?10:33
babbageclunkdimitern: Just eg "godef: no declaration found for sort.Strings"10:34
babbageclunkdimitern: still works fine for local types and cross-package ones.10:37
dimiternbabbageclunk: ah, you might have GOROOT set (or unset - can't remember what was supposed to work)10:37
dimiternbabbageclunk: I use GOROOT=/usr/lib/go-1.6 and that works with golang-1.6 from xenial/main10:39
babbageclunkdimitern: thanks, I'll try that. Wonder why it stopped.10:39
babbageclunkdimitern: yay, thanks! Worked great.10:41
dimiternbabbageclunk: \o/ :)10:41
babbageclunkdimitern: man, felt like I had a phantom limb for a bit.10:43
dimiternbabbageclunk: re godef not working? :)10:45
babbageclunkdimitern: yeah, just got so used to hitting a key to pull up the type of a thing that when it stopped I felt like I had an itch in a place that I didn't have a place anymore.10:47
dimiternbabbageclunk: I know what you mean - I use and depend on godef all the time; I only wish it worked better for stuff in export_test.go :/10:48
dimiternbabbageclunk: I have M-. bound to godef-jump and C-M-. to godef-jump-other-window10:51
babbageclunkdimitern: Man, I love godef-jump-other-window10:52
dimiternbabbageclunk: another very useful package to add is go-eldoc btw - it shows type info in the minibuffer10:54
babbageclunkdimitern: ooh, just as you move around? I've been hitting C-c C-d for that, but something that was always there would be nicer.10:55
dimiternbabbageclunk: yeah, e.g. with point at 'errors.IsNotFound' it shows 'errors.IsNotFound: func(err error) bool'10:56
dimitern(assuming errors is already imported)10:56
dimiternbabbageclunk: most of the tricks in http://dominik.honnef.co/posts/2013/03/writing_go_in_emacs/ and its follow-up are good to know10:57
babbageclunkdimitern: nice, thanks - I've got go-mode set up, but not all of these.10:59
* dimitern steps out for ~1h11:55
* dimitern is back12:47
mgzdimitern: `juju help add-subnet` mentions `juju create-subnet` - which doesn't exist?12:53
dimiternmgz: oh, yeah - it's under a feature flag "post-net-cli-mvp", but the help doc should mention it12:55
dimiterns/should/shouldn't/12:55
mgzokay, that's a new flag to me12:57
mgzshould it work on aws/maas at present? (create-)12:57
dimiternmgz: no, it doesn't work.. well, the flag it changes a few things if enabled, like registering extra networking commands, but the commands themselves are not fully implemented (end-to-end)13:01
alexisbjam, fwereade ping13:05
alexisbjam and fwereade officially owe me a beer at the sprint13:28
alexisbvoidspace, babbageclunk, dimitern, mgz, again really well done on the maas 2.0 stuff!13:32
dimiternalexisb: indeed! it's exciting to see all coming together so quickly!13:33
babbageclunkGah, does anyone know why strace -c -f is giving me negative times?13:45
mupBug #1576509 changed: Race in macaroon-bakery <blocker> <ci> <race-condition> <regression> <test-failure> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1576509>13:48
mupBug #1576527 changed: listSuite.TestListJSON got null (showSuite too) <blocker> <ci> <go1.6> <regression> <test-failure> <unit-tests> <windows> <juju-core:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1576527>13:48
mupBug #1576911 changed: github.com/juju/juju/environs/bootstrap timeout (sigquit) <blocker> <centos> <ci> <go1.6> <regression> <test-failure> <unit-tests> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1576911>13:48
* babbageclunk is out code clubbing.13:57
mupBug #1576509 opened: Race in macaroon-bakery <blocker> <ci> <race-condition> <regression> <test-failure> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1576509>14:00
mupBug #1576527 opened: listSuite.TestListJSON got null (showSuite too) <blocker> <ci> <go1.6> <regression> <test-failure> <unit-tests> <windows> <juju-core:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1576527>14:00
mupBug #1576911 opened: github.com/juju/juju/environs/bootstrap timeout (sigquit) <blocker> <centos> <ci> <go1.6> <regression> <test-failure> <unit-tests> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1576911>14:00
mupBug #1576509 changed: Race in macaroon-bakery <blocker> <ci> <race-condition> <regression> <test-failure> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1576509>14:12
mupBug #1576527 changed: listSuite.TestListJSON got null (showSuite too) <blocker> <ci> <go1.6> <regression> <test-failure> <unit-tests> <windows> <juju-core:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1576527>14:12
mupBug #1576911 changed: github.com/juju/juju/environs/bootstrap timeout (sigquit) <blocker> <centos> <ci> <go1.6> <regression> <test-failure> <unit-tests> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1576911>14:12
voidspacealexisb: thanks14:19
mupBug #1578237 opened: lxd Raw response debug logging too verbose <jujuqa> <logging> <lxd> <juju-core:Triaged> <https://launchpad.net/bugs/1578237>14:24
natefinchbogdanteleaga: got a minute?14:47
natefinchor gsamfira?14:49
bogdanteleaganatefinch, what's up14:50
natefinchbogdanteleaga: I changed jujud's TLS configuration to only support TLS 1.2 last week... however, this is proving a problem for windows machines trying to download tools.  Seems like the .Net WebClient doesn't always default to having TLS 1.2 enabled.14:51
natefinchbogdanteleaga: I think it's something we can enable in the cloudconfig script we send up, but wanted to get your input on it... this code specifically: https://github.com/juju/juju/blob/master/cloudconfig/userdatacfg_win.go#L19714:52
bogdanteleagayup, noticed it yesterday, got a fix, but still testing it across windows versions14:52
natefinchbogdanteleaga: oh, awesome. Thanks!14:52
mupBug #1537153 changed: juju deploy --config option ignored when deploying a bundle <2.0-count> <juju-release-support> <juju-core:Invalid> <https://launchpad.net/bugs/1537153>14:54
mupBug #1565872 changed: Juju needs to support LXD profiles as a constraint <adoption> <juju-release-support> <lxd> <juju-core:Triaged> <https://launchpad.net/bugs/1565872>14:54
mupBug #1578254 opened: Race in apiserver/common and apiserver/proxyupdater <blocker> <ci> <race-condition> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1578254>14:54
katconatefinch: standup time15:02
perrito666does anyone have the critical bugs link?15:23
natefinchperrito666: blockers: https://bugs.launchpad.net/juju-core/+bugs?field.tag=blocker+15:24
perrito666tx15:24
perrito666almost everything is in progress sweeeeet15:24
mupBug #1554819 changed: juju help create-model is misleading <juju-release-support> <juju-core:Triaged> <https://launchpad.net/bugs/1554819>15:24
mupBug #1554819 opened: juju help create-model is misleading <juju-release-support> <juju-core:Triaged> <https://launchpad.net/bugs/1554819>15:33
perrito666does anyone oppose at me taking https://bugs.launchpad.net/juju-core/+bug/1578254 ?15:37
mupBug #1578254: Race in apiserver/common and apiserver/proxyupdater <blocker> <ci> <race-condition> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1578254>15:37
mupBug #1554819 changed: juju help create-model is misleading <juju-release-support> <juju-core:Triaged> <https://launchpad.net/bugs/1554819>15:42
mupBug #1561212 changed: register logic can lead to user lockout <2.0-count> <docteam> <juju-release-support> <juju-core:Fix Released> <https://launchpad.net/bugs/1561212>15:43
mupBug #1576313 changed: windows: uniter tests fail because logs get dumped to stderr <blocker> <ci> <regression> <test-failure> <windows> <juju-ci-tools:Triaged> <juju-core:Won't Fix by hduran-8> <https://launchpad.net/bugs/1576313>15:43
natefinchperrito666: go for it.  I was going to look at it, but hadn't gotten there yet15:45
perrito666sorry15:45
* perrito666 notices that all our criticals are in progres :D go team15:45
natefinchperrito666: it's all the same to me15:45
natefinch\o/15:46
natefinchsinzui: did you get in touch with the MAAS guys about the centos7 image?15:50
sinzuinatefinch: oh, sorry, they never replied. I will poke again15:50
natefinchsinzui: do you mind if I split https://bugs.launchpad.net/juju-core/+bug/1576873 into two bugs?  One for centos and one for windows?  They're likely to be fixed in different ways by different people15:51
mupBug #1576873: Juju2 cannot deploy centos or windows workloads on maas 1.9 <blocker> <centos> <ci> <maas-provider> <regression> <windows> <juju-core:In Progress by natefinch> <https://launchpad.net/bugs/1576873>15:51
mgznatefinch: that sounds sensible16:00
natefinchmgz: there's no built-in like duplicate this bug or split this bug into two or anything, is there?16:00
natefinchmgz: nevermind, copy and paste is quick enough16:02
mgznatefinch: yeah, that should do16:02
natefinchthere - https://bugs.launchpad.net/juju-core/+bug/157828616:04
mupBug #1578286: Juju2 cannot deploy windows workloads on maas 1.9  <blocker> <ci> <maas-provider> <regression> <windows> <juju-core:In Progress by natefinch> <https://launchpad.net/bugs/1578286>16:04
natefinchalso edited the original to remove the windows-specific stuff16:04
mgzthanks nate16:04
mupBug #1578286 opened: Juju2 cannot deploy windows workloads on maas 1.9  <blocker> <ci> <maas-provider> <regression> <windows> <juju-core:In Progress by natefinch> <https://launchpad.net/bugs/1578286>16:07
redirericsnow: I'm here, give me a few minutes and I am ready16:36
ericsnowredir: k16:37
rcjWho can answer rackspace provider questions?16:47
redirericsnow: in Moonstone16:49
rcjericsnow, the rackspace development account loaned to juju from cpc has a litter of monitoring entities for old juju machines from prior deployments.  These might be old but I'd like you to check and ensure that the provider does clean these up correctly if it's not doing that already.16:50
rcjericsnow, I'm seeing them @ https://intelligence.rackspace.com/cloud/entities16:50
ericsnowrcj: OTP, but you might want to check with mgz, abentley_, or sinzui16:52
rcj^ mgz, abentley_, sinzui: Can someone from Juju track this down?  I'd like to hand this off.16:53
mgzrcj: what exactly is a monitoring entity?16:54
rcjgood question16:54
sinzuircj: my question too16:54
rcjsomething in the UI at that spot in the webapi.  we see a bunch on the test account we loaned out for use in provider development.16:54
sinzuircj: I delete servers every few days, but not Odd_Bloke's server16:55
Odd_Blokesinzui: < me, earlier> I've never used RAX.  < rcj> You have a server in there.  < me> ... oh ...16:56
rcjsinzui, not Odd_Bloke's servers.  this is just some entity, probably created at vm creation, that is hanging around presumably left by the juju rax provider given the entity "juju-<hash>-machine-#" names.  Wanted to make sure someone was aware and would look at the provider to ensure a clean user experience.16:56
mgzrcj: so, we're not deliberately creating those, I guess we might get them as a side effect of creating machines?16:56
rcjmgz, ack16:57
sinzuiOdd_Bloke: oh, it is gone now, but there was one for 2 weeks16:57
mgzin which case it's unclear why the matching destroy doesn't clear them16:57
Odd_Blokesinzui: Yeah, I know; I am not a smart man.16:57
Odd_Bloke:p16:57
mgzrcj: I have no idea what to do about this17:00
mgzit's not even clear from the docs how you configure the behaviour17:01
sinzuimgz: rcj: cannot find this in the UI to get a list17:01
sinzuimgz: I think we need a bug for juju and one for ourselves. We need a cleanup script for rackspace now17:02
sinzuiah, rackspace inteligence17:03
rcjhttps://intelligence.rackspace.com/cloud/entities17:08
mupBug #1541445 changed: empty uuid in cache.yaml after destroy-controller <2.0-count> <juju-release-support> <juju-core:Invalid> <https://launchpad.net/bugs/1541445>17:13
redirericsnow: same thing different branch http://reviews.vapour.ws/r/4767/17:42
niedbalskisinzui, mgz I'd might need a fix on the upstart service (1.25); What's the target branch for doing a merge proposal?17:42
sinzui1.2517:43
sinzui^ niedbalski17:43
mupBug #1578327 opened: juju 1.25.5: Juju Login with invalid user tag should return CodeUnauthorized <landscape> <juju-core:New> <https://launchpad.net/bugs/1578327>17:43
sinzuiniedbalski: https://github.com/juju/juju/tree/1.2517:43
niedbalskisinzui, yep, but afaik this is part of packaging and it seems that the debian sources are not on the same tree, right?17:44
sinzuiniedbalski: That is the branch we will make 1.25.6 from.17:44
sinzuiniedbalski: 1.25 does have different packaging rules, but to be clear NO juju has upstart or systemd rules in packaging.17:46
sinzuiniedbalski: jujud generates its system files on installation. So jujud supports both systemd and upstart. It chooses which files to generate based on the host OS. this is also true for windows and centos17:47
redirericsnow: tx17:48
niedbalskisinzui, cool. It seems that the service definition template lives in github.com/juju/juju/service/upstart/upstart.go, thanks!.17:49
sinzuifab17:49
sinzuiballoons: r=me17:50
balloonssinzui, ack. I'll remove the tweaks to the jobs to call my code and enable the flow17:53
sinzui:)17:54
mupBug #1578337 opened: no command to remove controllers <juju-core:New> <https://launchpad.net/bugs/1578337>18:28
mupBug #1578337 changed: no command to remove controllers <juju-core:New> <https://launchpad.net/bugs/1578337>18:31
frobwareI was running `juju upgrade-juju --upload-tools' and ran into "ERROR POST https://192.168.19.150:17070/model/63ffd949-a9c4-4319-8b70-c33d505cc7ef/tools?binaryVersion=2.0-beta7.2-xenial-amd64&series=trusty,quantal,saucy,yakkety,wily,precise,vivid,xenial,raring,utopic Forbidden Port" - is this a known issue?18:36
mupBug #1578337 opened: no command to remove controllers <juju-core:New> <https://launchpad.net/bugs/1578337>18:43
katcothe true test of a tiling wm... my mouse cursor disappears =|19:11
redirkatco: i3?19:29
katcoyep19:29
katcoredir: yep19:29
redirkatco: mine doesn't19:29
katcoredir: doesn't what?19:29
redirdisappear19:29
katcoredir: oh, no, i wouldn't expect it to. i've been using i3 for a few years, i've just triggered some kind of bug19:30
rediroic19:30
katcoredir: and i expect it's not a bug in i319:31
natefinchmy cursor disappears about once a month, but I use unity19:31
katconatefinch: redir: it seems to happen when i mouse over the ubuntu terminal19:31
katconatefinch: redir: not all the time, but when the bug happens that seems to be what i was doing19:31
redirrunning the 1.x tests wipes out all open terminals or spams them with systemd messages19:31
natefinchkatco: easy fix, don't do that ;)19:31
katcoo.019:31
katconatefinch: you're absolute right. i should be using the emacs terminal ;p19:32
redirkatco: ahhh, I have terminator set as sensible-terminal19:32
natefinchredir: yeah, the systemd thing with the 1.x tests is super annoying19:32
mgzdid you guys open a bug for that?19:33
natefinchmgz: you mean for the mouse thing?19:35
mgzI mean for running the tests doing weird things19:35
natefinchmgz: I think there was one open for it.... don't remember exactly19:36
* redir thought it was a feature19:48
redirericsnow: another same thing different branch review http://reviews.vapour.ws/r/4768/ -- no major rush as the target is blocked.19:49
redirbrb reboot19:50
mupBug #1578373 opened: unblock panic: invalid memory address or nil pointer dereference <jujuqa> <panic> <unblock> <juju-core:Triaged> <https://launchpad.net/bugs/1578373>20:35
natefinchmoved a true unit test to a test suite that doesn't do setup or teardown, shortened test length by 2/3rds :/20:50
perrito666anyone could check this very brief yet critical patch? http://reviews.vapour.ws/r/4769/20:50
katconatefinch: \o/ keep doing stuff like that. we'll widdle away at this problem yet!20:51
katcowhittle?20:51
katcoi don't know how to spell that.20:51
perrito666still sounds foreign to me :p20:51
mupBug #1578376 opened: Cannot add MAAS credentials through juju add-credential <juju-release-support> <maas-provider> <usability> <juju-core:Triaged> <https://launchpad.net/bugs/1578376>20:53
mupBug #1578383 opened: incorrect extension on resource upload <juju-core:New> <https://launchpad.net/bugs/1578383>20:53
natefinchkatco: it's just one of many reasons why I don't like testsuites... it's too easy to fall into the trap of just adding another test to the existing suite, even if it doesn't need any of the setup or teardown.20:53
katconatefinch: yes. it would be much better to share functionality by making shared code functions, not methods.20:54
natefinchkatco: yep20:56
mgzkatco: I'm pretty sure 'widdle' is as in, to pee on20:58
katcomgz: o.o20:58
katcomgz: what do you brits know about the english language anyhow20:59
mgzso unless our test suite is on fire, I'm not sure we widdling away on it will help...20:59
katcomgz: ...well20:59
mgz:P20:59
katcomgz: it is a tire fire, so.20:59
natefinchlol20:59
natefinchwiddling on a tire fire probably doesn't help ;)21:00
natefinchespecially one of this size21:00
* natefinch pauses widdling for dinner21:01
=== natefinch is now known as natefinch-afk
babbageclunkSo this is what you folks talk about when we go to sleep.21:13
babbageclunk:)21:13
redirperrito666: why can't one use %#v with tomb?21:36
perrito666we think that might trigger, through reflection, reading on attributes being written in other routines21:40
redirThanks perrito66621:45
thumperugh21:52
thumperbit by my first golang 1.6 issue (I'm still on 1.5)21:53
mupBug #1576695 changed: Deployer 2 on trusty cannot talk to Juju2 because :tlsv1 alert protocol version <ci> <deployer> <maas-provider> <python2.7> <regression> <juju-core:Invalid> <juju-deployer:New> <https://launchpad.net/bugs/1576695>21:53
mupBug #1576873 changed: Juju2 cannot deploy centos workloads on maas 1.9 <blocker> <centos> <ci> <maas-provider> <regression> <cloud-images:New> <juju-core:Won't Fix by natefinch> <https://launchpad.net/bugs/1576873>21:53
mupBug #1577798 changed: azure 400 bad request creating networks <azure-provider> <blocker> <bootstrap> <ci> <juju-core:Invalid> <https://launchpad.net/bugs/1577798>21:53
babbageclunkhey menn0, thumper - any idea why strace starts giving me nonsense numbers when I pass -f (trace children)?21:56
thumperbabbageclunk: no idea sorry21:56
babbageclunkSeems like maybe it's something like it's subtracting too much from the times to compensate for tracing overhead. cProfile can do that sometimes in Python land.21:57
menn0babbageclunk: doesn't ring any bells for me either21:58
babbageclunk:(22:01
alexisbthumper, wallyworld ping22:02
babbageclunkAhh well, also passing overhead of 0 doesn't help.22:02
perrito666wow, I completely forgot about the graduated reviewer thing22:16
perrito666hey, could someone review http://reviews.vapour.ws/r/4769/diff ?22:17
wallyworldthumper: can you join https://plus.google.com/hangouts/_/canonical.com/moonstone22:40
perrito666oh great, we are in one of those "no reviewers" timezone holes :p22:42
axwwallyworld: when you're free, can I please get your eyeballs on http://reviews.vapour.ws/r/4765/22:46
wallyworldsure22:46
thumperwallyworld: still want me/23:00
thumper?23:00
thumperwas making coffee23:00
wallyworldthumper: all good, there was a question over payload persistence. eric will look to move it across tomorrow23:01
thumperok, cool23:01
axwwallyworld: http://reviews.vapour.ws/r/4770/ another one please23:34
wallyworldsure23:34
wallyworldaxw: ooops, i missed that in review23:35
axwwallyworld: all good, it's subtle23:36
perrito666wallyworld: I might need to ignore your comment, some checks might trigger a race even though is harmless23:46
wallyworldperrito666: not sure how. we already access the NotifyId or whatever it is from the result params, we just need to check the Error attribute too23:47

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