=== flaviamissi_ is now known as flaviamissi [06:46] mornings [08:29] morning wrtp [08:30] TheMue: hiya [08:31] wrtp: would you like to review a checkin? [08:34] wrtp: it's the https://codereview.appspot.com/6011047/ [08:42] TheMue: ok, will have a look [08:42] wrtp: great, thx [11:43] TheMue: review delivered [11:47] wrtp: ah, fine, thx [12:45] Morning everybody [12:47] niemeyer: moin [12:49] TheMue: Heya [13:02] niemeyer: yo! [13:03] niemeyer, TheMue, fwereade__: are we having a meeting now? [13:03] wrtp: i'm prepared [13:03] wrtp: Heya [13:03] wrtp, TheMue, fwereade__: Let's do it then [13:04] cool [13:04] niemeyer: you doing the invites? [13:05] wrtp: Yep [13:08] niemeyer, it was there for a moment; was empty on join, and then disappeared [13:08] fwereade__: I'm online with Frank already [13:08] niemeyer: i haven't seen anything yet [13:08] fwereade__: Try looking at my profile [13:08] wrtp: Ditto [13:09] niemeyer, still nothing :/ [13:09] niemeyer: i am [13:09] niemeyer: canonical profile or your normal one? [13:09] wrtp: My normal one [13:10] wrtp, fwereade__: Sent the invite once more [13:10] niemeyer: assuming this is the right one (https://plus.google.com/107994348420168435683/posts) i still see nothing [13:11] niemeyer: ah, found it by going direct to Hangouts [14:31] niemeyer: shall i start here https://wiki.canonical.com/UbuntuEngineering/Server/Squads/Juju ? it's a very empty page so far [14:32] TheMue: No, start here please: juju.ubutu.com/Roadmap [14:32] niemeyer: ok, thx, will do so [14:32] TheMue: Or here, even: juju.ubuntu.com/GoPort [14:34] niemeyer: yip, and i'll see how ot can be easily maintainable [14:35] niemeyer: pls don't get me wrong, i don't want a huge doc, i only want to take care the all of us can quickly get an overview on what's already done, what we do and what's left until 12.10 [14:35] TheMue: Sure, please make sure it stays up-to-date, though [14:36] TheMue: I've seen many such documents become a snapshot of their creation time [14:37] niemeyer: oh yes, i know what you mean and i share your headaches. but i've also often seen how important such a simple view could be (a kind of kanban view) [14:38] TheMue: Yep, as long as you have a plan on how to maintain that up-to-date, sounds great [14:38] niemeyer: i would do it as a part of our weeklys [14:39] niemeyer: as you said with those thre (scrum) statements: what's done, what's planned, what's blocking [14:39] niemeyer: and that could be immediately aligned with the little doc [14:40] niemeyer: thankfully the risk of new feature requests is relative low (port python is pretty clear) [14:41] niemeyer: i've seen enough projects starving due to get more things to do over time than they can be handled [14:44] niemeyer: btw, the branch to review i told about is https://codereview.appspot.com/6011047/. roger already made a review, so with your comments it should be done fast (at least i hope so *smile*) [14:46] TheMue: Sounds good, will have a look, possibly after lunch since Ale has a sharp timing today [14:48] niemeyer: enjoy your meal [14:50] TheMue: Thanks [16:04] wrtp: thx for review again, i think i'll give a changed update() a try and check how it'll look [16:05] TheMue: np [16:06] need to be off sharpish today; so, good evenings all! [16:07] fwereade__: have a good evening, cheerio [16:08] wrtp: ping...just noticed you submitted a nat'l holiday for May 5th (monday), however we'll be at UDS that week...were you planning on swapping that later? [16:09] robbiew: yes, if that's ok - i did it before i knew when uds was, i think. [16:10] robbiew: i don't quite know what the policy is w.r.t. national holidays during company events [16:11] wrtp: cool..yeah, just swap it for sometime before or later [16:15] fwereade__: Cheers man [16:15] robbiew: oh, i just forgot that national holidays have to be submitted. we've one on may, 1st [16:15] fwereade__: have a nice evening [16:16] TheMue: it's no biggie [16:16] robbiew: but it has been a good reminder. ;) [16:16] :) [16:18] robbiew: had done a little press work again: http://www.pressebox.de/pressemeldungen/canonical/boxid/500212 (this time for AWSOME) [16:20] nice [16:53] TheMue: As a hint, there are 4 different branches on that one code review [16:54] TheMue: Changes to the watcher package, and three different watcher types [16:54] TheMue: If you want to collaborate with the review queue issue, a great way is to avoid aggregating such changes in a single large change set [16:54] TheMue: Helps you, helps the reviewers [16:55] niemeyer: yip, started smaller but grown too fast. will control it better next time [16:56] TheMue: you should probably know that "yip" in english usually means the bark of a small dog :-) [16:56] TheMue: In fact, can you please break it down? [16:58] TheMue: I won't be able to provide you a full review in time for you to do anything with it today.. breaking this into smaller chunks will be time well spent today [16:58] niemeyer: one review delivered [16:59] wrtp: Right, he can easily address those points while breaking it down into smaller chunks [16:59] wrtp: TheMue was very concerned today with the review queue size, and I agree.. let's start some action towards making it more manageable [16:59] niemeyer: i was talking about a review of your code :-) [16:59] wrtp: Oh, woohay! [16:59] wrtp: Thanks ;-) [16:59] Oh, crap, meeting time [16:59] niemeyer: FWIW i thought TheMue's review wasn't too bad. [16:59] robbiew: are you joining too? [17:00] niemeyer: it consisted of related changes, easy enough to understand together [17:00] niemeyer: it's cancelled [17:00] * wrtp has to go. see y'all tomorrow [17:00] robbiew: Oh?> [17:00] apparently no one thought it was important to tell us [17:00] lol [17:00] most folks are at ODS [17:01] Heh :( [17:02] niemeyer: will see how i can split it. but for today i'm off. [17:02] niemeyer: rather than caching the key->keyid map, might it be better to have a Counter method on store that returned an object that could be used to inc and dec a particular counter. [17:02] ? [17:02] wrtp: thanks for the yip-hint [17:03] TheMue: If you're concerned with the review queue size, smaller branches are a *huge* help.. [17:03] TheMue: np. "yup" is more convention BTW. [17:03] wrtp: hehe, ok [17:04] wrtp: I don't understand what you mean [17:04] wrtp: How's one thing related to the other? [17:04] niemeyer: take it back, i'm not sure it helps [17:05] niemeyer: i thought it would be easy to push the cache back into the store server. but it's dynamically creating keys all the time, so it won't help. [17:06] anyway, gotta go [17:06] wrtp: Cheers [17:11] It's not clear to me why an ExistenceWatcher is watching the content too.. [17:12] Also, what happens if the content changes? [17:15] hazmat: ping [17:15] niemeyer, pong [17:15] hazmat: Yo [17:16] hazmat: Was just trying to remember what was that non-expected case of watchers in zookeeper [17:16] niemeyer, how so.. exist watches track all events [17:16] hazmat: is it existsw that watches for content changes too? [17:16] yes [17:16] hazmat: Aha, ok, cheers [17:18] the only sig difference between them is that you can set an exists watch on a non existant node [17:20] them = get vs exists [17:20] hazmat: Ah, right [17:21] hazmat: What about the opposite.. do you recall if a getw gets a sig on removal? [17:22] TheMue: Btw, I think ExistenceWatcher should be merged on ContentWatcher.. [17:25] niemeyer, it does [17:26] switching sessions/rooms bbiam [18:21] mthaddon: ping [20:01] hey gang... FYI [20:01] 12:59 Ok, folks PRECISE is the new dev focus for charms. [20:01] 13:00 charm promulgate will now promulgate to precise [20:01] 13:00 lp:charms/* is now precise [20:01] 13:00 all branches have been copied forward [20:01] 13:00 oneiric, may ye rest in stability.. aaaahhhmmeeen === marrusl_ is now known as marrusl