[01:32] bcsaller, you around? [01:33] gary_poster I am [01:33] hey bcsaller. How is your import support intended to be used within the context of our current goals? [01:35] gary_poster: For the store, you create a new db and a topo configured for a bundle, import the YAML into the db and render the topo into some container in the store [01:35] gary_poster: its not about trying to replicate what the server side deployer does, though that option could exist down the road [01:36] bcsaller, ah! I see. looking around [01:38] bcsaller, this branch does not actually hook up the YAML parsing, right? If it does, I'm missing it [01:39] gary_poster: no, there is no UI on it yet and the YAML isn't required to trigger the function, but the tests show what the parsed YAML looks like, its just imported deployer styled data [01:39] bcsaller, cool. thanks, was trying to see where it fit in. [01:40] won't get a chance to review it myself: finishing up closing email with goals etc, [01:40] I could connect the import code to it though (which makes sense) but we can't really transform many ghosts into real deploys yet [01:40] for example I don't think we have any special support around pending relations yet (though this can tag those) [01:41] gary_poster: this is a needed step to rendering bundles in the store [01:41] and we'll want the import spelling to match what frankban is building in the charm server [01:41] right bcsaller, got it now. It will also help with letting the fakebackend dupe the deployer [01:41] but we are not there yet [01:41] should be easy to change, but its in sync what what is exported now [01:41] right [01:42] cool, thank you! by "import spelling" I just mean the APIs that we will use to pass the YAML to the server [01:42] ahh, ok [01:42] thanks for taking a peek :) [01:43] np :-) [01:58] hey huwshimi, I assigned you to #1207045, #1207041, and #1207039. Do you want to be unassigned from any of them? We need to have them resolved by end of next week. [01:58] <_mup_> Bug #1207045: Lots of whitespace below the charms on the landing page of the charm browser in fullscreen

[01:58] <_mup_> Bug #1207041: Opening up charm browser details view shifts environment left in IE10

[01:58] <_mup_> Bug #1207039: Inspector settings viewlet has nested scroll bars in IE10

[02:02] gary_poster: I started work on them but put it aside to work on other things. I can pick that up again. I wasn't able to figure out Bug #1207041 [02:02] <_mup_> Bug #1207041: Opening up charm browser details view shifts environment left in IE10

[02:03] huwshimi, I don't want to shift you around randomly, but we do need it done when I said, and the sooner the better. What is a reasonable schedule, given your other goals? [02:03] huwshimi, I will take your name off that one now [02:04] gary_poster: I can pick it up again this afternoon [02:04] cool thanks huwshimi [02:04] np [02:11] hatch you around? === schwuk is now known as schwuk_away === schwuk_away is now known as schwuk [13:02] benji: is aaron out today? [13:03] bac: nope, I expect he'll be in momentarily [13:03] yea, next week he's out [13:06] rt [13:38] hi abentley, to avoid having staging get the multiple ingesting of baskets bug (due to id vs _id) i'd like to merge the branch you reviewed last night but with the bundle display disabled in search.pt to avoid having a 404 link. seems safer than waiting for my next branch to land, which will be no earlier than tomorrow. [13:39] bac: That sounds fine to me. It's not just staging, though. Production also has this bug. [14:01] abentley: sorry, i got disconnected. did you reply to my question? [14:02] bac: Yes. "That sounds fine to me. It's not just staging, though. Production also has this bug." [14:02] abentley: ok. i'll merge it now. [14:14] morning [14:15] morning [14:43] hatch, Morning [14:58] bac, benji, or anyone else who wants to look at some python code: could you please review https://codereview.appspot.com/12903044 ? [14:58] frankban: sure [14:58] darn, I was too slow [14:58] benji: i need two reviews ;-) [14:59] yay! [14:59] heh, thank you both [15:15] sooo hows everyones day so far? [15:17] I get to go fishing after work today so the later the day goes the better it gets :P [15:17] oh awesome - I almost made it out kiting yesterday but the wind started to die after work so I wasn't going to make the drive [15:17] I need to start work about 2h ealier in the off chance that there will be wind [15:17] haha [15:18] rick_h: what kind of fish are by you? [15:18] the lake I go to is pike, bass, crappie. There's a lot more around the area though [15:18] ahh cool [15:19] honestly, I just like going out by the lake as it's really low traffic and chilling for a while [15:19] yeah for sure - I hear there are private lakes in the US in the south [15:31] sinzui: 1x1? [15:36] abentley, yep [15:43] * hatch really misses his second monitor [15:46] jcsackett: bac has access to the analytics I think. I was hoping he'd reply to the email but he's afk it looks like [15:48] noooooooooo, lbox hanging....oh the hate [15:49] jujugui guichat in 10, kanban now [15:49] Pff. [15:49] jujugui reviews please https://codereview.appspot.com/12797046 will swap reviews for reviews [15:49] rick_h, you're on. [15:49] rick_h: got one [15:49] https://codereview.appspot.com/12900043/ [15:49] Makyo: mohohahaha I beat you [15:50] hatch, My clock STILL says 9:49! [15:50] Just flipped over. [15:50] oh noes! my first go review [15:50] Makyo: just either lucked out or is doomed [15:50] Thankfully still in JS :o) [15:50] ah, tricky filenames with go...just not at the end [15:53] there's bac! get him [15:54] AHHHHH [15:54] that was me charging [15:54] er? [15:55] bad internet day [15:56] rick_h: review done [15:56] hatch: thanks [15:57] next time qa before you propose :P [15:57] * hatch internet slaps rick_h [15:57] whoa, hatch advocating more tests? [15:57] I know right? lol [15:57] hatch: :P yea, that z is embarrassing [15:57] lol [15:57] rick_h: did you need something? [15:57] hatch: oh well, at least my code has tests [15:57] haha [15:57] bac: was going to hit you up on the call, mramm is asking about analytics on jujucharms.com and I thought you had access [15:58] bac: wanted to see if you could reply to the email if you do? [15:58] bac: was to the -peeps mailing list [15:58] jujugui call in2 [15:58] rick_h: do you have a yaml file for qa? then I can qa once you update [15:58] hatch: no, I was looking around and didn't find one. [15:59] rick_h: oh, ok [16:01] frankban, starting without you [16:16] bac: just ping me when you want to talk then [16:16] rick_h: are you planning on expanding those tests? (I just saw that you updated the propose) [16:16] bcsaller: ok. we don't need to talk if you're just going to say "be aware but you don't really need to do anything now." [16:17] hatch: I'll look at it. [16:17] hatch: I don'pt want to get too 'test all the things' in one test [16:17] the extra asserts should be pretty trivial to add [16:17] bac: thats about the size of it, there will be a method you call with a container node and some bundle data and this will draw it [16:17] at some point in the future [16:17] hatch: but if they're not touching too many parts at once I'll add them [16:17] bcsaller: ok, then lets defer [16:17] fair enough [16:18] rick_h: yeah - my idea was that we just want to make sure the plumbing is hooked up :) [16:18] rick_h: I lgtm;d so you can submit whenever [16:27] afternoon luca [16:32] grabbing food, will have review day afterwards. [16:35] hatch: Hi Jeff [16:36] hatch: How's it going? [16:36] it's going good, and you? [16:49] bcsaller: in my hacking on this databinding stuff I've found some very odd bugs....like where we will have 80+ bindings in `this._bindings` [16:49] but I can't find out how to repro them :/ [16:49] so I just wanted to make you aware [16:49] hatch: I'm good :) [16:50] hatch: remember that we bind each node with a valueChange, not just the container, so its possible for something with many config values to see a lot, but 80 does sound high [16:50] yeah the one that was 88 had 32 bindings the next time [16:51] so that system needs a refactor....or something [16:51] but we already knew that [16:55] I'm just surprised it has no negative effects haha === schwuk is now known as schwuk_away [17:46] hatch: I was thinking about it but haven't verified this yet. I think the fillSlot stuff might bind again because the DOM for the slot can be re-rendered. It might be rebinding the whole set at that point w/o clearing the list first. If thats the case it would take some thought to fix because clearing the list would break the contract (bindings can store state as this.whatever). [17:47] But I think that might be the issue and we could manually clear the old bindings for the inspector in the slot being filled [17:48] yeah there is a UI repercussion of this in which the unit_count does not get updated when opening the inspector [17:48] so we will have to fix this before we release off flag [17:48] jcsackett: qusetion in bound with my LGTM though [17:49] bcsaller: sorry just trying to do something else atm so I haven't put any thought into it [17:50] I was just trying to think how it could happen [17:50] yeah it happens probably 50% of the time [17:50] so the inconsistency is my biggest worry [18:00] I was so totally just burnt by loose typing js [18:01] ouch it hurts [18:01] don't do that [18:10] this is my favorite docstring ever: [18:10] """Ensure the future is done and it contains the expected results.""" [18:12] bcsaller: should this not trigger the update function? app.env.add_unit('mediawiki', 10) - it adds the units into the db and the status bar ... [18:12] that's pretty good [18:13] hatch: which update method, the bindings.unit_count.update one? it should happen after the delta comes back [18:14] yeah [18:14] it's definitely not [18:14] :/ [18:14] at least I'm not goin nuts [18:14] in the console try a change listener on the value and do it manually [18:15] sory I don't follow [18:15] guichat? [18:15] taken [18:16] ring ring [18:21] bcsaller: review inbound, let me know if I'm missing stuff and we can chat. [18:21] rick_h: thanks, on a call now [18:22] bac, abentley, sinzui: ready to finish our discussion? [18:22] benji: Sure. [18:22] * sinzui finds drink [18:23] * bac does same [18:27] benji, abentley, sinzui: the doc is at https://docs.google.com/a/canonical.com/document/d/17bgbReU6JJMoUHSJeo8egG5zoSM0fpPIs7U8F1-Piyk -- in case i cannot reopen it inside the hangout [19:18] benji, you might recall my nagios branch. I have a separate merge into the unstable charm where I resolved conflicts. Do you have time to review it? https://codereview.appspot.com/12942043 [19:18] sinzui: my pleasure [19:31] sinzui: oh, I forgot to mention that since the shell script uses [[ you don't have to quote the variable to avoid expansion [19:33] oh yes. Thanks benji. [19:44] sinzui: Could you please review https://code.launchpad.net/~abentley/charmworld/versioned-ids-for-charms/+merge/180225 ? [19:49] sinzui: Sorry about the size. It's gotten hard to break out bits. [19:53] * sinzui looks [20:00] nice post Makyo [20:00] Just figured i'd brag :) [20:05] isn't that what blogs are for? [20:13] Yep! [20:15] jujugui has anyone started on any of the IE cards yet? [20:15] hatch, doing investigation of the viewBox attribute now. [20:16] kewl [20:56] abentley, sorry the the delay. I replied with a question. [20:58] sinzui: No worries. This branch took weeks to finish, so an hour reviewing it is nothing :-) [21:07] a [21:07] bcsaller: if user is looking at the inspector and someone else ups the units...should it up the input as well? [21:08] I'm thinking yes [21:08] that inputs story is a little confusing [21:15] bcsaller: another bug - when I deploy a service then open the inspector fast - the line that we changed in model.js var sum = service.get('units').size(); throws an error that service.get('units') is undefined [21:16] I lied, it also happens when deploying a ghost from dragging [21:16] I can repro it 100% of the time now so I"ll make a ticket [21:28] uh oh someone broke ci!!!!! [21:34] hatch: yes to the first one. for the second that list is added on the first delta of a unit of that service I think. We can see the other case when the binding is setup so it would be units = service.get('units'); sum = units && units.size() || 0; [21:34] https://bugs.launchpad.net/juju-gui/+bug/1212443 [21:34] <_mup_> Bug #1212443: Drag to deploy to soon after pageload and throws console error [21:34] ahh that makes sense [21:40] The CI looks like it might be a saucelabs issue, might just queue another and see what happens [22:22] bcsaller: looks like you were right :) [22:40] jujugui has anyone started on adding constraints to the ghost? [22:45] jcsackett: kickin around? [22:58] Morning [23:11] morning huwshimi [23:11] what are you workin on today? [23:12] hatch: Finishing off some IE10 stuff [23:13] great - so you were saying that the browser shifting things left you wanted to pass off? [23:13] * hazmat starts playing the story of the ghost in the background [23:14] hatch: Well, at this point I haven't figured it out. I left a note on the bug from what I discovered but I haven't managed to figure out the real cause. If someone else wants to take a look that'd be great :) [23:14] hazmat: diggin a little Phish at EOD? :) [23:14] hatch, indeed [23:14] hatch, no constraints though ;-) [23:15] huwshimi: ok cool - I've been stuck in service constraint and databinding land - it seems like everything I've had to touch is broken lol [23:15] so I want to make sure we can get this stuff delievered on time [23:15] hazmat: :)