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