[00:08] katco: good evening, not that you should be still be working now, but i've commented on your PR. so you have something to look forward to when you wake up tomorrow :-) [01:07] wallyworld: would you be satisfied if I just doc commented syncToolsAPI? i.e. say that it's a subset of state/api/Client? [01:07] (thanks for review btw) [01:07] axw: yep, sure [01:07] just to assist the casual reader [01:07] cool [01:07] yup [01:39] thumper: http://paste.ubuntu.com/8174218/ [01:39] my fix at the end was very like that [01:40] ok, let me see if I can make a repro out of it [01:47] wallyworld: that is a long email, will read it fully later, but... good effort [01:47] thumper: yeah, it is long, sorry [01:47] don't be sorry [01:48] i'm not, well, that's what i say to the girls [01:48] wallyworld: that's a yello card mate [01:48] davecheney, any word on pr 617? [01:48] just when you have a minute [01:48] lets keep the induendo below the audible level [01:48] bodie_: i haven't looked at it [01:48] i'm not the on call reviewer today [01:49] it's friday, gotta have a little fun [01:49] gotcha -- can you leave a comment or something to the effect that it's cool if someone else has a look then? I don't want this thing to get trapped in limbo [01:49] I just have a bunch of stuff depending on it https://github.com/juju/juju/pull/617#discussion_r16817826 [01:50] just since you'd left a negative comment [01:51] wallyworld: I have to agree with davecheney on this... [01:51] sigh [01:51] sorry dude [01:51] and since we're on totally different timezones, and nobody else will respond to it since you'd left a comment [01:51] there is a time and place, but this isn't it [01:52] thumper: it's not like you haven't been "guilty" before :-P [01:52] wallyworld: doesn't make it right [01:52] I'm happy to be called out on it [01:53] I have to agree, wallyworld should be more careful [01:54] I mean, if you're going to try and make someone take offence, do it properly [01:54] * bodie_ furiously scribbles notes [01:54] * wallyworld makes a mental note who has a thin skin for next time [01:55] wallyworld: casual sexium isn't about having a thin skin [01:55] * bodie_ compares notes with wallyworld and rebases his changelist [01:55] it wasn't sexism, but anyway [01:56] define sexism: prejudice, stereotyping, or discrimination, typically against women, on the basis of sex. [01:56] anyways, let's move on [01:57] well, where were we. oh yes. so, I've noticed that when a core member on a 12 hour offset timezone comments on something, other team members are hesitant to add their two cents to discussion, and thus my review gets trapped in the equivalent of a HTTP syn timeout on a 12 hour latency window [01:57] I chatted about the PR with TheMue and alexis today and both indicated they wanted davecheney to be the one to respond to that comment on 617 [01:58] thumper: http://paste.ubuntu.com/8174325/ [01:58] repro for the gccgo bug [01:58] filing an issue upstream now [01:59] if someone could at least acknowledge what I'm saying, maybe I wouldn't wonder as much whether the cold shoulder has anything to do with wanting to keep us out of the loop [01:59] I'd say more casual sexual harassment rather than sexism... however... moving along [02:00] https://code.google.com/p/go/issues/detail?id=8612 [02:00] or, you know, you could post a three word reply on that pr, such as "someone else take this" and it would be totally cool [02:00] oops, that was 4... [02:00] davecheney: cool, ta [02:01] cool email wallyworld, thanks for taking the time [02:01] np [02:07] so I'm taking that as a "no, your PR is going to sit in limbo until I'm the next on call reviewer, and/or you're on my ignore list now" [02:07] I really need to go live a few weeks to the countries where you guys are from so I get to learn local english [02:08] * perrito666 feels like the hispanic comic relief on an adam sandler movie half the time [02:08] LMAO [02:09] perrito666, you're cool by me, I never notice your accent ;) [02:10] bodie_: trust me I sound terrible when speaking for long periods of time, I am really bad at managing my air and I get really exasperated when trying to speak in english at the same speed than spanish [02:10] bodie_: nope, that's not it [02:10] i was pulled off onto another crash this morning [02:10] the OCR reviewer rotates daily to avoid anyone being on the critical path [02:11] I just don't like feeling like I'm talking into thin air at timse [02:12] but I understand you guys are totally slammed [02:12] just frustrating trying to push our stuff through when nobody will respond to a thread because someone who's only online when it's 10pm where I live owns the review [02:13] perrito666, english was invented for people who mutter under their breath :P [02:13] bodie_: i think you need to raise that with alexis [02:13] yeah, I did this morning [02:13] bodie_: i've said the same to her in the past [02:13] i'm not sure what I can do [02:13] davecheney, she and TheMue both said they wanted the response on that thread to come from you [02:14] nah, that can't be how it works [02:14] you could just say something like "I'm content for someone else to look at this" or "bodie_, I'll be able to review it next thursday" or something other than... ya know, nothing [02:14] he told me that martin is reponsible for landing your stuff [02:14] so, all i know is eveyrone knows a different story [02:14] TheMue has replaced martin for working with the actions team [02:15] I'm just trying to make things work, man :) [02:15] if we have a blocker, I need to figure out how to get around it [02:15] bodie_: yup, i hear you [02:16] ok, lets do this I have a decent overlap with all of you guys, mostly because I have no life and I hardly sleep, bodie_ could you privmsg me with a brief on what you need? davecheney could you tell me what alexis told you so I can try to unlock it tomorrow? [02:16] maybe you could either respond to that comment, or make a mention to them in #jujuskunkworks, either of which will take no more than 20 seconds [02:16] that way they won't feel like you own the thread [02:17] or, I can pass it along tomorrow, or I can wait for your response when you're able [02:17] I just want to know what to expect so I can be effective here [02:23] lol [02:47] thumper: good news [02:47] upstream have a fix for the gccgo crash [02:47] now comes the hard part ... [02:47] getting it into ubuntu? [02:47] :P [03:08] thumper: https://code.google.com/p/gofrontend/source/detail?r=75739d377426 [03:08] we have a fix [03:08] what's the best way to report this ? [03:08] an issue on ubuntu/gccgo ? [03:09] I guess [03:09] right-o [03:09] i'll try that [03:10] might need you and sinzui to ride shotgun on this one [03:10] who is in charge of gccgo? [03:10] inside canonical? [03:11] dokku maybe ? [03:12] the lack of an owner probably explains why the last fix hasn't landed for 5 months [03:32] thumper, davecheney: i have some gccgo patches i want in ubuntu too [03:32] thumper, davecheney: i think the official process is "hound doko relentlessly" [03:32] mwhudson: I suggest we push through alexisb to pat [03:32] mwhudson: who does doku report to? [03:32] thumper: slangasek i think [03:33] btw, there is already a newer gccgo in trusty-proposed [03:33] have you guys tried that? [03:34] (it obviously doesn't fix this problem that just got fixed, but doko will probably ask...) [03:34] mwhudson: thumper `https://bugs.launchpad.net/ubuntu/+source/gccgo-4.9/+bug/1362906 [03:34] Bug #1362906: internal compiler error: in comparison, at go/gofrontend/expressions.cc:6508 [03:35] mwhudson: nope, wasn't fixed until 30 minutes ago [03:35] mwhudson: pls mail thumper and I with your bug numbers [03:35] i'll add mine [03:35] and it'll be another thing on alexisb 's plate [03:35] https://bugs.launchpad.net/ubuntu/+source/gcc-4.9/+bug/1361940 [03:36] Bug #1361940: patches for cgo on arm64

[03:36] davecheney: this is all for cgo though, that probably shouldn't be on alexisb's plate? [03:36] if it doesn't affect juju or other canonilcal developed products [03:36] probably not [03:37] docker docker docker [03:39] docker docker docker malkovich docker [04:08] axw: do you mind if I add another 10 files and 500 lines to that backups PR [04:09] lol [04:09] axw: but seriously, thanks (to you, davecheney, and waigani) for the reviews [04:10] the rest of my backups patches are *much* smaller [04:11] ericsnow: welcome [04:14] who controls the juju account on github? [04:15] ericsnow: :) [04:15] nps [04:16] ericsnow: all the team leads are owners I believe [04:16] axw: okay [04:16] axw: I'll bug nate about it tomorrow [04:17] axw: nps? [04:26] ericsnow: no problems [04:26] axw: I figured as much [04:46] axw: thanks, i was wondering about wording of that message. i wanted to explicitly call out that there could be an instance running; but as you say, i think mentioning destroy --force will be enough [04:47] cool === mup_ is now known as mup [06:25] axw: i think testing failure bug 1202039 is either fixed, or will be with your current work? [06:25] Bug #1202039: Provider tests are architecture-specific

[06:27] wallyworld: I think they're already working on i386 on ppc64 right? so it's fixed already? [06:28] CI runs the tests on i386 and ppc64 I'm pretty sure [06:28] axw: yeah, that was my thinking too, i'll mark as fix committed [06:28] just wanted to double check === uru_ is now known as urulama [07:36] morning all [08:00] morning [08:58] dimitern: ping [09:00] TheMue, hey [09:01] dimitern: as I as OCR cannot review myself would you mind to take a look at https://github.com/juju/juju/pull/626 ;) [09:01] dimitern: and morning btw :D [09:01] TheMue, sure :) [09:02] dimitern: thx [09:04] wallyworld: just noticed I cannot edit the bugs doc [09:04] would like to strike out a line [09:34] fwereade: ping [09:46] wallyworld: got time to chat about a test I want to try and fix? [09:47] anyone know how to get a list of all charms? i used to use http://manage.jujucharms.com/api/2/charms?text= but it doesn't seem to produce many results now === meetingology` is now known as meetingology [09:51] rogpeppe1: manage.jujucharms.com is still how I know how to get a list of them === Tribaal_ is now known as Tribaal [09:51] wwitzel3: do you have a better URL than that? [09:51] wwitzel3: i only get about 34 out of that [09:52] rogpeppe1: sounds like you're getting trusty only [09:52] wwitzel3: actually 20 only [09:53] oh .. hrmm [09:53] there are 38 charms for trusty .. so if you're only getting 20 [09:53] wwitzel3: it looks like there's a limit kicking in, but adding limit=100, for example, doesn't give me any more [09:54] wwitzel3: have you actually got a current-ish list, by any chance? [09:54] rogpeppe1: https://manage.jujucharms.com/charms/trusty [09:55] rogpeppe1: that's the HTML version if that helps anyway [09:55] wwitzel3: it's a hassle (i'll have to manually scrape the html) but better than nowt [09:56] rogpeppe1: sadly it doesn't do an ajax request (that I can see) to get the results === jcsackett_ is now known as jcsackett [09:56] rogpeppe1: was hoping it might be using the API to render that list [09:56] wwitzel3: yeah [09:57] wwitzel3: it's annoying because it *did* work once. i even wrote a little program to automate getting all charms. [10:04] rogpeppe1: yeah, according to the lp repo for that API it hasn't changed since 2013-10 [10:05] rogpeppe1: but something must have :/ [10:06] wwitzel3: well, the last-modified time on my getcharms.go file is 2013-08, so the 2013-10 change might've broken it i guess [10:06] wwitzel3: which lp repo is it, BTW? [10:08] rogpeppe1: also doest his give you better results http://manage.jujucharms.com/api/3/search ? that isn't techincally an unstable API, but if 2 stopped working for you [10:08] rogpeppe1: https://code.launchpad.net/charmworld [10:08] wwitzel3: i only see 41 results there [10:09] wwitzel3: (assessed by searching for "files": in the JSON :-)) === psivaa is now known as psivaa-off === ev_ is now known as ev [10:23] rogpeppe1: http://bazaar.launchpad.net/~juju-gui-bot/charmworld/trunk/view/head:/charmworld/search.py#L141 [10:24] rogpeppe1: I feel like someone else was running in to this 20 limit thing recently [10:25] wwitzel3: i tried using limit=9999 [10:32] TheMue, reviewed [10:32] TheMue, ping me if something is not clear please :) [10:32] dimitern: yeah, seen a lot of it (and commented some back). thanks a lot, will mark it as wip and continue [10:33] TheMue, cheers! [10:33] rogpeppe1: yeah, sorry I couldn't get it working, the code itself seems to respect limit, but then overrides it somewhere with that default [10:35] wwitzel3: yeah, i couldn't see where that happens. [10:35] wwitzel3: and the limit doesn't always appear to be 20 though [10:35] TheMue, about the dummy provider running "unsafe" networker - it certainly makes sense to use "safe" by default, but there should be a way to test the "unsafe" one if needed - we can isolate what the networker does from the system (config, interfaces), so it's fine [10:36] dimitern: is it ok to do this in a second, follow-up PR? this one already grows and grows. ;) [10:37] rogpeppe1: yeah, if you leave it off all together, it seems to be 41 [10:38] TheMue, i'm not saying to add more tests, just to tweak the dummy provider a bit so RequiresSafeNetworker can return true or false, just by adding a useSafeNetworker bool in the dummy provider struct and a SetUseSafeNetworker(bool) method on the dummy provider to set that flag [10:39] TheMue, it's like 5-6 more lines :) [10:39] wwitzel3: actually no, with a more accurate count, it always seems to be 20 [10:40] wwitzel3: (counted with http://paste.ubuntu.com/8177535/) [10:41] dimitern: ah, ok, haven't been sure that in this case the "unsafe" (/me still not really happy with that term) doesn't make any harm to the own system [10:46] TheMue, should we do a quick standup? [10:46] dimitern: yes, already comming === bodie_ is now known as Guest3502 === uru_ is now known as urulama === ev_ is now known as ev === urulama_ is now known as urulama === BradCrittenden is now known as bac [12:37] wallyworld: is there any assigning system for the tests that need rework? [12:38] perrito666: raise a bug and assign yourself to it. cross out the item in the doc so others can see there that that one is taken [12:39] many failing tests already have bugs [12:39] so if you go to raise a bug, it should tell you if there is one already [12:39] include the test name in the bug description [12:40] so that it has a better chance of finding any dupe [12:41] I guess that created bugs are more important than the ones not yet there [12:42] no necessarily [12:42] there's failures that people haven't created bugs for [12:42] and i found today that bugs i looked at had already been fixed [12:42] but the bugs were still open === jheroux_away is now known as jheroux [12:43] so pick a failure that you think you can fix - all fixes are good [12:43] wallyworld: interesting, i can't add an event to the team calendar =/ [12:44] katco: that's no good, that means you are not allowed to have the holiday [12:44] wallyworld: you are afunny one ;) [12:44] oh, you think i was joking [12:44] :-P [12:44] wallyworld: we're planning on going to our botanical garden's japanese festival :) [12:44] cool, sounds fun [12:44] oldest and largest in the country! [12:45] well, that's something i didn;t know [12:45] http://www.missouribotanicalgarden.org/things-to-do/events/signature-events/japanese-festival.aspx [12:45] oops, says "one of" the oldest/largest [12:46] ah, you americans always exaggerate [12:46] world series baseball = usa plus canada [12:46] I am amazed on how US people actually measure things in order to be able to have "the largest" for almost anything :p [12:46] hey we are THE BEST at exaggerating. no one is better, anywhere. [12:46] wallyworld: its like in columbus times "the known world" [12:47] and me? i'm the most humble person alive. [12:47] you too? [12:47] we can't both be [12:47] hey i'm WAY more humble [12:47] of course [12:47] lol [12:47] you have reason to be [12:47] LOL [12:47] so nice to work with people who appreciate a dry sense of humor :) [12:47] don't know what you mean [12:48] lets say each of you is the humblest in his/her timezone [12:48] what did you ask me yesterday? have i told you today? [12:48] lol [12:48] no you haven't [12:48] fark off! haha [12:48] :-( you hate me [12:49] wallyworld: dont be like that, we all hate you, its because you are a manager [12:49] perrito666: oh, that's a low blow [12:49] haha [12:50] way to hurt a guy [12:50] no, wallyworld, i definitely don't hate you. i quite like working with you, axw, and everyone i've had the pleasure to work with so far [12:50] with the exception of mgz_, traitor! [12:50] katco: i was joking :-) [12:51] katco: nahh, dont piss off the guy that can lock your commits [12:51] ;_; [12:51] haha [12:51] mgz_: i kid, i kid! [12:52] * perrito666 accidentally spills yerba all over his kb [12:56] wwitzel3: sorry i missed your ping before, i was at soccer. did you still have a question? [13:05] need to cycle my server, brb === mthaddon` is now known as mthaddon [13:12] wallyworld: I cant write the doc either [13:13] perrito666: i changed the permissions, have you tried refeshing? [13:14] tx [13:33] perrito666: you around? [13:33] wallyworld: yeah, I was going to try to fixup a test from the list [13:33] natefinch: I am I though you where not here [13:33] perrito666: yeah, sorry, busy morning. I'm in the hangout on the calendar event [13:34] wwitzel3: ok, or it can be any other one you may have come across that isn't there [13:34] or it can be improving the cmd/juju tests to use mocks [13:35] dimitern, do you have 10 minutes spare for a quick question, possible via a hangout [13:36] mattyw_, sure, just give me 15m [13:44] dimitern, great, just ping when it's a good time === Guest3502 is now known as bodie_ [13:52] natefinch: interesting fact, cold medicine hinders my already low spoken english skills :p [13:53] perrito666: haha [13:53] perrito666: btw, it's "beats around the bush" [13:53] ahh well close enough [13:53] yeah, I knew what you meant [13:54] natefinch: it is hard to translate those, the spanish one is nothing like it [13:54] yeah... those kind of sayings are kind of crazy, because the actual words don't make any sense [13:54] literal translation of spanish one is "dodges the lump" [13:54] to give you an idea [13:54] rofl [13:54] of how far it is [13:58] omg, even though its not ergonomic I must say, thinkpad keyboard that emulates the one in the laptop is awesome [13:58] I wish I had found a newer one, I actually got one ibm branded :p [13:59] haha [14:01] I also with it was bt, this one actually has a 2.5M cable :p [14:14] is there a place we could put consts representing config key-strings? seems silly and error-prone to repeat them all over the codebase. [14:16] katco, how about in environs/config ? [14:17] dimitern: that seems like a fine place [14:17] dimitern: could probably even put it in environs/config/config.go [14:18] dimitern: is this a small enough shift that i can just introduce this concept? :) [14:19] katco, go for it :) [14:19] dimitern: awesome, thanks for the input! :) [14:20] hook execution question [14:21] do hooks in a service group execute serially or is that only on the unit level? [14:23] boy documenting all public values seems silly sometimes [14:23] // ProvisionerHarvestModeKey stores the key for this setting. [14:23] ProvisionerHarvestModeKey = "provisioner-harvest-mode" [14:23] ahh tautological comments [14:24] katco: you could expand on what exactly does that key means [14:24] i know... i understand wanting good documentation, but an absolute rule doesn't make sense [14:24] perrito666: that's already covered in the actual setting [14:24] perrito666: this is literally the key to get the setting in the map [14:25] i especially dislike repeating the name of the thing in the comment. hungarian notation all over again. [14:47] hi marcoceppi_ [14:47] hey bac [14:48] marcoceppi_: lazyPower_ has filed a bug regarding supporting tags as a constraint for MAAS. you've seen it i think. [14:48] bac: yup [14:49] marcoceppi_: in the definition tags are a comma-separated list. until now we've marked bundles with constraints listed as a comma-separated as deprecated. clean implementation of tags will require us to reject those deprecated bundles. [14:50] i.e. constraints: 'k1=v1 k2=v2 tags=a,b,c' is valid bug 'k1=v1,k2=v2' is not [14:51] bac: the error we're getting is tags is not a valid constraint [14:51] * marcoceppi_ checks the bundle file [14:52] marcoceppi_: yes. i'm trying to fix that. [14:52] oh, I see [14:52] one sec [14:52] marcoceppi_: to properly support tags will require not allowing comma-separation in the other constraint key value pairs [14:53] bac: maybe constraints are all wrong [14:53] is it possible to run charm-proof against the set of all bundles to see if it is even an issue? [14:53] maybe they should be a YAML list [14:53] bac: I can download them all and see [14:53] marcoceppi_: that would be great [14:53] constraints: [14:53] - key: v1 [14:53] - key1: v2 [14:53] hazmat: ^^ [14:54] marcoceppi_: that may be the best long term solution but i thought you needed a fix now. [14:55] well, I'll look at all the bundles we have [14:55] and see what each is doing for constraints [14:55] changing proof rules already touches way too many parts... === hatch__ is now known as hatch [15:01] oh, bugfix by accident, nice. field of a struct has been prepared but never been set. now I deeded it. [15:01] TheMue: lol [15:02] perrito666: yeah, gave me a panic during tests :D [15:04] TheMue: which one (bug)? just in case [15:05] perrito666: it never has been filed. it has been a field for checking the authorization of reading a machine [15:06] perrito666: the function (it's a function type) has been created, but the field never set [15:06] perrito666: and I now wanted to check if I can read => *bang* [15:06] ouch, I wonder if its there on the bugs ian added to the doc which are not all reported [15:09] have to check === jheroux is now known as jheroux_away [15:57] Anyone up for reviewing my log rotation change? It's pretty small: https://github.com/juju/juju/pull/512 === ChanServ changed the topic of #juju-dev to: https://juju.ubuntu.com | On-call reviewer: see calendar | Open critical bugs: 1363130 1363143 [16:18] natefinch, I'll take a look [16:25] mattyw: what is a vanity url? [16:26] github.com/perrito666/projectname [16:26] perrito666, A url with someones name in it [16:26] perrito666, I'm not happy with the name vanity url either :) [16:27] but that's what they're called in the mailing list [16:27] and who am I to go against consensus? [16:27] I would think a vanity url is something like beautifulpopleonly.com/perrito666/thewonderfulwonderfulman/ [16:27] :p [16:28] now that is a vanity url [16:32] perrito666: can you get me one like that too? [16:32] :p [16:32] perrito666, I'd be happy with that one - at least make it obvious [16:33] perrito666, what I'm happy and not happy with is inconsistent and changes daily [16:33] we might want to rething our mentor policy to not be 12 hours distance with our reviewers [16:34] perrito666: quit complaining... you're on here 24/7 anyway [16:34] jcw4: Id say around 20 [16:34] but yes [16:34] :) [16:35] its the corners of the week which are a problem, my mentor will not be here until sunday (my sunday) [16:35] yeah, that's a bummer. [16:35] perrito666, how many corners does a week have? I thought they were circular? [16:35] they feel circular sometimes [16:35] I have to say though, working with folks all over the globe make it more than worth it for me :) [16:36] if weeks where circular in two days it would be last monday [16:36] spiral then [16:37] uff, this bug is a heavy thing to pinpoint, I am pretty sure I know what is going on, I am not so sure on the how part [16:38] my weeks have no corners, only hills and valleys [16:43] perrito666, mattyw: a vanity url, in the context of Go is a url which is not the same as the url to the repo. So, for example, the package that gets imported when you do import "labix.org/v2/mgo" is actually hosted on launchpad, so the real url is launchpad.net/mgo (or something like that). [16:46] natefinch, I didn't know that - thanks very much [16:46] mattyw, perrito666: Go supports this through the go tool which, if it doesn't recognize the url of the import path, will fall back to trying to get a web page from that URL, and it'll look for a header that looks something like [16:47] It's pretty nice because it means that if you control the domain, you can move your code from one host to another and not break anyone else's code that imports using the vanity url. [16:51] natefinch, now I know the proper definition of vanity url my comment probably makes no sense [16:52] natefinch, I was just wondering if accepting a number of dependencies that had team members names in was ok [16:53] has anyone else seen this error using the local provider on tip? http://paste.ubuntu.com/8180167/ [16:58] mattyw: are you sure your code is up to date? [16:59] perrito666, it was tip [17:00] mattyw: looks a lot like someone made one of those automatic changes on the code and forgot to change a return somewhere [17:01] perrito666, I was trying to recreate this https://bugs.launchpad.net/juju-core/+bug/1363143 but I can't even get that far [17:01] Bug #1363143: local lxc deployments fail to create machines

[17:01] mattyw: checking [17:01] perrito666, the mentorship matching is my fault, we did the best we could but most of the mature skills on the team are in +12 timezone, you are one of the few with potential overlap [17:02] alexisb: sorry didnt meant to make you feel bad [17:02] mattyw: and this blows running bootstrap right? [17:02] perrito666, yeah [17:02] * jcw4 wonders about relocating to UTC+12 [17:02]