ericsnowperrito666: it's buried away in the connection code in provider/gce/google00:00
perrito666k tx :) ill dig it up00:00
ericsnowperrito666: k00:00
ericsnowperrito666: gotta run; catch me in the morning if you have more questions00:01
perrito666tx a lot ericsnow00:01
ericsnowperrito666: np, good luck :)00:01
marcoceppiericsnow: ping on https://github.com/juju/juju/pull/304401:37
perrito666marcoceppi: eric left a bit over an hour ago01:42
menn0master is unblocked again people04:10
anastasiamacmenn0: tyvm :D04:12
mupBug #1486675 changed: supportedSeriesWindowsSuite.TestSupportedSeries fails <blocker> <ci> <regression> <unit-tests> <windows> <juju-core:Fix Released by bteleaga> <https://launchpad.net/bugs/1486675>04:14
mupBug #1486675 opened: supportedSeriesWindowsSuite.TestSupportedSeries fails <blocker> <ci> <regression> <unit-tests> <windows> <juju-core:Fix Released by bteleaga> <https://launchpad.net/bugs/1486675>04:23
mupBug #1486675 changed: supportedSeriesWindowsSuite.TestSupportedSeries fails <blocker> <ci> <regression> <unit-tests> <windows> <juju-core:Fix Released by bteleaga> <https://launchpad.net/bugs/1486675>04:32
dooferladdimitern / TheMue: please take a look: http://reviews.vapour.ws/r/2423/07:39
dimiterndooferlad, sure, looking07:40
dimiterndooferlad, so you went with returning an error with --networks given and these are the only tests that failed?07:42
dooferladdimitern: yeo07:42
dimiterndooferlad, good!07:43
* dooferlad tries testing juju with go 1.5 for fun07:45
dimiterndooferlad, reviewed07:47
dimiterndooferlad, btw do you have a windows machine around that you can try running juju unit tests on?07:47
dooferladdimitern: not really. I have my gaming box, but I haven't got it set up for development07:48
dooferladdimitern: https://aws.amazon.com/windows/ ?07:48
dimiterndooferlad, it's fine then, np07:49
dimiterndooferlad, after the yesterday's sync up with master net-cli *almost* got blessed :)07:49
dimiterndooferlad, a couple of tests remaining, one of which is windows specific and "native" to net-cli - bug 146980707:50
mupBug #1469807: ListSuite.TestOutputFormats fails on Windows <ci> <regression> <test-failure> <unit-tests> <windows> <juju-core:Incomplete> <juju-core net-cli:Triaged> <https://launchpad.net/bugs/1469807>07:50
dimiterndooferlad, we need to sort this out before we can merge net-cli into master07:51
mattywcmars, tasdomas wallyworld https://github.com/wallyworld/juju/pull/2908:00
cmarsmattyw, tasdomas fwereade wallyworld PTAL, http://reviews.vapour.ws/r/2432/08:13
dooferladdimitern: are you sure you want me to remove the SupportsNetworking check in deploy.go?08:26
dooferladdimitern: do we make sure that networking is supported somewhere else?08:26
dimiterndooferlad, since we'll be using constraints, it's fine to drop that08:31
dimiterndooferlad, because constraints are not guaranteed to be supported and are verified at provisioning time anyway08:31
dooferladdimitern: thanks. Pushing changes now and will $$merge$$08:32
TheMuedimitern: I yesterday fixed the comments for my latest branch, only go logical trouble with the test of the FinishedWorker. the fact, that the runner stops a worker after Wait() returned nil is tested elsewhere, so due to the simpleness of the FW we need only to assure that it implements worker (simple static assignment) and if Wait() returns nil08:34
TheMuedimitern: so I would like to drop the one I added and create a new real finishedworker_test.go08:35
* dooferlad thinks that localServerSuite.TestNetworkInterfaces has a race08:35
* TheMue fetches his chequered flag08:36
dooferladdimitern: have you seen this? http://pastebin.ubuntu.com/12134188/08:46
dimiternTheMue, let's discuss it during standup?08:52
dimiterndooferlad, looking08:52
TheMuedimitern: sure08:52
dimiterndooferlad, yes, but it's hard to reproduce on my machine08:53
dooferladdimitern: I think I have seen the problem, but I wanted to check I wasn't duplicating work08:53
dimiterndooferlad, there's a bug about it08:53
TheMuedimitern: could you point me again to the test you mentioned?09:31
dimiternTheMue, have a look how newWorkerStarter() is used09:39
TheMuedimitern: do you mean newTestWorkerStarter()?09:42
dimiternTheMue, yes, that's the one09:43
TheMuedimitern: yeah, it works with the newTestWorkerStarter, which works with the testWorker which really has a goroutine in the backend. we don't have that.09:45
TheMuedimitern: IMHO TestOneWorkerFinish() tests that any worker finishes if Wait() returns nil09:47
TheMuedimitern: opposite to TestOneWorkerRestart()09:48
TheMuedimitern: but here this is achieved by telling the goroutine to end with err == nil09:49
dimiternTheMue, ok, then no need to test the finishedworker in runner_test?09:57
TheMuedimitern: that's what I meant, yes. only one that says "yes, I'm a Worker" and "yes, I return nil on Wait()".09:58
dimiternTheMue, just check newAddresser in cmd/jujud/agent/machine.go returns FinishedWorker{} when disabled?09:58
TheMuedimitern: that's in the worker_test right now, but can see how to do it in machine_test too10:00
dimiternTheMue, in machine_test, I'd suggest saving the original agent.NewAddresser value (from export_test), then patching it with another which calls it internally and asserts the result is finishedworker10:02
TheMuedimitern: ok10:02
TheMueall: core team meeting?10:03
dimiternI'll skip it this time I'm afraid10:04
mattywwallyworld, axw is this basically what you wanted? https://github.com/wallyworld/juju/pull/31/files#diff-abee59b3dd781e907de49c2651b6a332R3210:04
perrito666hey people I need to take my wife to work I cant make it to the meeting sorry10:09
dooferladdimitern: goamz has changed a lot :-| at least I found a real bug in juju (TestNetworkInterfaces relied on map ordering)11:07
dooferladdimitern: ah, go-amz, not goamz11:08
dimiterndooferlad, yeah11:29
dimiterndooferlad, what's the ordering bug about?11:29
mupBug #1486965 opened: Destroying the leader sets the service status to terminated <juju-core:New> <https://launchpad.net/bugs/1486965>11:42
mupBug #1486965 changed: Destroying the leader sets the service status to terminated <juju-core:New> <https://launchpad.net/bugs/1486965>11:54
mupBug #1486965 opened: Destroying the leader sets the service status to terminated <juju-core:New> <https://launchpad.net/bugs/1486965>12:00
dooferladdimitern: that is https://github.com/juju/juju/pull/305212:21
dooferladdimitern: no idea why I didn't create a pull request before lunch12:21
dooferladit was a map ordering problem12:22
dooferladhttps://github.com/go-amz/amz/pull/61 is about thread safety12:22
dooferladand now, windows!12:22
dimiterndooferlad, awesome! looking12:22
dimiterndooferlad, both approved and the goamz one merged12:30
dooferladdimitern: thanks!12:30
fwereadetasdomas, cmars: I think we should talk about uniter availability again, I don't quite get how that last branch solves the problem12:40
tasdomascmars, fwereade sure12:40
wallyworldfwereade: https://github.com/wallyworld/juju/pull/3212:56
wallyworldaxw: https://github.com/wallyworld/juju/pull/3212:56
fwereadedooferlad, TheMue: dimitern is battered by storms and without power13:14
dooferladfwereade: thanks for the update. Yikes!13:14
perrito666yet he manages to communicate with the guy in an island :p13:18
fwereadeperrito666, and whose own power is out again, the house is being rewired :)13:20
mattywwallyworld, could you take a look at https://github.com/wallyworld/juju/pull/31 and let me know if it seems reasonable?13:21
mattywfwereade, you might be interested as well ^^13:21
frankbanericsnow: when you ahev time, could you please take a look at https://github.com/juju/juju/pull/3035 ? it's a mp for a long-running branch that we are using to implement the embedded gui story. thanks!13:28
bachi i've got a juju-core pull request on github and reviewboard.  with a 'shipit' vote from davecheney.  does it require a second? when ready to merge, do you just press the 'shipit' button on reviewboard?13:29
abentleysinzui: Want to talk about weekly test candidates?13:32
sinzuiabentley: indeed. have you been listening to my thoughts this morning?13:33
wwitzel3ericsnow: ping13:33
abentleysinzui: Err, not intentionally?13:35
sinzuidooferlad: or anyone working on net-cli, ping who when bug 1469807 has a fix merged. I want to enure we test it13:35
mupBug #1469807: ListSuite.TestOutputFormats fails on Windows <ci> <regression> <test-failure> <unit-tests> <windows> <juju-core:Incomplete> <juju-core net-cli:Triaged> <https://launchpad.net/bugs/1469807>13:35
dooferladsinzui: I am looking at it now. Will let you know13:37
dooferladTheMue: Could you take a look at https://github.com/juju/cmd/commit/4a5e8abce36462f89d92d28bd69b54c7845db818 please?13:48
dooferladTheMue: I pushed to the wrong branch, but I have tested that this works :-(13:49
TheMuedooferlad: yep13:56
TheMuedooferlad: a direct usage of target with the defer of it's closing isn't possible?13:59
dooferladTheMue: no, because the type of target doesn't have a Close13:59
dooferladbut File does13:59
TheMuedooferlad: ah, and the assigning is a kind of downcast, yes13:59
TheMuedooferlad: ok13:59
katconatefinch: standup?14:04
natefinchkatco: oops, coming14:05
ericsnowfrankban: yeah, I'll take a look14:06
frankbanericsnow: ty!14:07
TheMueAh, my last strange tests are now working too, phew.14:07
mattywwallyworld, another one https://github.com/wallyworld/juju/pull/3414:07
TheMueGrmpf, still an issue. Those two tests running together by pattern work, also isolated, but inside the whole suite they don't work.14:21
TheMueError is "connection is shut down" when using the api in the MachineSuite of jujud/agent/machine_test.go14:24
dooferladTheMue: http://reviews.vapour.ws/r/2437/14:29
TheMuedooferlad: *click*14:30
dooferladJust a rev bump to bring in the change I just made to cmd/juju14:30
TheMuedooferlad: done14:30
dooferladTheMue: thanks!14:31
=== xwwt_ is now known as xwwt
TheMuedooferlad: yw14:31
TheMuedooferlad: I'm still looking for my strange test failures. normally one test shouldn't effect another one. my both work when called single or together, but with the whole suite one fails14:32
dooferladTheMue: timeouts?14:32
dooferladTheMue: races?14:33
TheMuedooferlad: connection of the API14:33
TheMuedooferlad: here I have an idea14:33
TheMuedooferlad: I exchange a function variable, so this has to be done atomically14:34
ericsnowcmars: you have a patch up to run the feature tests only upon request?14:54
cmarsericsnow, correct. though, that will be set by default in jenkins14:54
cmarsericsnow, so it is on request in your dev env14:54
ericsnowcmars: a big +1 from me, but why the change from opt-out?14:55
ericsnowcmars: BTW, this might have helped :/  http://reviews.vapour.ws/r/1647/14:56
cmarsericsnow, unit tests are appropriate for development, integration tests are appropriate for CI14:56
ericsnowcmars: totally agree; perhaps I'm assuming too much to think there was a rationale for opt-out originally14:56
ericsnowcmars: PTAL @ my review on http://reviews.vapour.ws/r/2432/15:02
TheMuedooferlad: seen the mail? do you know the branches?15:22
dooferladTheMue: just replied15:22
dooferladTheMue: no, there is nothing on his github15:22
TheMuedooferlad: just seen your reply15:23
dooferladTheMue: I assume he is working on https://canonical.leankit.com/Boards/View/101652562/116729781 and maybe https://canonical.leankit.com/Boards/View/101652562/11665276015:24
TheMuedooferlad: yeah, and I don't see branches for it in his account15:26
TheMuedooferlad: but at least I found my failure, a misunderstanding of the helpers of the test suite15:58
marcoceppiericsnow: ping16:20
ericsnowmarcoceppi: hey16:21
marcoceppiericsnow: I had some questions on https://github.com/juju/juju/pull/304416:21
ericsnowmarcoceppi: yep16:21
marcoceppiericsnow: > for clarification does this mean that if I have a script called `juju-foo` in my path `juju foo` will no longer work?16:22
katcomarcoceppi: no different concept of plugins16:22
marcoceppicool, thanks16:22
katcomarcoceppi: this has nothing to do with juju plug-ins16:22
marcoceppijust making sure16:22
* marcoceppi puts all the red flags away16:22
dimiterndooferlad, TheMue, o/16:23
ericsnowmarcoceppi: yeah, what katco said :)16:24
dooferladdimitern: o/16:24
natefinchmarcoceppi: we are disabling bundles and the config-changed hook, though16:25
dimiterndooferlad, so I need to merge your changes from net-cli into my branch and push it16:25
ericsnowmarcoceppi: I should have made it more clear that this relates to the workload processes feature16:25
dooferladdimitern: ok, I will look out for a review request16:26
dooferladdimitern: will let you get on with it for now. Ping me if there is anything you want. I am idling/hangout out with the daughter (first steps!) until given somethine else to do.16:27
dimiterndooferlad, cheers, I'll try to be quick16:27
dooferlad(since it is EOD)16:27
dimiterndooferlad, :) ok, sounds good16:27
dooferladI will see what I can do this evening once she is in bed.16:28
dooferlad(if anything is needed)16:28
marcoceppinatefinch: wait...what16:50
natefinchmarcoceppi: lol16:52
natefinchmarcoceppi: just messing with you16:52
natefinchericsnow: you around?17:33
ericsnownatefinch: yep17:33
natefinchericsnow: have you tried the process removal code?  I call it and returns no errors and... then the process still exists.  Maybe I'm missing something somewhere17:35
ericsnownatefinch: there are tests, but we certainly may have missed something between the layers17:36
natefinchcannot for the life of me get log level = trace to work17:42
TheMuedimitern: ah, back online. I'm idling here too for any task. latest branch is ready for 2nd review (after fighting with testing troubles this afternoon)17:50
dimiternTheMue, cool, I'll have a look then17:51
wwitzel3cmars: ping17:51
dimiternI'm ready with mine as well - all tests pass, now need to merge17:51
dimiternTheMue, reviewed18:06
TheMuedimitern: thx, will take a look in a few moments18:24
natefinchgah, rolling over a 401k is such an outdated process... mailing paperwork and then mailing paper checks... what is this, 1970?18:29
* natefinch is going to receive a check in the mail worth approximately half the cost of his house :/18:29
natefinchold brokerage: "But we're sending it priority mail, so we can track it."  Fantastic.18:30
dimiterndooferlad, TheMue, if any of you are still around I'd appreciate a quick look on this; it's quite big, but it's also well tested, including a feature test which confirms CLI works end-to-end to state for spaces list|create18:39
dooferladdimitern: can do18:39
dimiterndooferlad, TheMue, tyvm, http://reviews.vapour.ws/r/2440/18:41
dooferladdimitern: I almost had several comments, but it looks good!18:58
dimiterndooferlad, awesome!18:58
dimiterndooferlad, I have a similar one for subnets CLI, but it's already very late18:59
dooferladdimitern: up to you. I can come back after I have eaten18:59
dimiterndooferlad, well, if I'm not done within the next half hour with it, it won't happen, but there are a few tests still left19:00
TheMuedimitern: reviewd19:16
dimiternTheMue, thanks!19:18
katcoericsnow: hey going to be just a little late19:58
ericsnowkatco: no worries19:59
mupBug #1487191 opened: TestAddOkay mismatch <ci> <regression> <unit-tests> <juju-core:Incomplete> <juju-core feature-proc-mgmt:Triaged> <https://launchpad.net/bugs/1487191>20:16
mupBug #1487191 changed: TestAddOkay mismatch <ci> <regression> <unit-tests> <juju-core:Incomplete> <juju-core feature-proc-mgmt:Triaged> <https://launchpad.net/bugs/1487191>20:25
bogdanteleagadid the procedure for using development tools change? I can't seem to be able to bootstrap anymore, it complains about tools not found20:35
mupBug #1487191 opened: TestAddOkay mismatch <ci> <regression> <unit-tests> <juju-core:Incomplete> <juju-core feature-proc-mgmt:Triaged> <https://launchpad.net/bugs/1487191>20:37
cheryljbogdanteleaga: have you tried using --upload-tools?20:46
bogdanteleaganope, I have them uploaded somewhere and I'm using agent-metadata-url20:47
bogdanteleagait was a permissions issue though I fixed it now20:48
cheryljbogdanteleaga: ah, cool20:48
bogdanteleagathough, it still seems like it defaults to devel even though I tried specifying releases20:48
cheryljbogdanteleaga: iirc, wallyworld was doing some work around that, but I'm not sure where things landed.20:49
bogdanteleagacherylj, I found this https://github.com/juju/juju/pull/298520:49
bogdanteleagaI'll try again soon, it may have tried to read devel anyway for no reason and fail like that20:50
cheryljanyone else having problems with reviewboard not creating review requests for PRs?22:03
waiganicherylj: I just put one up, worked for me22:30
thumperabentley: call?23:07
abentleywaigani, thumper: Sure.  I'm in https://plus.google.com/hangouts/_/canonical.com/jes-feature?authuser=123:08
thumperabentley: we are in the one defined in the calendar23:08

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