/srv/irclogs.ubuntu.com/2012/04/16/#juju-dev.txt

=== flaviamissi_ is now known as flaviamissi
fwereade_mornings06:46
TheMuemorning wrtp08:29
wrtpTheMue: hiya08:30
TheMuewrtp: would you like to review a checkin?08:31
TheMuewrtp: it's the https://codereview.appspot.com/6011047/08:34
wrtpTheMue: ok, will have a look08:42
TheMuewrtp: great, thx08:42
wrtpTheMue: review delivered11:43
TheMuewrtp: ah, fine, thx11:47
niemeyerMorning everybody12:45
TheMueniemeyer: moin12:47
niemeyerTheMue: Heya12:49
wrtpniemeyer: yo!13:02
wrtpniemeyer, TheMue, fwereade__: are we having a meeting now?13:03
TheMuewrtp: i'm prepared13:03
niemeyerwrtp: Heya13:03
niemeyerwrtp, TheMue, fwereade__: Let's do it then13:03
fwereade__cool13:04
wrtpniemeyer: you doing the invites?13:04
niemeyerwrtp: Yep13:05
fwereade__niemeyer, it was there for a moment; was empty on join, and then disappeared13:08
niemeyerfwereade__: I'm online with Frank already13:08
wrtpniemeyer: i haven't seen anything yet13:08
niemeyerfwereade__: Try looking at my profile13:08
niemeyerwrtp: Ditto13:08
fwereade__niemeyer, still nothing :/13:09
wrtpniemeyer: i am13:09
wrtpniemeyer: canonical profile or your normal one?13:09
niemeyerwrtp: My normal one13:09
niemeyerwrtp, fwereade__: Sent the invite once more13:10
wrtpniemeyer: assuming this is the right one (https://plus.google.com/107994348420168435683/posts) i still see nothing13:10
wrtpniemeyer: ah, found it by going direct to Hangouts13:11
TheMueniemeyer: shall i start here https://wiki.canonical.com/UbuntuEngineering/Server/Squads/Juju ? it's a very empty page so far14:31
niemeyerTheMue: No, start here please: juju.ubutu.com/Roadmap14:32
TheMueniemeyer: ok, thx, will do so14:32
niemeyerTheMue: Or here, even: juju.ubuntu.com/GoPort14:32
TheMueniemeyer: yip, and i'll see how ot can be easily maintainable14:34
TheMueniemeyer: 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.1014:35
niemeyerTheMue: Sure, please make sure it stays up-to-date, though14:35
niemeyerTheMue: I've seen many such documents become a snapshot of their creation time14:36
TheMueniemeyer: 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
niemeyerTheMue: Yep, as long as you have a plan on how to maintain that up-to-date, sounds great14:38
TheMueniemeyer: i would do it as a part of our weeklys14:38
TheMueniemeyer: as you said with those thre (scrum) statements: what's done, what's planned, what's blocking14:39
TheMueniemeyer: and that could be immediately aligned with the little doc14:39
TheMueniemeyer: thankfully the risk of new feature requests is relative low (port python is pretty clear)14:40
TheMueniemeyer: i've seen enough projects starving due to get more things to do over time than they can be handled14:41
TheMueniemeyer: 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
niemeyerTheMue: Sounds good, will have a look, possibly after lunch since Ale has a sharp timing today14:46
TheMueniemeyer: enjoy your meal14:48
niemeyerTheMue: Thanks14:50
TheMuewrtp: thx for review again, i think i'll give a changed update() a try and check how it'll look16:04
wrtpTheMue: np16:05
fwereade__need to be off sharpish today; so, good evenings all!16:06
wrtpfwereade__: have a good evening, cheerio16:07
robbiewwrtp: 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
wrtprobbiew: yes, if that's ok - i did it before i knew when uds was, i think.16:09
wrtprobbiew: i don't quite know what the policy is w.r.t. national holidays during company events16:10
robbiewwrtp: cool..yeah, just swap it for sometime before or later16:11
niemeyerfwereade__: Cheers man16:15
TheMuerobbiew: oh, i just forgot that national holidays have to be submitted. we've one on may, 1st16:15
TheMuefwereade__: have a nice evening16:15
robbiewTheMue: it's no biggie16:16
TheMuerobbiew: but it has been a good reminder. ;)16:16
robbiew:)16:16
TheMuerobbiew: had done a little press work again: http://www.pressebox.de/pressemeldungen/canonical/boxid/500212 (this time for AWSOME)16:18
robbiewnice16:20
niemeyerTheMue: As a hint, there are 4 different branches on that one code review16:53
niemeyerTheMue: Changes to the watcher package, and three different watcher types16:54
niemeyerTheMue: If you want to collaborate with the review queue issue, a great way is to avoid aggregating such changes in a single large change set16:54
niemeyerTheMue: Helps you, helps the reviewers16:54
TheMueniemeyer: yip, started smaller but grown too fast. will control it better next time16:55
wrtpTheMue: you should probably know that "yip" in english usually means the bark of a small dog :-)16:56
niemeyerTheMue: In fact, can you please break it down?16:56
niemeyerTheMue: 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 today16:58
wrtpniemeyer: one review delivered16:58
niemeyerwrtp: Right, he can easily address those points while breaking it down into smaller chunks16:59
niemeyerwrtp: TheMue was very concerned today with the review queue size, and I agree.. let's start some action towards making it more manageable16:59
wrtpniemeyer: i was talking about a review of your code :-)16:59
niemeyerwrtp: Oh, woohay!16:59
niemeyerwrtp: Thanks ;-)16:59
niemeyerOh, crap, meeting time16:59
wrtpniemeyer: FWIW i thought TheMue's review wasn't too bad.16:59
niemeyerrobbiew: are you joining too?16:59
wrtpniemeyer: it consisted of related changes, easy enough to understand together17:00
robbiewniemeyer: it's cancelled17:00
* wrtp has to go. see y'all tomorrow17:00
niemeyerrobbiew: Oh?>17:00
robbiewapparently no one thought it was important to tell us17:00
robbiewlol17:00
robbiewmost folks are at ODS17:00
niemeyerHeh :(17:01
TheMueniemeyer: will see how i can split it. but for today i'm off.17:02
wrtpniemeyer: 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
TheMuewrtp: thanks for the yip-hint17:02
niemeyerTheMue: If you're concerned with the review queue size, smaller branches are a *huge* help..17:03
wrtpTheMue: np. "yup" is more convention BTW.17:03
TheMuewrtp: hehe, ok17:03
niemeyerwrtp: I don't understand what you mean17:04
niemeyerwrtp: How's one thing related to the other?17:04
wrtpniemeyer: take it back, i'm not sure it helps17:04
wrtpniemeyer: 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
wrtpanyway, gotta go17:06
niemeyerwrtp: Cheers17:06
niemeyerIt's not clear to me why an ExistenceWatcher is watching the content too..17:11
niemeyerAlso, what happens if the content changes?17:12
niemeyerhazmat: ping17:15
hazmatniemeyer, pong17:15
niemeyerhazmat: Yo17:15
niemeyerhazmat: Was just trying to remember what was that non-expected case of watchers in zookeeper17:16
hazmatniemeyer, how so.. exist watches track all events17:16
niemeyerhazmat: is it existsw that watches for content changes too?17:16
hazmatyes17:16
niemeyerhazmat: Aha, ok, cheers17:16
hazmatthe only sig difference between them is that you can set an exists watch on a non existant node17:18
hazmatthem = get vs exists17:20
niemeyerhazmat: Ah, right17:20
niemeyerhazmat: What about the opposite.. do you recall if a getw gets a sig on removal?17:21
niemeyerTheMue: Btw, I think ExistenceWatcher should be merged on ContentWatcher..17:22
hazmatniemeyer, it does17:25
hazmatswitching sessions/rooms bbiam17:26
niemeyermthaddon: ping18:21
m_3hey gang... FYI20:01
m_312:59 <SpamapS> Ok, folks PRECISE is the new dev focus for charms.20:01
m_313:00 <SpamapS> charm promulgate will now promulgate to precise20:01
m_313:00 <SpamapS> lp:charms/* is now precise20:01
m_313:00 <SpamapS> all branches have been copied forward20:01
m_313:00 <SpamapS> oneiric, may ye rest in stability.. aaaahhhmmeeen20:01
=== marrusl_ is now known as marrusl

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!