[00:14] hatch: you around tomorrow? We can chat about this with kadadms [00:14] yeah, I'm getting the yard un-wintered :) [00:14] cool [00:14] it sounds like it's on the right path though [00:15] yea, I think we had a good conversation about a good path [00:15] but I did want to make sure you were involved if you had feedback [00:15] I'll have to take a deeper look at it [00:16] but it seems good at face value [00:27] rick_h_ what time tomorrow do you want to have a chat? [00:28] hatch: not sure, kadams seems to start around 10, but I've got a delivery 9-11 (eastern) [00:28] after standup maybe? [00:28] sure that works for me [00:29] hatch: coolio [00:31] how goes the battle with the NUC? [00:31] good, have maas installed but have to figure out the network stuff [00:32] ATT isn't going to get to my new network stuff until later in the month [00:32] so might just sit on it until then when I can put them on the network and not have ot change it around [00:32] wth, how hard can it be for them to flick a switch? hah [00:32] well, to get business class I've got to get on a different pipe that's not been run down my street [00:32] so they've got to do that first, then schedule my install after that [00:33] oh haha [00:33] WELL THEN! [12:12] redir: qa'd and commented. One small code change request [12:31] rick_h_: did your qa work after blowing away the ES index manually? [12:35] jujugui: heading to Michigan's equivalent of the DMV to change the title on a car. Not sure when I'll be back at the keyboard, but it will definitely be before standup. [12:35] bac: I did a full emptydb and and cleared the index manaully and then I forgot to run the app so proof would pass [12:35] after doing all of that I did finally get it to ingest correcty lol, been too long since I worked on charmworld [12:35] kadrgr [12:35] bah, how dare he leave so I can't tab complete his name [12:35] rick_h_: i'm looking for implications for deploy, things we may need to do in a migration. [12:36] bac: well I can't say for sure what state my old lxc container was in and such [12:36] I think it was a combo of bad things I did on my part [12:36] rick_h_: ok. well, perhaps we'll just wait to see what happens when jenkins lands it on qa [12:37] bac: rgr [12:40] my mom called b/c the internet didn't work. took 20 minutes for her to admit she had a router and to find it. [12:44] lol [12:45] didn't know routers are secrets to be ashamed of [13:00] code change related to: http://bit.ly/1lsXCfv [13:00] rick_h_: ^^ [13:01] redir: cool, that's a link to a bug? [13:01] I understand that's the background of the change, but wouldn't a skip of the test do the same as commenting it out? [13:01] I am going to a cafe for a an hour or two -- while someone works in our apt. Should be back for the stand up. [13:01] redir: rgr, thanks for the heads up [13:02] rick_h_: I recommended a skip in that bug and abentley said he didn't think so [13:03] ok, so he's arguing the test should stand as is [13:03] I haven't looked deeply at what test under test.ini does though. [13:03] right now it's commented out [13:04] I think a skip > commented out and a working test > skipped test [13:04] I don't really understand what he is saying [13:04] so for this branch, I'm ok with a skip. And a card in maintance to look at this bug [13:04] I agree with you [13:04] but honestly, the goal if you working on the store is to remove this so I'm +1 skip and carry on for now [13:04] there's a bug noting the issue, maybe put the bug number in the skip message [13:05] and call it a day [13:05] good idea [13:05] will do [13:05] thanks [13:05] morning, all. [13:06] morning jcsackett [13:06] bbiab [13:29] delivery man is here, biab [13:40] back [13:59] rick_h_: got a sec to chat imp details on change summary? have a question with changing configs pre deploy, specifically. [14:02] jcsackett: sure thing [14:03] rick_h_: https://plus.google.com/hangouts/_/gymhrqahxkwfy5tn6duz2v72yaa?authuser=2&hl=en [14:32] morning hatch, when you get settled want to run an interview presentation question by you before emailing it [14:35] sounds good, ha ve it in a doc somewhere? [14:35] hatch: yep [14:36] hatch: shared with you [14:36] cool, looking [14:36] just as I get over my cold, I get allergies lol [14:36] hah! welcome to spring! [14:39] modified to run desctructive by default under the test make target or not under testdebug target. http://paste.ubuntu.com/7462988/ I could reverse the logic and make test nondestructive, then add a testci target that is. Then sinzui only needs to change that target on CI, IIUC. [14:39] guihelp thoughts? [14:42] I think I like the latter better. Then we only have to deal with thte booger once changing the ci config, rather than all developers needing to deal with it when running tests [14:42] redir: looking and rereading the bug filed [14:43] cool heading back to apt for call. hopefully they re done in my office. [14:43] brb [14:50] jujugui call in 10 [14:54] back [14:55] redir: cool, let's chat after the hangout. I think I better understand abentley's point and the issue now. [14:55] k [14:56] redir: watch for stray pdbs :-) [15:00] it isn't committed. still needs doc'ed etc. but thanks for the reminder. I have been known to dangle a pdb in a commit or two [15:01] jujugui call now [15:06] redir: but where did your Quickstart come from? probably the PPA. the card is for getting into main. [15:07] OIC, tx bac [15:14] hey guys, when doing and api search (e.g. http://manage.jujucharms.com/api/3/search?text=mongodb), is there a way to say "only give me charms" or "only give me bundles" ? [15:15] s/and/an/ [15:18] tvansteenburgh at the moment you can only get 'all' bundles [15:18] fixes are in the works for enhancing the search though [15:19] hatch: okay thanks [15:19] tvansteenburgh: charm:mongodb [15:19] tvansteenburgh: bundle:mongodb [15:20] rick_h_ so I think I'll trash the card that's on the board and create a series of them to 'hook up drop targets for unplaced units' [15:20] hatch: jump in hangout to follow up? [15:20] sure [15:20] in the standup hangout [15:20] hatch: rgr [15:21] hatch: fyi ^ [15:21] oh that's interesting, is that new? [15:21] hatch: we did that a few months ago, so newish? [15:21] yikes where have I been [15:22] rick_h_: cool, thanks! [15:22] hatch: https://plus.google.com/hangouts/_/canonical.com/daily-standup?authuser=1 [15:22] tvansteenburgh: np, let me know if you hit any issues [15:41] rick_h_: any suggestions on how to handle this: http://manage.jujucharms.com/api/3/search?type=approved&text=charm: [15:42] i want "all approved charms" but this query returns bundles too [15:42] tvansteenburgh: hmmm, thinking [15:44] tvansteenburgh: http://manage.jujucharms.com/api/3/search?type=approved&text=&series=trusty and update for each series you want the data for. [15:44] tvansteenburgh: bundles don't have series so you should get those filtered out? [15:45] rich_h_: will bundles ever have series? [15:45] tvansteenburgh: not that I'm aware of [15:45] tvansteenburgh: what's this for? If it's something we need to support we can look into it perhaps [15:46] charmworldlib is a python library on top of this api. i'm working on adding bundle support to the lib [15:46] annnnnd I'm working outside! [15:46] what bundle feature of bundle support is this doing? [15:46] hatch: :P [15:46] ok I needed a blanket, but damnit I'm outside! [15:47] when i do Bundles.search() i want to make sure i only get bundle data back [15:47] tvansteenburgh: ok, but doing bundle:term should work for that. [15:47] tvansteenburgh: what's the recommended charm search part for? [15:47] tvansteenburgh: chat? [15:47] rick_h_: yeah, chat [15:48] tvansteenburgh: https://plus.google.com/hangouts/_/gzypx4siy6biijpzjj4hq2ey6ma?authuser=1&hl=en [15:53] that causes an error, so I skipped the test committed, will look at the test separately. [15:53] so now I change status to approved? rick_h_ bac? [15:58] redir: ok, yea skip and land and add a follow up kanban card for the test [15:58] redir: make sure to put that bug number into the new card as the "card id" field [15:58] having never landed before I am guessing I change the status to approved [15:58] then magic happens, rick_h_ [15:59] redir: yep [15:59] then if all goes well CI will pick it up and try to land it. That CI tarmac stuff is still running the show right bac? [15:59] and where would you like the card added [15:59] maintenance? [15:59] redir: might as well stick it in Maint. branch start [15:59] tracking [15:59] k [15:59] tx [16:00] redir: but time box it. Look at that test update for today, if there's no advancement let's move it back as a todo item and get you back onto the core store stuff please [16:03] k. approved revision == 515 but the skip is in 516, does that matter? [16:04] hmm, it might complain about revisions not approved. sec [16:04] redir: ah, yea you need to fill out the commit field [16:04] see the landing bot being all helpful like that [16:05] * redir likes robots. [16:07] * rick_h_ goes to get lunchables [16:24] * rick_h_ is back from lunchables [16:25] mmmm lunchables [16:35] rick_h_: after looking at the charmworld code i realized the error of my ways. http://manage.jujucharms.com/api/3/search?type=approved&text=charm works just fine [16:35] tvansteenburgh: ooh, good call. I didn't think to ditch the : [16:35] the source always wins [16:35] +1, thanks for your help [16:47] chickadees are out :-) [16:55] In the saddle for the afternoon [16:55] guihelp: looking for eyeballs on https://github.com/juju/juju-gui/pull/320 [16:56] kadams54: can help look shortly [17:12] oh no, leankit has hired all of those little bastards from vegas: http://leankit.com/blog/wp-content/uploads/2014/04/8-WIP-Personas1-570x1024.jpg [17:15] Minions. They're called minions. ;-) [17:18] haha minions rock [17:20] lol [17:26] kadams54 I think it was you who said the container tokens get the data-id attribute set somewhere so it didn't need to be in the template [17:26] do you remember where it is set? [17:27] If I had a time machine I'd go back and punch past-me in the mouth for saying that. [17:27] Oh, yeah, that's right [17:28] kadams54: I've put some feedback in that I think drives this a bit deeper. Can you give it a read through and we can try to figure out if it makes sense and break the work. Maybe some this branch and some as a follow up? [17:28] hatch: here: https://github.com/juju/juju-gui/blob/develop/app/widgets/machine-token.js#L129 [17:29] Will look through [17:29] Thanks rick_h_ [17:29] kadams54: thank you, it's a good start on the work we need. [17:29] kadams54 ohh it was the machine token, not the container token [17:29] ok the container token maybe didn't ever have an id set [17:31] Yeah, container token doesn't right now, but machine and unplaced unit do [17:31] got it [17:31] Both of those work the same way now [17:31] Used to be machine had it in the template and unplaced unit didn't… or the other way around [17:33] redir: what's the card in tracking you've got there? [17:33] in which? [17:33] in Proj A [17:33] one of the store bugs I started on [17:33] rick_h_: [17:34] redir: ok, I'm going to move it back, tracking is for stuff outside of our control we're monitoring [17:34] * redir looks to see where it should be [17:34] redir: like getting quickstart updated in trusty. We've handed off the work and are watching others who do the final landing bits. So we're tracking their progress towards our goal [17:34] redir: I'm just moving it back into ready to code [17:35] and tomorrow then just move it to 'branch start check list" [17:35] or later today! [17:35] redir: or later today :) [17:35] can't reproduce anyhow [17:35] but more on that later [17:35] ok [17:37] rick_h_: are you rharding on lp? [17:37] redir: yes [17:38] so....containers don't have unique id's? [17:39] hatch: machine containers? They should. [17:40] the object passed into the container token doesn't have anything resembling an id [17:40] containers = this.get('db').machines.filterByParent(parentId); [17:40] ok, well a container is just a machine and machine's have ids [17:40] ok I'll look into this filterByParent method then [17:40] so the machine id of the containers should be their id [17:40] it might not be passing back all the data you need [17:40] right [17:41] so my neighbour just asked me if I have the day off...I said no....this is work [17:41] kehehe [17:41] hah [17:42] would like this screen to be a little brighter though [17:43] this is pretty darn cool https://chrome.google.com/webstore/detail/octotree/bkhaagjahfmjljalopjnoealnfndnagc [17:43] ^ shows the github repo in a tree format [17:43] nice [17:47] guihelp easy review -> https://code.launchpad.net/~reedobrien/charmworld/reindex_test/+merge/219569 [17:47] redir: on it [17:47] zomg that's HUGE [17:47] merci [17:47] ;) [17:48] redir: would you like your branch to the bug? [17:49] redir: recall the --fixes=lp: thing on commit will do it for you... [17:49] ? [17:49] link branch to bug [17:49] oh [17:49] I put it in the commit message [17:50] didn't realize it was cli time req't [17:51] rick_h_ oh I think I found the issue - if the filter returns an empty array it renders the token as bare metal [17:51] so no id [17:51] hmmm [17:51] hatch: ah, right well that's true [17:52] well NOW you tell me! [17:52] :P [17:52] well you said a container, bare metal isn't a container :P [17:52] okI'll let it slide......this time! [17:53] redir: sorry, if i confused you. if you do 'bzr ci -m "my wonderful changes" --fixes=lp:123456' the branch will be linked to the bug on LP the next time you push. [17:53] bac how do I link them after the fact? [17:53] just stick it in a comment/ [17:53] no [17:53] ? [17:53] redir: in the UI in the upper right there's a link for it [17:53] On the bug page there's a "Link branch" and in the branch there's a "Link to bug" or something like that [17:54] You need to do it from the bug page [17:54] go to the branch page and click 'link to bug report' [17:54] yeah what rick said [17:54] hatch: or the branch page [17:54] or that [17:54] :) [17:54] LP is a little confusing sometimes [17:54] they are ambidextrous [17:54] hah [17:54] hatch: how would you suggest rewording "Link to bug report" to be clearer? [17:55] bac I don't think it could be clearer [17:55] I mean, it's confusing because you can't do it from the merge proposal [17:55] when did LP merge proposals get in-line comments??? [17:56] I don't see said comments [17:56] hatch: on https://code.launchpad.net/~reedobrien/charmworld/reindex_test/+merge/219569 do you not see several places that reference "inline comments"? [17:57] hatch: it could be that it is a feature only enabled for beta testers or LP devs or something [17:57] I see link in bug -> branch, but not in branch -> bug [17:57] I see your comment which references the inline comments but there are no inline comments [17:57] redir: did you make the change i requested? [17:58] there should be an in-line comment below line 21 [17:58] I didn't se a requested change [17:58] but yes I just saw that [17:58] redir: so maybe in-line comments don't work so well [17:58] comment [17:58] haha [18:00] looks like it is there now [18:01] you prolly need to reapprove for the new commit [18:01] redir: and you must put the commit message into the merge proposal before sending it on [18:01] yeah. [18:01] :( [18:01] I did but didn't save before clicking see inline comments [18:02] redir: done. you can toggle the approve now and it should be happy [18:02] rick_h_: had a chance to read through your comments and digest. Let me know when you've got a few minutes to chat about it. [18:03] kadams54: k, otp atm and will get back [18:03] guibot doesn't miss a beat [18:03] thanks bac [18:16] kadams54: got time? [18:16] Yup [18:16] kadams54: join me in the standup hangout? [18:17] k [18:22] rick_h_ do you use google play music? [18:23] hatch: yep [18:23] it's suggesting me songs from you hah [18:23] lol [18:23] I just started using it and man I'm confused [18:23] sorry about that, I listen to a lot of kids stuff and a lot of soundtracks on there [18:23] yea, it can be confusing. [18:23] it's nice though, if you subscribe, to just get to anything. I almost always live in "listen now" area [18:24] well apparently I can upload my music to it? But I have to pay to do so? [18:24] maybe not? [18:24] I can only download a song twice [18:24] .....It's just confusing when it's not a streaming service [18:27] hatch: yea, there's the music manager app you can get [18:27] I used that to do my uploads [18:27] I think you only have to pay over 20k songs [18:27] and OMG yay my tablet is back from Delta! [18:27] I think it needs some better onboarding [18:27] redir: hey, guess what i found. bin/sync-index [18:27] haha it was actually your tablet?? [18:28] rick_h_: you left it in the seatback pocket/ [18:28] s///?/ [18:28] bac: heh, no I pulled it out of that pocket and put it on my seat so I wouldn't forget it [18:28] wow [18:28] then I pulled my bag down from the storage above and left it [18:28] rick_h_: i was thankful to get back my stainless water bottle [18:28] so I was just stupid, while being thoughtful to make sure I didn't forget it [18:29] but yay, because there's not a good tablet I'd buy and don't want to pay full price for another N10 [18:29] bac does it do what you think? [18:29] or work? [18:29] redir: it seems to! miracles. [18:29] I saw it and think it wasn't doing what I wanted [18:29] but that could have been me breaking shit [18:30] * redir likes miracles nearly as much as robots [18:30] redir: it seems to just put the charms in mongo into ES. perhaps it doesn't index bundles...i haven't looked [18:30] trumpets blare, angles sing:) [18:30] angels even [18:31] me runs car^H^H^Hbike to shop [18:31] redir: yes, it does not know about bundles [19:15] grrr flamingo double charged me [19:16] hmm, I should take a look [19:16] thx for mentioning that [19:16] redir: :( [19:17] looks like they charged me for the internet [19:18] jujugui ^ did they charge anyone else for their room internet? [19:18] hatch: No [19:18] ugh that place is such a bess [19:18] mess* [19:18] * bac looks [19:18] hatch: Though I should re-check my credit card just to verify - is this a charge you just noticed show up? [19:18] hatch: did you check out at the desk? [19:18] kadams54 it went through on saturday [19:19] bac no.....you still have to do that? I didn't know it was 1990 still :P [19:19] the saturday we left that is [19:20] hatch: i did it so that i'd know if there were outstanding charges [19:20] Makyo how about you? Did they charge you for your internet in the room? [19:21] hatch: Verified. No charges for internet. [19:22] rick_h_: https://github.com/juju/juju-gui/pull/320 is ready for another round of review + QA. [19:22] kadams54: rgr, thanks. [19:22] ok I'll check with sarah to make sure we aren't being double charged thx [19:23] They tried to bill me for the internet service, but I refused to pay it before I left. I had them remove that and a call to hotel tech support. So they agreed, then billed me. And then billed me again a week later for roomservice a second time. [19:24] lol [19:24] I hope that place is stricken from the sprint list [19:24] stricken with prejudice [19:26] hatch, checking. [19:27] hatch: by 'that place' i hope you mean 'nevada' [19:27] bac lol, there are some really nice places in Nevada [19:27] i can't see us sprinting at any of them [19:28] bandwidth at the bottom of the canyon is miserable [19:28] haha....well that may be valid [19:28] hatch, no, because they used your card for incidentals/ [19:28] rofl [19:44] rick_h_: just submitted my expenses. Should I send you the receipts for when you approve, or should I just send them along to expenses@canonical.com? Is there any other info I need to include in that e-mail so that they can connect my receipts to my expense report? [19:45] kadams54: you can just send them along. [19:46] * redir redoes math and they only overbilled me by a dollar. Amazingly, the receipt is pretty inscrutable. [19:46] kadams54: please make sure to rebase down the 3 commits. [19:46] rick_h_: I'd rather not… they're not related commits [19:47] Or rather, they're distinct units of work within themselves [19:47] "Logical commits" per hatch :-) [19:47] kadams54: ah, I just saw the dupe tests and assumed it was gardening [19:47] kadams54: all good then, nvm [19:47] That was drive by [19:47] :D [19:48] kadams54: reviewed [19:50] Thanks! I'll make the comments changes and then land it [19:50] kadams54: ty much [19:50] kadams54: and you're in a good place for the follow up then? [19:50] yeah [19:50] cool [19:51] Follow up being to setup the array of machine token and improve rendering logic to only redo the changed bits [19:51] s/token/tokens/ [19:51] kadams54: rgr [19:51] I'll setup a card for that shortly. [19:51] k [20:06] erm should expenses show up in canonical admin under my expenses after submitting, but before approving? [20:06] redir: yeah. Did you hit the Update link at the bottom before sumitting? [20:06] I thought I hit update [20:07] but I see nothging in the anywhere [20:07] I hope I am not submitting 20 copies of this:) [20:07] Hmm… I had that happen once when I tried to save a draft, but I think I forgot to hit update. [20:07] Fortunately it took when I re-entered everything, hit update, and then submitted [20:08] oic [20:08] redir: have to check with mramm, I think your stuff goes to him atm [20:08] you update then save [20:08] rick_h_: it does according to this [20:08] just want to put some receipts away... [20:09] * rick_h_ thinks that's a nice way of saying...not my problem! :P [20:09] rick_h_: noted [20:09] * redir prepares to bother the ramm [20:11] and with that, I think I'm going to run away for the day [20:11] have a good night all [20:11] * rick_h_ runs fast as fast can be...trips over the dog...doh [20:18] woah....apparently the ecs can brick chrome [20:18] lol [20:19] night [20:20] Ah yes, I did that a few times. [20:33] I filed a bug on it [20:33] that definitely shouldn't happen [20:33] :) [20:39] what's the ecs? [20:42] environment change system [20:42] environment change sets [20:47] rick_h_: how did DL know it was yours? just from the seat, label, constant phone calls? [20:48] lol oh right I forgot it was renamed to sets from system [20:48] bac: so I filled out the form, told them my seat, described the item, listed the movies and such on it, and they just replied "we think we found your item" [20:48] so not sure which item made my case [20:49] * Makyo dogwalks [20:49] I'm surprised someone didn't steal it [20:49] like the clean-up crew or something [20:49] maybe there is still SOME good in the world lol [20:49] hatch: they thought about it, but stole the lost iPad instead. [20:49] haha [20:50] probably the truth [21:03] Makyo does the 'ghost' machine need an id that is representative of it's end position..... ie) container vs machine style id's? [21:35] hatch, no, I don't think so. Just something that can be deleted later/ [21:35] ok - I'm just trying to track down a bug and trying to narrow down the issues [22:26] how hard is it to run charmworld locally? [22:28]