/srv/irclogs.ubuntu.com/2012/06/26/#juju-dev.txt

fwereadeniemeyer, tyvm for reviews, good comments00:28
* fwereade looks at time, goes to bed00:28
niemeyerfwereade: My pleasure, good stuff too.. both seem almost mergeable00:28
niemeyerfwereade: Have a good night!00:28
fwereadeniemeyer, cheers :)00:28
Aramgood night00:38
niemeyerOkay, I'll get some food and head to bed.. looks like we'll have a meeting at 4AM (ouch)05:22
=== nijaba_ is now known as nijaba
=== meetingology` is now known as meetingology
TheMueMorning09:03
fwereadeTheMue, heyhey09:04
fwereadeTheMue, was just wondering about FlagWatcher... is it ok to assume that watched nodes will never have their content changed?10:16
TheMuefwereade: Have to look if it fires by accident. It is intended to just recognize if a node is created or removed. It first has been the exposed watcher, but niemeyer then recommended to name it flag watcher to be more flexible.10:25
TheMuefwereade: Btw, the ServiceUnitsWatcher is for review at https://codereview.appspot.com/6325062. Your work helped a lot.10:26
fwereadeTheMue, I think it'll fire with every content change, won't it?10:26
fwereadeTheMue, sweet10:26
TheMuefwereade: It uses the content watcher, so yes, it may be.10:28
TheMuefwereade: Hmm, even if it is not intended to be used with node with content it's better to change it. Thx for the hint.10:30
fwereadeTheMue, cool, np10:30
TheMuedavecheney: Hi10:37
davecheneyTheMue: howdyu10:38
TheMuedavecheney: Howdyu? If it's the abbreviation I assume: fine. ;)10:38
fwereadedavecheney, heyhey10:40
TheMuefwereade: The fix is in: https://codereview.appspot.com/633605810:57
fwereadeTheMue, I feel it needs a test :10:59
fwereade:)10:59
Aramhi.10:59
Arammeeting in one minute?10:59
davecheneyyy10:59
TheMueAram: Think so.10:59
Aramwhere? :).11:01
TheMuefwereade: OK, will add one to be sure.11:01
davecheneyAram: g+ ?11:01
Aramok11:01
Aramhi niemeyer11:01
fwereadeAram, niemeyer: heyhey11:01
TheMuedavecheney: Yes G+11:01
davecheneysup!11:01
niemeyerHeya11:01
niemeyerIs it party time yet?11:01
TheMueniemeyer: Hi11:02
TheMueniemeyer: Only waiting for mark to send the invitation.11:02
niemeyerWhere's Mark?11:03
TheMueniemeyer: Not yet seen *sigh*11:03
niemeyerOkay, I'll kick it off then11:03
TheMueniemeyer: Thx11:03
davecheneysigh11:04
davecheneysigned in with the wrong account11:04
davecheneytwo secs11:04
niemeyerdavecheney: I can invite the other one two11:05
niemeyerdavecheney: If you want11:05
davecheneynah11:05
davecheneyjust logging in now11:05
niemeyerAram, William: Invite is out11:06
fwereadeniemeyer, cheers11:06
mrammniemeyer: Aram: TheMue: fwereade:   good morning/afternoon/evening all11:20
fwereademramm, heyhey, come join us on g+11:20
niemeyermramm: Heya11:21
mrammcan somebody throw me the hangout link?11:22
Aramlaptop overheated11:23
niemeyermramm: Done11:24
niemeyermramm: https://plus.google.com/hangouts/_/02365fc5d5c685631f4c5d7068a7bab6c7e9954311:24
niemeyerdavecheney: https://codereview.appspot.com/6333056/11:45
davecheneyniemeyer: yes, i was looking at that one11:45
niemeyerdavecheney: I haven't looked, so I have no idea about what's there, but that's what rog had to say:11:46
niemeyerJun 22 13:39:56 <rog>   niemeyer: i've got a preliminary CL that starts the provisioning agent, but i haven't done the tests yet.        https://codereview.appspot.com/6333056/11:46
niemeyerJun 22 13:40:10 <rog>   niemeyer: someone might want to take it on while i'm away11:46
davecheneyi'll make it work11:46
davecheneyniemeyer: good luck with your presentation11:46
davecheneyi'm going to a live screening in AU at 2am I think thursday morning11:47
davecheneym_3 is in town, i'll bring him along11:47
niemeyerdavecheney: oh, nice11:47
niemeyerdavecheney: The one I'll be in is early friday11:48
davecheneyhmm, is that the 2nd day or the third day11:48
niemeyerdavecheney: Third11:48
davecheneyi'll have to catch it on video11:48
davecheneyi doubt the owner of the cafe the local GDC group have taken over will put up with us for 3 days11:49
niemeyerdavecheney: hehe :)11:49
davecheneyniemeyer: good luck, what a coup!11:49
niemeyerdavecheney: THank you!11:49
niemeyerOkay, I need to get some more rest, or they'll kick me out when they see I'm unable to speak at all :)11:49
niemeyerSee you all later!11:50
TheMueniemeyer: Good night11:50
TheMueLunchtime11:59
TheMuefwereade: Thx for review.15:05
fwereadeTheMue, yw, hope it's helpful15:06
Aramhm?15:49
Aramerror: Failed to create merge proposal: Server returned 405 and no body.15:49
Aramfrom lbox15:49
Aramwho's fault? :).15:49
fwereadeAram, probably lp, give it a sec and try again15:55
Aramok15:56
fwereadegn all, I have quite the stack of reviews if anyone's of a mind... https://code.launchpad.net/juju-core/+activereviews16:05
Aramhave a good one16:07
Arammeh, lbox works for other branches16:07
Aramdamn16:07
=== jamespage is now known as hazmat_mk2
=== hazmat_mk2 is now known as jamespage
* Aram goes to the store to buy some tea.16:25
niemeyerHi all!18:13
Aramhi again niemeyer.18:17
Aramniemeyer: I have a problem with lbox18:17
Aram"error: Failed to create merge proposal: Server returned 405 and no body."18:17
AramI get this error on one branch, but not the others.18:18
niemeyerAram: Hmm18:20
niemeyerAram: Can you please post the "bzr info" for that one branch?18:21
Aramsure18:21
Aramniemeyer: http://paste.ubuntu.com/1061229/18:22
niemeyerAram: Hmm.. seems alright18:23
niemeyerAram: Can you please try the same operation with -debug?18:23
Aramlbox pr bzr?18:23
Aramlbox or bzr?18:24
niemeyerAram: lbox propose -debug18:24
Aramyes18:25
Aramniemeyer: http://paste.ubuntu.com/1061241/18:25
niemeyerAram: I think this is my fault18:27
niemeyerAram: lpad recently broke due to an API change introduced in Go itself18:27
niemeyerAram: Looks like I have done something wrong while working around the issue18:28
niemeyerAram: Looking right now18:28
Aramright18:28
niemeyerAram: Can you please confirm the version of lbox you have installed (dpkg -l lbox)18:40
Aramniemeyer:18:41
Aramwhite:mstate$ dpkg -l lbox18:41
AramDesired=Unknown/Install/Remove/Purge/Hold18:41
Aram| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend18:41
Aram|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)18:41
Aram||/ Name           Version        Description18:41
Aram+++-==============-==============-============================================18:41
Aramii  lbox           1.0-51.62.38.1 The Launchpad Toolbox18:41
niemeyerAram: Cheers18:41
niemeyerI'm having some trouble trying to figure how this request could possibly be done with a POST18:43
niemeyerI suspect I'm about to understand something curious on the http package18:46
niemeyerOr about my own logic, of course18:48
niemeyerAram: I know what's wrong.. just adding a test and will have a working version19:21
Aramgreat19:21
niemeyerBuilding..19:33
Aramniemeyer: hmm... I see new versions on launchpad, but apt-get update isn't picking them, is there some kind of delay/cache?20:00
Aramah, the amd64 package still needs building20:03
Aram4 minutes20:03
Aramor not :)20:35
niemeyerAram: Yeah20:36
niemeyerAram: It's super slow for whatever reason20:36
niemeyerAram: lbox should be good20:57
niemeyerAram: Can you please see if it solves your problem?20:57
niemeyerI'm going to grab some food if so20:57
Aramyes20:57
Aramchecking now20:57
Arammeh, still don't see the update.20:59
AramI can see it built on launchpad20:59
Arambut apt-get doesn't pick it yet20:59
Aramokay, updating now21:03
Aramtesting...21:04
Aramniemeyer: works, thanks.21:06
niemeyerAram: Woohay21:06
niemeyerOkay21:20
niemeyerLunch has gone by already.. will find something reasonable outside21:20
Aramoh man, hope you didn't miss lunch because of fixing lbox :).22:45
Aramadded two reviews to the queue22:45
Arammorning davecheney22:47
davecheneymorning!22:47
niemeyerAram: Thanks!  Nah, was just excited pushing stuff forward :)23:02

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