| davecheney | sinzui: which version of Go does the win2k12 builder use ? | 00:32 |
|---|---|---|
| sinzui | davecheney: It does build. The canonistack-slave builds windows and centos with go 1.6 and has done since the announcement on April 10 | 00:37 |
| davecheney | sinzui: which version of Go does the win 2k12 builder use ? | 01:00 |
| davecheney | menn0: https://github.com/juju/juju/pull/5388/files | 01:00 |
| sinzui | davecheney: what ships with trusty | 01:01 |
| davecheney | ^ quick review to help debug 1581157 | 01:01 |
| mup | Bug #1581157: github.com/juju/juju/cmd/jujud test timeout on widnows <blocker> <ci> <regression> <test-failure> <unit-tests> <windows> <juju-core:Triaged by dave-cheney> <https://launchpad.net/bugs/1581157> | 01:01 |
| davecheney | sinzui: i'm sorry I don't understand, i'm askig about the windwso builder | 01:01 |
| davecheney | sinzui: http://reports.vapour.ws/releases/3964/job/run-unit-tests-win2012-amd64/attempt/2384#highlight | 01:01 |
| davecheney | which version of Go was used on this CI run ? | 01:01 |
| sinzui | davecheney: IS will not support us using a windows builder so we dont. we cross-compile from a trusty host. the ALL CI builders use the golang-1.6 package. No other is accepted. Trusty currently has 1.6-0ubuntu1~14.04 | 01:03 |
| davecheney | thanks, i undrstand now | 01:04 |
| mwhudson | can someone point me to the code that launches mongo during a bootstrap? | 01:05 |
| sinzui | davecheney: PS, the cross-compile setup is your own script. No one has matched your script for ease of use | 01:05 |
| mwhudson | (manual provider, if that matters) | 01:05 |
| davecheney | a script ? you don't need a script for go 1.6 | 01:19 |
| davecheney | GOOS=windows GOARCH=amd64 go ... | 01:19 |
| davecheney | is all you need | 01:19 |
| cherylj | mwhudson: cmd/jujud/bootstrap.go is what initially calls out to start mongo. | 01:21 |
| mwhudson | cherylj: i think i've found it now, thanks | 01:22 |
| mwhudson | ubuntu@autopkgtest:~$ journalctl -u juju-db | head | 01:22 |
| mwhudson | -- Logs begin at Fri 2016-05-13 12:51:41 NZST, end at Fri 2016-05-13 13:20:58 NZST. -- | 01:22 |
| mwhudson | May 13 12:57:46 autopkgtest systemd[1]: Started juju state database. | 01:22 |
| mwhudson | May 13 12:57:46 autopkgtest mongod[2960]: 2016-05-13T12:57:46.932+1200 W CONTROL [main] No SSL certificate validation can be performed since no CA file has been provided; please specify an sslCAFile parameter | 01:22 |
| mwhudson | May 13 12:57:47 autopkgtest mongod[2960]: Invalid command: ignored | 01:22 |
| mwhudson | ^ looks bad | 01:22 |
| cherylj | :/ | 01:22 |
| mwhudson | and yeah, hacking at the service file makes it go | 01:23 |
| mwhudson | which is a bit eeeeeeeerrrrrrrrrm | 01:24 |
| mwhudson | ooo kkaaay | 01:31 |
| mwhudson | so mongo is running with this command line: | 01:31 |
| mwhudson | /usr/lib/juju/mongo3.2/bin/mongod --dbpath /var/lib/juju/db --sslOnNormalPorts --sslPEMKeyFile /var/lib/juju/server.pem --port 37017 --syslog --journal --replSet juju --ipv6 --quiet --oplogSize 1009 --auth --keyFile /var/lib/juju/shared-secret --storageEngine wiredTiger | 01:31 |
| mwhudson | but if i run that same command it says | 01:31 |
| mwhudson | 2016-05-13T13:31:41.359+1200 W CONTROL [main] No SSL certificate validation can be performed since no CA file has been provided; please specify an sslCAFile parameter | 01:31 |
| mwhudson | help? | 01:32 |
| mwhudson | oh i see it always says that | 01:34 |
| menn0 | davecheney: looking now. was lunching | 01:34 |
| mwhudson | well wtf | 01:36 |
| menn0 | davecheney: ship it | 01:36 |
| mwhudson | apparently you have to say --sslPEMKeyPassword=ignored now, and --sslPEMKeyPassword ignored doesn't work | 01:36 |
| wallyworld | axw: be right there | 02:00 |
| redir_ | axw: wallyworld: here's http://reviews.vapour.ws/r/4822 FWIW. I'll look back in later, but don't think there' much more I can do tonight. | 02:05 |
| wallyworld | redir_: np, ty | 02:05 |
| wallyworld | have a good evening | 02:05 |
| redir_ | and https://github.com/fatih/vim-go/issues/814 is what happened to my editor. Syntastic stopped working... | 02:08 |
| === rodlogic is now known as Guest75713 | ||
| mwhudson | whisky tango foxtrot: https://bugs.launchpad.net/ubuntu/+source/juju-mongodb3.2/+bug/1581284 | 02:17 |
| mup | Bug #1581284: --sslPEMKeyPassword parameter must be passed with = on yakkety <juju-mongodb3.2 (Ubuntu):New> <https://launchpad.net/bugs/1581284> | 02:17 |
| natefinch | mwhudson: welcome to the joys of working with mongo | 02:18 |
| === rodlogic is now known as Guest21818 | ||
| wallyworld | davecheney: i'm not sure about the approach used here http://reviews.vapour.ws/r/4822, if you get a chance, you you take a look to givr your opinion? | 02:50 |
| marcoceppi | o/ wallyworld I just realized charmsvg was uploaded to the store before resources were required/allowed. I'm about to patcha few things in the app and will upload again with resources | 02:50 |
| wallyworld | marcoceppi: ok, thank you | 02:51 |
| wallyworld | marcoceppi: and you'll load the resources to the store also? | 02:51 |
| marcoceppi | wallyworld: I have no choice ;) | 02:51 |
| wallyworld | duh | 02:52 |
| marcoceppi | wallyworld: if you want a really fun demo, you could have an old copy of python-jujusvg on your disk and "upgrade" the resource. I'll start rendering bundles with squircles | 02:52 |
| wallyworld | ooooh, that sounds kinda cool actually | 02:52 |
| mwhudson | https://github.com/juju/juju/pull/5389 <- easy review | 02:57 |
| wallyworld | mwhudson: lgtm. it is really only that one arg that needs the =? I guess is for on/off values? | 03:00 |
| mwhudson | wallyworld: there are about three other ones that would need it i think but we don't pass any of them | 03:01 |
| wallyworld | mwhudson: maybe add a comment to the code? | 03:01 |
| mwhudson | wallyworld: i guess it can't land because master is hilariously blocked? | 03:01 |
| wallyworld | yeah, we are about to do a beta7 even though not all races are done. should be unblocked real soon | 03:02 |
| wallyworld | mwhudson: btw, using your go 1.7 ppa. can't belive how farking slow the 1.6 compiler is | 03:02 |
| wallyworld | 1.7 is a *bit* better | 03:02 |
| mwhudson | wallyworld: oh god eyah | 03:02 |
| wallyworld | wtf went wrong? | 03:02 |
| wallyworld | why did it get so slow? | 03:03 |
| mwhudson | the conversion to go mostly | 03:03 |
| mwhudson | but the 1.7 *linker* is faster than the 1.4 linker now | 03:03 |
| wallyworld | so Go is a worse compiler than C? | 03:03 |
| mwhudson | (the compiler is still slower) | 03:03 |
| mwhudson | maybe partly that, but also C mechanically translated to Go is probably not optimal Go | 03:04 |
| mwhudson | the go compiler is also better in 1.7, so that helps | 03:04 |
| wallyworld | at least they are dog fooding now | 03:05 |
| mwhudson | wallyworld: i added a comment | 03:05 |
| wallyworld | ty | 03:05 |
| natefinch | the explanation I heard is that the compiler is kind of a bad use case for garbage collection, it produces tons of objects and throws them away. if you set GOGC=800, I think it's like 20% faster, but it'll use gobs of memory | 03:06 |
| mwhudson | yeah, a lot of work went into making the compiler a more natural Go program this cycle | 03:06 |
| mwhudson | natefinch: that's part of it | 03:06 |
| mwhudson | natefinch: although not quite right, the problem is that it allocates tons of objects and they *never become garbage* | 03:06 |
| natefinch | ahh | 03:06 |
| mwhudson | so the gc is just spinning cpu and not acheiving anything | 03:06 |
| mwhudson | (some stuff becomes garbage of course, but relatively little) | 03:07 |
| mwhudson | wallyworld: so when can this land? when master is unblocked, but do i have to pay attention and notice when that's happened? | 03:08 |
| wallyworld | mwhudson: sadly you need to pay attention. but there's normally an email to juju-dev | 03:09 |
| wallyworld | mwhudson: is it holding up anything urgent? | 03:09 |
| mwhudson | wallyworld: it's holding up the migration from -proposed to -release in yakkety | 03:10 |
| mwhudson | wallyworld: but i can smuggle it in as a distro patch there | 03:10 |
| wallyworld | mwhudson: ok, in that case - add the blocker tag to the bug | 03:10 |
| wallyworld | and land with $$fixes-xxxxx$$ | 03:10 |
| wallyworld | where xxxx is the bug number | 03:10 |
| wallyworld | mwhudson: so long as you don't break xenial builds | 03:11 |
| mwhudson | wallyworld: not sure i have permission to do that? | 03:11 |
| mwhudson | it's bug 1581284 fwiw | 03:12 |
| mup | Bug #1581284: --sslPEMKeyPassword parameter must be passed with = on yakkety <blocker> <juju-core:New> <juju-core (Ubuntu):New> <juju-mongodb3.2 (Ubuntu):New> <https://launchpad.net/bugs/1581284> | 03:12 |
| wallyworld | it's already marked as a blocker | 03:12 |
| mwhudson | i just did that now | 03:12 |
| mup | Bug #1581284 opened: --sslPEMKeyPassword parameter must be passed with = on yakkety <blocker> <juju-core:New> <juju-core (Ubuntu):New> <juju-mongodb3.2 (Ubuntu):New> <https://launchpad.net/bugs/1581284> | 03:12 |
| mwhudson | but i don't think i can do the $$fixes-$$ thing? | 03:12 |
| wallyworld | you can | 03:12 |
| mwhudson | maybe i can, let's find out i guess | 03:12 |
| wallyworld | if you can $$merge$$ you can | 03:13 |
| mwhudson | i don't think i can $$merge$$... | 03:13 |
| wallyworld | ok, i'll do it, we'll have to fix those permissions | 03:13 |
| wallyworld | mwhudson: i just added you, can you try to merge? | 03:15 |
| mwhudson | wallyworld: so i just comment "$$fixes-1581284$$"? | 03:15 |
| wallyworld | yup | 03:15 |
| mwhudson | wallyworld: i guess i should file another bug about how utterly unobvious it all is when the juju-db systemd unit fails to start properly... | 03:20 |
| wallyworld | that would be good | 03:20 |
| wallyworld | our observability in that area is shite | 03:20 |
| * mwhudson can only agree | 03:23 | |
| mwhudson | particularly awesome is the fact that after the bootstrap fails, /etc/whatever/juju-db.service is a dangling symlink | 03:24 |
| mwhudson | wallyworld: huh i guess the bot hadn't noticed the tag yet? | 03:28 |
| wallyworld | mwhudson: it didn't like your tag for some reason. it acted on mine, but hasn't picked up that that bug number is valid | 03:29 |
| mwhudson | hmm | 03:30 |
| wallyworld | ah, needs to be crtical | 03:30 |
| wallyworld | i think, i'll try that | 03:31 |
| mup | Bug #1581297 opened: confusing failure mode if juju-db systemd unit fails to start up <juju-core:New> <https://launchpad.net/bugs/1581297> | 03:36 |
| mwhudson | oh probably | 03:36 |
| mwhudson | https://launchpad.net/ubuntu/+source/juju-core/2.0~beta6-0ubuntu2.16.10.2 fwiw | 03:37 |
| wallyworld | mwhudson: in case you didn't notice, your fix landed | 04:23 |
| axw | wallyworld: it occurred to me that we can manage ZFS from within the controller with the LXD provider, since the controller can manage its own config | 05:34 |
| axw | wallyworld: so we could have a ZFS storage provider | 05:34 |
| wallyworld | hmmmm | 05:34 |
| axw | or LVM | 05:34 |
| wallyworld | interesting | 05:34 |
| === blahdeblah_ is now known as blahdeblah | ||
| davecheney | natefinch https://bugs.launchpad.net/juju-core/+bug/1581337 | 06:44 |
| mup | Bug #1581337: natefinch/npipe/v2: Closing a pipe is not safe in a threaded environment <juju-core:New> <https://launchpad.net/bugs/1581337> | 06:44 |
| mup | Bug #1581337 opened: natefinch/npipe/v2: Closing a pipe is not safe in a threaded environment <juju-core:New> <https://launchpad.net/bugs/1581337> | 07:04 |
| === frankban|afk is now known as frankban | ||
| voidspace | dimitern: ping | 08:53 |
| dimitern | voidspace: pong | 08:53 |
| voidspace | dimitern: I've sent you a question about transactions in private chat | 08:57 |
| babbageclunk | voidspace: 'cause it had swears in it. | 08:59 |
| dimitern | voidspace: yeah, I'm replying | 08:59 |
| voidspace | babbageclunk: I'm quite happy swearing here | 09:02 |
| Mmike | Hi, lads! Which mongod version is juju 2.0 going to use? | 09:12 |
| dimitern | Mmike: it's using 3.0 now, but efforts are made to make it use 3.2 before the release | 09:13 |
| Mmike | dimitern: ack! dimitern is it using the default storage engine for used version? mmap in 3.0 and Tigersomething in 3.2 ? | 09:14 |
| dimitern | Mmike: correct (IIRC) - mmap (..something) in 3.0 and wiredtiger in 3.2 | 09:15 |
| Mmike | dimitern++ ack, thnx | 09:15 |
| fwereade | dimitern, pushed tests for http://reviews.vapour.ws/r/4796/ -- not *super* happy with them, just because there's so much *to* test, would very much appreciate your opinion | 09:28 |
| dimitern | fwereade: thanks, will have a look | 09:30 |
| fwereade | dimitern, ...no rush though | 09:30 |
| fwereade | dimitern, maybe best wait, they could use some docs and I should address your comments on the rpeviosu version too | 09:30 |
| dimitern | fwereade: ok, won't be able to get to it for a few hours anyway I'm afraid | 09:32 |
| fwereade | dimitern, perfect ;p | 09:33 |
| fwereade | dimitern, btw, re Annotate/Annotatef, do you feel particularly strongly? | 09:54 |
| fwereade | dimitern, the way I see it Annotatef-everywhere is easier to tweak and has no real disadvantages | 09:55 |
| hoenir | it is stable enough to upgrade to 16.04 lts ubuntu server with maas2.0 for juju? | 10:02 |
| dimitern | fwereade: not strongly, and you have a point - we should likely replace Annotate with Annotatef across the codebase | 10:03 |
| mattyw | fwereade, dimitern either of you know of a juju client api that I can use to get the CharmURL of a given unit?, there appears to be one for services, I'd like to find one for units | 10:06 |
| tasdomas | mattyw, isn't the charm url a service level property? | 10:07 |
| tasdomas | mattyw, i.e. if you know the unit's service, the charm url will be that of the service | 10:07 |
| dimitern | mattyw: it's been a while since I was in those parts of the API I'm afraid | 10:08 |
| mattyw | tasdomas, correct, I could make a number of calls to get the answer, I'm interested to know if there's a single one that will do all the work I want | 10:08 |
| dimitern | tasdomas, mattyw: the service's charm url might be different during upgrades | 10:08 |
| mattyw | dimitern, ok no problem - I pinged because I saw you were around | 10:08 |
| mattyw | dimitern, yeah, I think that's ok for these circumstaces | 10:08 |
| fwereade | mattyw, tasdomas: a unit's charm url should only set once the charm has been downloaded and verified, iirc -- I don't think there's a specific external api for it, but I'm pretty sure it's available via status | 10:16 |
| fwereade | dimitern, cheers | 10:17 |
| mattyw | fwereade, ok thanks | 10:17 |
| fwereade | mattyw, tasdomas: (also applies when upgrading -- in general, a new value will be set when the charm has overcome all remote hurdles to installing a new charm) | 10:22 |
| fwereade | dimitern, hey, Worker should be called Mortal | 10:27 |
| * fwereade wonders if there's a clean anglo-saxon word for waiting until something dies | 10:27 | |
| dimitern | fwereade: lol why not? | 10:27 |
| fwereade | dimitern, I think it's actually closer | 10:28 |
| fwereade | dimitern, I just replied pointing out that "any type of task" isn't quite right because all worker *actually* signifies is something whose validity is bounded in time | 10:29 |
| dimitern | fwereade: Ephemeral ? | 10:30 |
| fwereade | dimitern, Mortal.Kill > Ephemeral.Dissipate, or whatever it'd have to be for the metaphor to be consistent | 10:31 |
| dimitern | :D | 10:31 |
| dimitern | .Transmigrate() | 10:31 |
| fwereade | haha | 10:31 |
| fwereade | dimitern, and it goes a lot better with tomb and catacomb ;) | 10:33 |
| dimitern | fwereade: yeah | 10:34 |
| fwereade | dimitern, (hmm, also, mortal.Reap vs worker.Stop) | 10:35 |
| dimitern | fwereade: I love how you always strive to find meaningful and unambiguous names, which (unlike me) are also usually nice and short | 10:35 |
| * fwereade had to fight himself not to call RestartWorkers a CruciformWorkers, on the basis that dan simmons references are probably a bit obscure | 10:36 | |
| fwereade | dimitern, ty :) | 10:36 |
| babbageclunk | fwereade - vigil? | 10:36 |
| fwereade | babbageclunk, nice! | 10:36 |
| babbageclunk | fwereade: Ha, that would have been great. | 10:36 |
| babbageclunk | fwereade: although vigil isn't really specific to waiting for someone to die, so maybe clear enough | 10:37 |
| fwereade | babbageclunk, it's the best yet for sure | 10:37 |
| fwereade | babbageclunk, I was mostly failing to come up with vulture metaphors | 10:38 |
| babbageclunk | fwereade: eww | 10:38 |
| dimitern | frobware, voidspace, dooferlad, babbageclunk: http://reviews.vapour.ws/r/4824/ fixes bug 1580417; please take a look (at least some of you :) | 11:15 |
| mup | Bug #1580417: Allow a different VPC ID to be used for hosted models <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1580417> | 11:15 |
| frobware | dimitern: need to finish up on my PR, then will take a look. Might be a little while. | 11:16 |
| === rodlogic is now known as Guest45407 | ||
| dimitern | frobware: no worries, when you have time - thanks! | 11:17 |
| === rodlogic is now known as Guest44812 | ||
| === rodlogic is now known as Guest56723 | ||
| rogpeppe | anyone know where i can find a description of the configuration attributes accepted by the gce provider? | 13:40 |
| rogpeppe | juju generate-config used to be a thing but doesn't seem to exist any more | 13:40 |
| rogpeppe | and there don't seem to be any docs for >=2.0 in https://jujucharms.com/docs | 13:41 |
| mup | Bug #1580501 changed: cloudimg-base-url parameters not in Juju2 anymore <cpe-sa> <orangebox> <juju-core:Incomplete> <https://launchpad.net/bugs/1580501> | 13:42 |
| mup | Bug #1581537 opened: Can't destroy service -- juju status "failed" & unknown relation 0 error in log <juju-core:New> <https://launchpad.net/bugs/1581537> | 13:42 |
| mup | Bug #1540394 changed: Juju does not handle 429 "too many requests" from Azure <2.0-count> <azure-provider> <blocker> <bootstrap> <ci> <juju-core:Fix Released by axwalk> <https://launchpad.net/bugs/1540394> | 13:54 |
| mup | Bug #1579051 changed: Race in juju/controller/destroy and TestDestroyCommandConfirmation <blocker> <ci> <intermittent-failure> <race-condition> <regression> <juju-core:Fix Released by thumper> <https://launchpad.net/bugs/1579051> | 13:54 |
| marcoceppi | cherylj: o/ | 13:55 |
| cherylj | hey hey marcoceppi | 13:56 |
| marcoceppi | cherylj: we're doing some office hours in like 4 mins if you want to drop by | 13:56 |
| cherylj | I can't join the office hours party today :( | 13:56 |
| marcoceppi | aww, okay :) | 13:56 |
| cherylj | I'm stuck at the dealership waiting for my car to get fixed :/ | 13:56 |
| marcoceppi | bleh! anything I should pass along? beta7 teaser maybe? | 13:57 |
| cherylj | MAAS2 support | 13:57 |
| cherylj | no longer behind a feature flag | 13:57 |
| cherylj | and create-model will be renamed add-model | 13:57 |
| marcoceppi | \o/ | 13:57 |
| cherylj | we're hoping to release today, but we'll see how things go :( | 13:57 |
| mattyw | fwereade, ping? | 14:00 |
| dimitern | cherylj: hey, do you mean are we releasing 2.0 today or it's the next beta? | 14:01 |
| cherylj | next beta | 14:01 |
| cherylj | lol | 14:01 |
| cherylj | beta7 | 14:01 |
| dimitern | ah :) | 14:01 |
| cherylj | sorry :) | 14:02 |
| dimitern | ok, got worried for a sec.. | 14:02 |
| cherylj | yeah, that would be bad | 14:02 |
| fwereade | mattyw, pong | 14:03 |
| mattyw | fwereade, I'm doing some stuff that involves calling this api https://github.com/juju/juju/blob/master/api/service/client.go#L121 | 14:03 |
| mattyw | fwereade, after I do I start getting errors about there being an upgrade in progess | 14:04 |
| fwereade | mattyw, from that call? or from elsewhere? | 14:04 |
| mattyw | fwereade, I'm tracking down if there's a bug in some of our stuff, but basically that api shouldn't be responsible right? | 14:04 |
| mattyw | fwereade, I don't know, it's likely from somewhere else | 14:04 |
| fwereade | mattyw, yeah, I'm pretty sure it should not do that :) | 14:04 |
| mattyw | fwereade, ok cool thanks | 14:05 |
| dimitern | frobware, voidspace, dooferlad, babbageclunk: hate to be a pest guys, but can at least someone have a look at http://reviews.vapour.ws/r/4824/ ? | 14:08 |
| babbageclunk | dimitern: looking now, sorry. Will be a nice change from trying to disentangle this db creation and population stuff. | 14:10 |
| dimitern | frobware: I've found an issue with the lxd broker that causes e.g. http://reports.vapour.ws/releases/3971/job/functional-container-networking-lxd/attempt/160, testing a fix now | 14:10 |
| dimitern | babbageclunk: tyvm! | 14:11 |
| === rodlogic is now known as Guest81137 | ||
| frobware | dimitern: can we sync please | 14:26 |
| dimitern | frobware: ok | 14:26 |
| mup | Bug # changed: 1571917, 1571932, 1571933, 1574649, 1578456 | 14:27 |
| frobware | dimitern: 1:1 ho | 14:28 |
| dimitern | frobware: omw | 14:28 |
| tvansteenburgh | HOME=/nonexistent juju version | 14:33 |
| tvansteenburgh | error: cannot load ssh client keys: mkdir /nonexistent: permission denied | 14:33 |
| tvansteenburgh | this is unexpected ^ | 14:33 |
| tvansteenburgh | context: running tests on the lp buildfarm | 14:34 |
| tvansteenburgh | actually, with pbuilder at the moment | 14:35 |
| tvansteenburgh | well this works: HOME=$(mktemp -d) juju version | 14:39 |
| tvansteenburgh | so, workaround achieved maybe | 14:39 |
| === rodlogic is now known as Guest5666 | ||
| mup | Bug #1581579 opened: resource-get fails with duplicate key error <juju-core:New> <https://launchpad.net/bugs/1581579> | 15:33 |
| * dimitern really hates it when adding logging to an otherwise intermittently racy state method makes the race disappear | 15:39 | |
| cmars | hi, i seem to be unable to connect to the controller mongo database lately.. are there some instructions somewhere on how to do that? | 15:40 |
| cmars | thinking it might be related to the move to mongodb 3.2? | 15:40 |
| cmars | by connect, i mean with the mongo client, to poke around and debug things | 15:41 |
| dimitern | cmars: I wasn't able to since the upgrade to 3.2 | 15:41 |
| cmars | dimitern, ack, thanks for confirming. i'll email the list | 15:42 |
| === rodlogic is now known as Guest63438 | ||
| frobware | dimitern, voidspace, dooferlad, babbageclunk: if you have some cycles... http://reviews.vapour.ws/r/4826/ | 15:45 |
| dimitern | frobware: looking | 15:46 |
| redir_ | fwereade: yt? | 15:51 |
| babbageclunk | dimitern, dooferlad: halp! Why might my state.runTrasactions call be hanging? | 15:57 |
| babbageclunk | :( | 15:57 |
| dimitern | babbageclunk: hanging how? | 15:58 |
| dimitern | babbageclunk: what are you trying to do? | 15:58 |
| babbageclunk | dimitern: Yeah, that probably needs some context... | 15:58 |
| dimitern | :) | 15:58 |
| dimitern | don't panic ;) | 15:59 |
| babbageclunk | dimitern: I'm pulling apart the initialisation so that we don't recreate the collections and indexes for each test. | 15:59 |
| babbageclunk | But creating a model isn't just the schema, it's also some data in the collections. | 16:00 |
| babbageclunk | So now the teardown clears out the collections and puts that data back in. | 16:00 |
| dimitern | babbageclunk: right.. | 16:01 |
| babbageclunk | dimitern: So basically that's this bit: https://github.com/juju/juju/blob/master/state/open.go#L135 | 16:01 |
| babbageclunk | Pulled out into a separate function. | 16:02 |
| dimitern | babbageclunk: ok | 16:02 |
| babbageclunk | But when it gets to the st.runTransaction call (L175) it hangs. | 16:02 |
| dimitern | babbageclunk: but if you're just moving all that code from L135 to the end, keep in mind the defer-s above | 16:03 |
| dimitern | babbageclunk: show me your diff perhaps? | 16:04 |
| babbageclunk | dimitern: hmm. I hadn't really paid attention to that. I don't *think* it matters - the test has an open state at this point and a cleanup calling state.Close(). | 16:05 |
| babbageclunk | but I might be wrong. | 16:05 |
| babbageclunk | Ok, hang on - I'll commit and push. | 16:05 |
| dimitern | babbageclunk: yeah, looking at it closely the defer shouldn't cause it to hang | 16:05 |
| babbageclunk | babbageclunk: it's all a bit rough at the moment. | 16:05 |
| babbageclunk | duh, typed my nick | 16:06 |
| dimitern | babbageclunk: for some extra debugging from mgo/txn in tests, try txn.SetLogger(c) and txn.SetDebug(true) | 16:06 |
| babbageclunk | feeling very Friday afternoony. | 16:06 |
| babbageclunk | dimitern: Ooh, that sounds extremely useful! | 16:06 |
| dimitern | frobware: LGTM | 16:13 |
| frobware | dimitern: ty | 16:13 |
| voidspace | dimitern: http://reviews.vapour.ws/r/4827/ | 16:15 |
| dimitern | \o/ not such a bad Friday, 13 after all - the VPC fix is confirmed to work | 16:15 |
| voidspace | dimitern: I think it's fully tested by the existing tests (I had failing tests along the way) | 16:15 |
| voidspace | dimitern: yay \o/ | 16:16 |
| dimitern | voidspace: yay indeed :) looking at your PR | 16:16 |
| voidspace | dimitern: very simple for spaces, assuming I didn't miss anything... | 16:17 |
| cmars | perrito666, so the connection info hasn't changed or anything -- still the same username/password and everything? | 16:23 |
| perrito666 | exactly | 16:24 |
| perrito666 | tag for the user and password for password | 16:24 |
| perrito666 | cmars: install the shell and check just in case | 16:24 |
| cmars | perrito666, ok great, thanks! | 16:25 |
| perrito666 | cmars: ping me if you need a hand | 16:25 |
| dimitern | voidspace: reviewed | 16:25 |
| voidspace | dimitern: thanks | 16:26 |
| perrito666 | could anyone sign a forwardport? http://reviews.vapour.ws/r/4828/ | 16:26 |
| voidspace | dimitern: I don't think there's any point in adding a new spaceGlobalKey name for a single use | 16:27 |
| voidspace | dimitern: maybe entityGlobalKey taking entity name and provider id | 16:27 |
| voidspace | networkEntityGlobalKey maybe | 16:27 |
| voidspace | dimitern: and do you want a ":" separating entity name and provider id? Your email used "#". | 16:28 |
| voidspace | dimitern: it probably doesn't matter but given you've explicitly said ":" in your comment I'd like to check | 16:28 |
| dimitern | voidspace: a generic helper to construct providerIDDoc.ID is better, yeah - taking a globalKey and providerID | 16:29 |
| dimitern | voidspace: as for ":" vs "#" - I've realized some entities use multiple # in their global keys (e.g. LLD and their addresses), hence the ":" | 16:30 |
| voidspace | dimitern: ok, cool - ta | 16:30 |
| dimitern | perrito666: reviewed | 16:31 |
| perrito666 | dimitern: tx | 16:31 |
| perrito666 | ah tx for that spotting :D | 16:31 |
| frobware | cherylj: can I $$merge$$ some outstanding PRs now that beta7 appears to be cut? | 16:32 |
| cmars | perrito666, got connected with the shell. ssl didn't verify though. nbd for poking around in the shell, but where would I find the certs to connect properly? | 16:32 |
| perrito666 | cmars: out of the top of my head /var/lib/juju/agent/blablabal/cert | 16:33 |
| cmars | perrito666, ok, thanks! | 16:33 |
| mup | Bug # changed: 1321442, 1543283, 1557143, 1564054, 1564577, 1567518, 1569467, 1572746, 1574607, 1575332, 1575760, 1575768, 1575808, 1576270, 1576376, 1578373, 1579127, 1581284 | 16:34 |
| mup | Bug #1581600 opened: should be able to host multiple controllers in a single mongodb <juju-core:New> <https://launchpad.net/bugs/1581600> | 16:34 |
| babbageclunk | dimitern: Sorry, got sidetracked, here's where it's hanging in my version: https://github.com/babbageclunk/juju/blob/state-tests/state/open.go#L151 | 16:43 |
| === rodlogic is now known as Guest63003 | ||
| babbageclunk | dimitern: Hmm, probably clearer to see the commits: https://github.com/babbageclunk/juju/commit/aa4d79e1bb8a47ac6b558c36ac9a39378d75f56a | 16:44 |
| dimitern | babbageclunk: will have a look in a bit, otp | 16:44 |
| babbageclunk | dimitern: And then this change to juju/testing: https://github.com/babbageclunk/testing/commit/40e8403b275140383d044253c8d437b77d221d60 | 16:45 |
| babbageclunk | dimitern: No worries - I'm stopping now anyway, brain is too fried. | 16:45 |
| dimitern | babbageclunk: have a nice weekend :) will post comments on your commit if needed | 16:46 |
| babbageclunk | dimitern: Thanks! You too. | 16:48 |
| voidspace | babbageclunk: o/ | 16:49 |
| voidspace | dimitern: PR updated http://reviews.vapour.ws/r/4827/ | 16:50 |
| voidspace | still running tests | 16:51 |
| redir_ | bbiab | 16:51 |
| dimitern | voidspace: provisional LGTM - please double check | 16:59 |
| voidspace | dimitern: kk | 17:00 |
| mup | Bug #1581612 opened: api: Connection interface not well documented <juju-core:New> <https://launchpad.net/bugs/1581612> | 17:04 |
| katco | ericsnow: natefinch: redir: o/ | 17:19 |
| ericsnow | katco: \o | 17:19 |
| cherylj | frobware: I've unblocked master, so you can land | 17:20 |
| frobware | cherylj: ack. thx | 17:21 |
| alexisb | dimitern, ^^^ | 17:27 |
| alexisb | alright all, I am going to be down for a bit replacing some parts on my labtop | 17:27 |
| dimitern | cherylj, alexisb: thanks! | 17:27 |
| alexisb | biab, hopefully :0 | 17:27 |
| katco | alexlist: gl! | 17:32 |
| katco | oops wrong alex postfix :) | 17:32 |
| katco | ericsnow: natefinch: don't forget, last day to get cards on the tanzanite board + clean up specs | 17:33 |
| natefinch | katco: crud, thanks, been heads down looking at a bug. Will clean up my spec now. | 17:35 |
| mup | Bug #1581627 opened: functional-container-networking-lxd fails with invalid parent device name <network> <juju-core:In Progress by frobware> <https://launchpad.net/bugs/1581627> | 17:52 |
| alexisb | yay, old style lenovo mouse buttons I missed you so! | 18:06 |
| rick_h_ | alexisb: lol, new laptop? | 18:18 |
| alexisb | rick_h_, no, just updated track pad | 18:19 |
| rick_h_ | oic | 18:19 |
| === akhavr1 is now known as akhavr | ||
| cmars | if i add some extra fields to a bundle.yaml, will juju still deploy it, ignoring them? | 18:21 |
| cmars | just wondering, for science | 18:21 |
| rick_h_ | cmars: I *think* so but have to verify. | 18:21 |
| cmars | i can find out soon enough :) | 18:21 |
| === frankban is now known as frankban|afk | ||
| hoenir | could someone review my patch ? http://reviews.vapour.ws/r/4809/ | 18:34 |
| === rodlogic is now known as Guest72878 | ||
| === rodlogic is now known as Guest18774 | ||
| redir | fwereade: let me know if you happen to be around and available... I know it is EoW there. Otherwise I'll hit you up monday. | 19:30 |
| perrito666 | cmars: still around? if so, the Cert is in agent.conf | 19:40 |
| cmars | perrito666, ok, got it | 19:44 |
| cmars | thanks | 19:44 |
| === rodlogic is now known as Guest56132 | ||
| natefinch | katco: can you do me a favor? Make a new repo under juju called fslock? | 20:08 |
| katco | natefinch: sure | 20:08 |
| katco | natefinch: done | 20:09 |
| natefinch | katco: thanks | 20:15 |
| === natefinch is now known as natefinch-afk | ||
| perrito666 | could anyone stamp this backport? http://reviews.vapour.ws/r/4830/ | 20:43 |
| katco | ericsnow: natefinch-afk: perrito666: i don't see any cards on the tanzanite board. will you have those ready for monday morning's meeting? | 20:46 |
| ericsnow | katco: yep | 20:47 |
| katco | ericsnow: ta | 20:47 |
| perrito666 | katco: i will, I am mid day still | 20:47 |
| katco | perrito666: ta | 20:47 |
| === rodlogic is now known as Guest76655 | ||
| katco | working on mine right now (finally got some time!) | 20:47 |
| * perrito666 does bureaucracy during the eods | 20:47 | |
| katco | perrito666: hehe :) | 20:47 |
| redir | perrito666: reviewed ^^ but that doesn't get you much yet. | 21:14 |
| * redir gets RSI scrolling through debug log output | 21:15 | |
| katco | time to go to a wedding... have a great weekend all | 21:36 |
| === rodlogic is now known as Guest11340 | ||
| mup | Bug #1581190 changed: juju 2.0: debug-log --include machine-X includes more data than just machine-x <landscape> <usability> <juju-core:Invalid> <https://launchpad.net/bugs/1581190> | 22:02 |
| perrito666 | redir: I appreciate it nevertheless | 22:08 |
| redir | perrito666: I was going to do bug #1576266 but I think it is only on windows... | 22:11 |
| mup | Bug #1576266: apiclientSuite.SetUpTest fails because no tools available <blocker> <ci> <go1.6> <test-failure> <unit-tests> <windows> <juju-core:Incomplete> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1576266> | 22:11 |
| redir | so I don't think I can repro | 22:12 |
| perrito666 | redir it is fixed by the patch you just reviewd most likely | 22:12 |
| redir | perrito666: oic it is failing because the url "file://\\\\localhost\\c$/users/admin" is invalid not because the tools arent' there. IIUC | 22:13 |
| perrito666 | true | 22:13 |
| perrito666 | it fails to fetch the tools because its doing a poor job looking for them :p | 22:13 |
| redir | hehe | 22:13 |
| perrito666 | talk about useful error cannot find transaction ObjectIdHex("5736512d29ee73099b9f641c") | 22:14 |
| redir | looks like more mongo fun | 22:20 |
| redir | or mgo fun I guess if there's transactions involved | 22:20 |
| redir | looks like there's nothing left but windows and maas bugs blocking 1.25. | 22:23 |
| redir | I think I am missing the required bits to help with those | 22:23 |
| mup | Bug #1506121 changed: cloud-images query format deprecated, lxc should use simplestreams <lxc> <simplestreams> <streams> <juju-core:Invalid> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1506121> | 22:32 |
| === rodlogic is now known as Guest78418 | ||
| === rodlogic is now known as Guest28531 | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!