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

anastasiamacis anyone working with tip of master?00:16
anastasiamaci cannot complile..00:16
anastasiamacmongo/open.go:120: undefined: tls.TLS_RSA_WITH_AES_128_GCM_SHA25600:16
anastasiamacmongo/open.go:121: undefined: tls.TLS_RSA_WITH_AES_256_GCM_SHA38400:16
anastasiamac(and yes i've ran godeps) :-D00:16
anastasiamacany suggestions?00:17
natefinchanastasiamac: using go 1.6?00:41
natefinchanastasiamac: my guess is you're using an older version that didn't have that defined00:42
anastasiamacnatefinch: no.. not go1.6 :( i guess i need to move \o/00:49
natefinchanastasiamac: we're officially a go 1.6 shop now. Which is nice except for the build times.  If you're feeling brave, use master of the go repo. They keep it really stable and nice, and compile times have dropped by about 25%00:52
anastasiamacnatefinch: my reservation was that CI is not fully 1.6?...00:53
anastasiamacnatefinch: u know feeling brave and being brave are a bit different ... m certainly not feeling too brave today :D00:54
natefinchanastasiamac: the call was made a day or two ago that we're abandoning go 1.2 entirely in CI.  The only reason we were keeping it around was for windows and I think centos tests that failed more on 1.6... but it was decided that we should just fix the 1.6 tests ,not rely on 1.2 to make those pass.00:54
natefinchanastasiamac: which I entirely agree with... given t hat we were shipping 1.6, but only testing on 1.200:55
natefinch(for windows and centos)00:55
anastasiamacnatefinch: sure.. we still have PRs that require 1.6 and we are holding off landing them coz .. well we are not brave :D00:56
natefinchanastasiamac: well, time to be brave, because we're 100% 1.6 now, AFAIK.00:57
cmarswallyworld, CQRS? http://martinfowler.com/bliki/CQRS.html00:58
wallyworldcmars: yeah, that's it00:58
cmarscool00:58
anastasiamacaxw: r we brave enough ^^^ to land azure PR once master is unblocked?00:59
axwanastasiamac: should be fine, I'll probably bump up to the latest SDK before doing so01:00
axwanastasiamac: intending to test it again today, and hten base the retry changes on top01:00
anastasiamacaxw: \o/01:00
redirgo 2016:)01:34
redirg'nite juju-dev01:34
=== blahdeblah_ is now known as blahdeblah
=== Makyo is now known as Makyo|Vacay
natefinchhmm... good to know.. for some reason ssh doesn't like my id_rsa key pair02:18
mupBug #1578898 opened: cmd/juju/commands: bootstrap tests are fetching GUI metadata from streams.canonical.com <juju-core:Triaged> <https://launchpad.net/bugs/1578898>03:10
mupBug #1578906 opened: Rackspace no longer works with Juju <blocker> <ci> <rackspace-provider> <regression> <juju-core:Triaged> <juju-release-tools:Triaged> <https://launchpad.net/bugs/1578906>04:10
=== rogpeppe2 is now known as rogpeppe
dooferladvoidspace: guess you are the only sapphire person online today so no standup?09:06
voidspacedooferlad: I was there briefly09:06
voidspacedooferlad: but on my own so I left :-)09:07
voidspacedooferlad: where's dimiter?09:07
dooferladvoidspace: no idea09:07
voidspaceah, holiday09:07
voidspacepublic holidays on two fridays in a row - nice :-)09:07
dooferladvoidspace: heh. I could do with a holiday. Or a day off sick.09:08
dooferladvoidspace: or just passing out for 24 hours09:08
voidspacedooferlad: TGIF09:08
dooferladvoidspace: something like that09:08
voidspace:-)09:09
voidspacenice and sunny here - and hopefully lasting into the weekend09:09
voidspacewe're planning a trip to bluebell woods09:09
voidspacedooferlad: you got any plans?09:09
dooferladprobably sleep09:09
dooferladLiz and I both have stinking colds09:09
voidspace:-(09:10
voidspacehope you get well09:10
voidspace*quickly09:10
dooferladthanks :-|09:10
voidspacetoo fast on the old enter key09:10
rogpeppeanyone know how to remove a model in state (using the state package API) ?09:15
rogpeppefwereade: I seem to have forgotten how to remove things from the state... ^ :)09:18
fwereaderogpeppe, model.Destroy()?09:18
rogpeppefwereade: that sets life to dead but doesn't seem to remove it09:18
rogpeppefwereade: i'm wondering if Cleanup is the thing to use09:20
fwereaderogpeppe, the last thing to do with a dead model is RemoveAllModelDocs09:21
rogpeppefwereade: ah, thanks. I think I might've expected that to be named Model.Remove09:22
fwereaderogpeppe, yeah, that would indeed be the sane thing to call it. not sure why the implementation details got leaked into the name there09:24
rogpeppefwereade: that works BTW, thanks!09:27
fwereaderogpeppe, cool :)09:27
rogpeppehere's a fix for juju using excessive numbers of mgo sockets in some cases: http://reviews.vapour.ws/r/4783/)10:25
mupBug #1579002 opened: state: uses too many mgo sockets in loops <juju-core:New> <https://launchpad.net/bugs/1579002>10:32
mupBug #1579002 changed: state: uses too many mgo sockets in loops <juju-core:New> <https://launchpad.net/bugs/1579002>10:35
mupBug #1579002 opened: state: uses too many mgo sockets in loops <juju-core:New> <https://launchpad.net/bugs/1579002>10:53
mupBug #1579010 opened: state: removing model can generate huge transactions <juju-core:New> <https://launchpad.net/bugs/1579010>10:53
bogdanteleagakatco, might be too late, but pong10:55
mupBug #1579010 changed: state: removing model can generate huge transactions <juju-core:New> <https://launchpad.net/bugs/1579010>10:56
mupBug #1579010 opened: state: removing model can generate huge transactions <juju-core:New> <https://launchpad.net/bugs/1579010>11:02
mupBug #1579051 opened: Race in juju/controller/destroy and TestDestroyCommandConfirmation <ci> <race-condition> <regression> <juju-core:New> <https://launchpad.net/bugs/1579051>12:30
mupBug #1462966 changed: worker/provisioner: multiple data races <race-condition> <juju-core:Fix Released> <https://launchpad.net/bugs/1462966>12:51
mupBug #1470297 changed: worker/uniter/storage: data race in test <race-condition> <unit-tests> <juju-core:Fix Released> <https://launchpad.net/bugs/1470297>12:51
mupBug #1519183 changed: featuretests: tests fail under -race because of crappy timing issues <2.0-count> <race-condition> <juju-core:Fix Released> <https://launchpad.net/bugs/1519183>12:51
mupBug #1579057 opened: Race in github.com/juju/juju/worker/catacomb/catacomb <blocker> <ci> <race-condition> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1579057>12:51
mupBug #1579059 opened: MainSuite.TestFirstRun2xFrom1x fails on windows <blocker> <ci> <regression> <test-failure> <unit-tests> <windows> <juju-core:Triaged> <https://launchpad.net/bugs/1579059>12:51
mupBug #1579062 opened: localHTTPSServerSuite no trusty arm64/ppc64el images <arm64> <blocker> <ci> <ppc64el> <regression> <test-failure> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1579062>13:21
mupBug #1571914 changed: github.com/juju/juju/cmd/jujud unit tests fail if xenial is the LTS <blocker> <test-failure> <juju-core:Fix Released by reedobrien> <juju-core 1.25:Fix Released by reedobrien> <https://launchpad.net/bugs/1571914>13:30
mupBug #1576021 changed: 1.25.6 cannot deploy on CI maas 1.9 or 1.8 <blocker> <ci> <maas-provider> <regression> <juju-ci-tools:Triaged> <juju-core:Invalid by dooferlad> <juju-core 1.25:Fix Released by dooferlad> <https://launchpad.net/bugs/1576021>13:30
mupBug #1576368 changed: blockdevice 2.0 schema check failed: model: expected string, got nothing <blocker> <ci> <deploy> <maas-provider> <juju-core:Fix Released by 2-xtian> <https://launchpad.net/bugs/1576368>13:30
katcofwereade: hey, having trouble figuring out how to implement a timeout with a worker (i.e. worker.Wait(), but also continue after a certain amount of time). is there any prior art?14:47
=== frankban_ is now known as frankban
alexisbkatco, happy friday!14:51
katcoalexisb: it is friday14:51
alexisbkatco, not urgent but when you have a chance we need to add this bug on the bug squad board w/ blocker tag: https://bugs.launchpad.net/juju-core/+bug/157890614:51
mupBug #1578906: Rackspace no longer works with Juju <blocker> <ci> <rackspace-provider> <regression> <juju-core:Triaged> <juju-release-tools:Triaged> <https://launchpad.net/bugs/1578906>14:51
natefinchkatco: at least no one will ask you to get up at 5:30 tomorrow (probably) :D14:52
katcoalexisb: i'll do that now14:52
fwereadekatco, sorry, restate please?14:53
katcofwereade: if i write a worker, and i only want to continue contingent on waiting on the worker or a timeout, can you point me to any prior art?14:55
katcofwereade: i want to do something like this: w := NewFooWorker; select { case <-w.Wait: case <- time.After(5*time.Minute)}14:56
fwereadekatco, ah ok -- I think I'd just do something like http://paste.ubuntu.com/16260449/ -- and then Wait for the worker which I'd trust to notify me if it hadn't really finished its job14:58
katcofwereade: (headed to meeting) where does the abort channel come from? that's what i can't figure out how to safely get out of a worker14:59
fwereadekatco, the abort channel is the abort channel for the agent, or whatever it is, that's running this local logic14:59
fwereadekatco, (leaving a goroutine leaked to kill an already-dead worker in the future, if the process survives that long, is not really a big deal but it's untidy ;p)15:00
fwereadekatco, (and it makes it hard to move the logic around safely too, I think)15:01
fwereadekatco, sane?15:02
katcoericsnow: standup time15:03
katcofwereade: will digest in a bit, sorry15:03
fwereadekatco, historical interlude: Dead() and Err() are the methods you might have to use on, say, an old-style watcher that mixes lifetime and notification concerns15:04
fwereadekatco, np :)15:04
fwereadekatco, if you *need* a Dead chan, you could build one like this: http://paste.ubuntu.com/16260672/15:07
alexisbfwereade, have you seen this: https://bugs.launchpad.net/juju-core/+bug/157905715:21
mupBug #1579057: Race in github.com/juju/juju/worker/catacomb/catacomb <blocker> <ci> <race-condition> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1579057>15:21
fwereadekatco, I think there's something we haven't quite figured out re: workers that should run indefinitely, vs workers that exist for a specific task: *most* of our workers are indefinite, and Kill->no-error makes sense there, narrowly; but for workers that are *expected* to complete of their own accord, ErrAborted makes sense15:21
fwereadealexisb, looking15:21
fwereadealexisb, looks like it's in the uniter remotestate stuff? can take a look shortly though15:25
alexisbfwereade, that would be awesome, thank you15:25
alexisbkatco, I added the remaining critical blockers for beta7 on the bug squad board15:31
alexisbkatco, if people are looking for something to do they are there :)15:31
katcoalexisb: ta... ericsnow picked up the rackspace one15:32
alexisbI saw that :)15:33
alexisband fwereade picked up one and dimitern picked up another15:33
katconatefinch: redir: more blocking bugs are on the board for when you're ready15:33
alexisbso just 2 left15:33
katcoalexisb: ... for now (dun dun dun!)15:33
alexisbnatefinch, katco: if nate could finish up the manual provider bug (including making sure 2.0 works) that would be awsome, and priority15:34
katcoalexisb: he has a PR up against the upstream project15:34
katcoalexisb: but i think it's contingent on them accepting it, or us vendoring that project and carrying the patch15:34
natefinchyep.. I'm writing a test for it now15:35
alexisbah I see15:35
alexisbok, sorry missed that piece15:35
alexisbnatefinch, please put test results in the bug15:35
katcofwereade: ok, digested your comments. so manage the timeout outside the worker and kill when it's timed-out?15:38
fwereadekatco, I think so, yeah15:39
fwereadekatco, and I think I've realised something about abort chans -- *workers* don't actually need them, because you can always ust Kill enternally15:40
fwereadekatco, it's only long-running *funcs* that need abort chans15:40
fwereadekatco, I think :)15:40
katcofwereade: hm. i'd have to use your pastebin to expose a channel that signals when the worker is dead?15:42
fwereadekatco, you can just Wait, can't you?15:42
katcofwereade: for reboot? no, as written, the reboot will go ahead after a timeout, even if we're still seeing containers up15:43
fwereadekatco, start the worker going, start the timeout-killer, wait for the worker to stop one way or the other15:43
katcofwereade: oh, so pass an abort into the worker that comes from the timeout-killer15:44
fwereadekatco, it doesn't even need that, does it? the interaction via Kill() is all we need15:44
katcofwereade: so something like: go func() { time.Sleep(5*time.Minute); worker.Kill() }?15:45
fwereadekatco, (and if you want it to be stopped when the enclosing context stops, manage that via catacomb and you get stop-when-parent-stopped for free)15:46
fwereadekatco, yeah15:46
fwereadekatco, that's the leaked goroutine I don't *really* care about -- the abort chan in the first pastebin would be to clean that up when the worker stopped15:47
mupBug #1579127 opened: Cannot deploy windows nano <blocker> <deploy> <windows> <juju-core:Triaged> <https://launchpad.net/bugs/1579127>15:51
fwereadehttps://github.com/juju/juju/pull/5356 hasn't bee picked up by RB for some reason, but fixes lp:157905716:26
fwereadecmars, you free to review^^?16:26
fwereadealexisb, sinzui: ^^16:26
cmarsfwereade, sure, looking16:27
cmarsfwereade, LGTM16:29
fwereadecmars, cheers16:30
fwereadealexisb, https://bugs.launchpad.net/juju-core/+bug/1579057 $$merge$$ing16:30
mupBug #1579057: Race in github.com/juju/juju/worker/catacomb/catacomb <blocker> <ci> <race-condition> <regression> <juju-core:Triaged by fwereade> <https://launchpad.net/bugs/1579057>16:30
alexisbthanks!16:32
mgzericsnow: I belive I have fixed rackspace16:41
ericsnowmgz: sweet16:41
perrito666well lxd hates me today16:42
mgzjust today?16:42
rick_h_mgz: yay, does it need the next beta to work?16:42
rick_h_mgz: or trunk I should say?16:43
mgzrick_h_: no, it seems like part of our image streams got deleted from the account16:43
rick_h_mgz: ok cool ty much!16:43
mgzI regenerated using the script from last time, and it passed the deploy test16:43
mgzrick_h_: I have a couple of (somewhat related) things to bug you about if you have a sec16:44
mgzrick_h_: we have a couple of failures in CI due to out of date MAAS images, which nominally cpc managed (but I think smoser knows most about)16:46
rick_h_mgz: ok16:46
mgzrick_h_: bug 1568895 and bug 157687316:47
mupBug #1568895: Cannot add MAAS-based LXD containers in 2.0beta4 on trusty <ci> <jujuqa> <lxd> <maas-provider> <juju-core:Triaged> <https://launchpad.net/bugs/1568895>16:47
mupBug #1576873: 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>16:47
mgzso, if you have pat or someone on hand to kick that would be ace, sinzui is also going to email and beg16:47
* rick_h_ is looking16:48
mgzrick_h_: subject #2, just making sure you've mentioned to john grimm this week that we really want someone on the server team who'll respond to packaging review requests etc from us16:49
mupBug #1579148 opened: dhclient needs reconfiguring after bridge set up <network> <juju-core:Triaged> <https://launchpad.net/bugs/1579148>16:51
mupBug #1578906 changed: Rackspace no longer works with Juju <blocker> <ci> <rackspace-provider> <regression> <juju-ci-tools:In Progress by gz> <juju-core:Invalid by ericsnowcurrently> <https://launchpad.net/bugs/1578906>17:06
mupBug #1578906 opened: Rackspace no longer works with Juju <blocker> <ci> <rackspace-provider> <regression> <juju-ci-tools:In Progress by gz> <juju-core:Invalid by ericsnowcurrently> <https://launchpad.net/bugs/1578906>17:18
sinzuimgz: rick_h_ : I am returning to the maas email now. I had to rescuse a child from school.17:19
mgzsinzui: had good news, rick said both bugs are being worked on17:21
sinzuithe maas issue?17:21
mupBug #1578906 changed: Rackspace no longer works with Juju <blocker> <ci> <rackspace-provider> <regression> <juju-ci-tools:In Progress by gz> <juju-core:Invalid by ericsnowcurrently> <https://launchpad.net/bugs/1578906>17:22
=== frankban is now known as frankban|afk
mupBug #1579173 opened: wily onfigSuite.TestNewModelConfig test failure lxd <blocker> <ci> <lxd> <regression> <test-failure> <unit-tests> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1579173>18:19
mupBug #1579173 changed: wily onfigSuite.TestNewModelConfig test failure lxd <blocker> <ci> <lxd> <regression> <test-failure> <unit-tests> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1579173>18:22
rick_h_sinzui: mgz yes, last report was images with backports were in biulding/testing18:27
sinzui\o/18:27
sinzuithank you rick_h_18:27
mupBug #1579173 opened: wily onfigSuite.TestNewModelConfig test failure lxd <blocker> <ci> <lxd> <regression> <test-failure> <unit-tests> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1579173>18:31
redirsinzui: is there an arm64/ppc64el system I can test on ?18:39
sinzuiredir: sure. I think you need a xenial host18:40
redirsinzui: I don't think I understand what you mean.18:41
sinzuiredir: I think the arm64 host is best, but the xenial ppc64el host is idle. Both hosts are on very stricited networks. I will need to pass you some ssh config to get to them18:41
redirsinzui: OK let me know.18:42
sinzuiredir: I think you are working on https://bugs.launchpad.net/juju-core/+bug/1579062 which most often happens on xenial18:42
mupBug #1579062: localHTTPSServerSuite no trusty arm64/ppc64el images <arm64> <blocker> <ci> <ppc64el> <regression> <test-failure> <unit-tests> <juju-core:In Progress by reedobrien> <https://launchpad.net/bugs/1579062>18:42
redirsinzui: correct18:43
sinzuiokay, I will get yoiu acces to both ppc64el and arm64. You can work on them as you need18:43
* redir goes to make tea18:43
redirsinzui: tx18:43
sinzuiredir: check you email19:01
redirsinzui: thanks. Did you add my key to the jump-host?19:05
sinzuiredir: I didn't...I thought I had done that for the s390x host. I will add it since your question implies I need to19:06
redirsinzui: cool. tx.19:08
sinzuitry it now redir19:09
redirsinzui: viola. merci19:10
redirsinzui: is this also expected to be an issue on 1.x?19:43
sinzuiredir: no, the failure wasn't see on 1.25 when it was tested yesterday19:45
redirOK. the provider/openstack tests pass on both platforms.19:49
redirsinzui:^ I'll refrain running the full suite since the hosts seem pretty oversubscribed.19:50
sinzuiredir: the arm64-slave host should be idle for about 60-90 minutes19:51
redirreview anyone? https://github.com/juju/juju/pull/535819:53
redirRB hasn't picked it up yet19:53
redirmanually created one http://reviews.vapour.ws/r/4785/20:04
redirBueller?20:14
natefinchredir, ericsnow: No space left on device: '/tmp/reviewboard.UipOWm'20:15
natefinch    20:15
ericsnownatefinch: gah20:15
redirwhoops20:15
ericsnownatefinch: time to add a cron job :)20:15
natefinchericsnow: cron job to delete garbage in tmp?20:15
* natefinch hi5's eric20:15
* ericsnow hi5's natefinch20:16
redirwhelp, now we know why it wasn't picking up new PRs20:16
redirthere's the actual PR https://github.com/juju/juju/pull/535820:17
redirnot a lot of there, there.20:17
natefinchoh man, json's policy of not allowing commas on the last item of a list is horrible20:19
natefinchredir: it's just an addition to somethnig in export_test?20:24
redirnatefinch: yep, so that tests on arm64 and ppc64el can find appropriate images in test runs20:24
natefinchahh, wacky20:25
redirprobably something I should have done in the lts updates, but...I didn't know.20:25
natefinchLGTM20:25
redirwell tools20:26
redirwe by default build amd64 tools for supported series and then whatever the host arch is.20:26
redirbut there were not images for in the exported index for those host arches20:26
redirnatefinch: tx.20:26
redir$$merge$$ing20:26
redirran the failing tests on the respective arches and it passed on both.20:27
natefinchcool20:30
* redir steps out for a bite20:37
redirnow redir really steps out.20:49
=== alexisb is now known as alexisb-afk
redirwhat is 08d:|?21:27
katcoericsnow: whoops meant to ping you here21:36
katcoericsnow: hm. review board not picking up my latest commit to a PR. suggestions?21:36
redirkatco: seems it is  out of disk21:38
katcoredir: how are you discovering this?21:38
redirI made a post manually from rbtools21:38
katcoredir: ah21:38
redirthen http://reviews.vapour.ws/r/4785/diff/#21:38
redirwell nate noted it above when he tried to review21:39
katcoredir: ah21:40
ericsnowkatco: looking into it21:41
katcoericsnow: ta21:41
ericsnowkatco: having trouble connecting to the juju environment21:41
redirmust be friday22:37
redirThe last blocker is windows.22:37
=== frankban is now known as frankban|afk

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