/srv/irclogs.ubuntu.com/2013/09/12/#juju-gui.txt

hatchgary_poster: blog post is up http://fromanegg.com/post/60978144670/juju-gui-0-9-0-released-now-with-ie10-support00:22
gary_posterawesome hatch, thanks!  the new charm search doesn't suck either, btw, if you are looking for features. :-)00:23
hatchright I forgot about that oops :) logging in to add that00:24
hatchupdated00:27
hatchOO boy saucy is fast compared to 12.0400:28
hatcheven in this vm, I wish I could run it on my laptop!00:28
=== frankban__ is now known as frankban
gary_posterhey frankban__ .  welcome back for two days. :-) I approved expenses.11:16
frankban__gary_poster: thanks!11:17
=== frankban__ is now known as frankban
frankbanthe connection here is not good at all :-/11:18
gary_poster:-/11:18
* frankban lunches12:26
hatchmorning12:35
rick_h_morning12:35
hatchhave you guys been following the thread about the new email client?12:51
rick_h_hatch: somewhat12:51
hatchI think I have figured out why linux apps are ugly12:52
hatch:)12:52
rick_h_lol12:52
hatchhaha12:52
hatchso I installed saucy - still getting everything up and running......12:54
hatchbut as far as styling goes12:54
hatchthe top bar and the window controls don't match the rest of the UX12:54
bachatch: you using vmware?12:54
hatchparallels12:54
hatchbac: it pretty much worked 100% out of the box. I just had to install the parallels tools to get 3d acceleration and being able to move the mouse over the window to control it12:55
bachatch: are you able to have it start in the display size you want?  mine boots into the highest res which is unusable.  not a big issue since i don't reboot much but it is annoying12:56
hatchI run it in windowed mode12:56
bachatch: i'm using vwmare12:56
bacoh, ok12:56
hatchlemme see what it does in fs mode12:57
hatchit is the highest res but looks proper12:58
hatchI am not sure what the ppi of my monitor is12:59
gary_posterbac has super duper high Mac res :-)13:01
gary_posterrietveld is giving me "error: old chunk mismatch" on my review13:02
hatchgary_poster: you can try running it again13:02
gary_posterwhat can I do to fix it?  repropose?13:02
hatchsometimes that helps13:02
bacgary_poster: i've never understood why that happens13:02
gary_posterok trying, thanks13:02
hatchyeah I'm pretty sure my monitor isn't 'retina' 2560x1600@30"13:03
rick_h_gary_poster: I've had to ditch it and use a diff branch name before as well. So create a new colo-branch, merge in the old branch, push to a new branch name, repropose13:04
hatchI'm going to land huw's branch13:05
hatchit QA's ok13:05
gary_posterrick_h_, ack :-/13:08
gary_posterhatch, awesome thanks13:08
gary_posterreproposing did not help13:08
hatchrick_h_ brought up http://www.reviewboard.org/ not sure if we want to switch but it does look pretty cool :)13:10
rick_h_hatch: heh, well we don't have the scripts/tools for it, a hosted instance, etc13:10
hatchwe have the technology!!!!13:10
hatchat least I'd hope we could host an app :P13:10
hatchalthough -not- hosting/paying for something is pretty awesome13:11
hatcheven if it does break all the time13:11
hatchgary_poster: we can just review on lp13:11
gary_posterhatch, ack thanks.  Trying one more time: I made a change to comments, pushed to lp, and reproposed...13:12
rick_h_bac: hatch updates here https://codereview.appspot.com/1351604413:14
gary_posteryay, that worked13:14
rick_h_hatch: with some comments on the conflictux stuff13:14
hatchcool i'll take a look in a second13:17
rick_h_cool, had to sync with trunk and resolve a conflict so pushing up again13:17
hatchrick_h_: so you don't think that we should add the ability to skip the conflict in databinding? My thoughts were that, because we were parsing the html to add the databinding, it could make sense to also use the html parsing to turn off features...13:18
hatchre skipconflictus13:19
hatchux13:19
rick_h_hatch: no, databinding's job is to say "Hey, this changed". What you do on that change is someone else's job. 13:19
hatchI don't feel strongly about this13:19
rick_h_hatch: in this case conflict says if classes/etc get added13:19
hatchyeah I suppose that's reasonable13:19
rick_h_hatch: and long term we might want to handle something but not all of the conflictux, so I feel that's up to conflictux to decide13:19
hatchyeah - it feels like it could be standardized somehow and moving it into the databinding would do that13:20
rick_h_so just 'don't fire change' seems like breaking twigs with an axe13:20
rick_h_hatch: I agree that it could use some cleanup. I'd prefer to move the databinding stuff to it's own module and try to see if we could pull apart some of the bits (say...events from databinding that conflict ux could watch for)13:20
rick_h_but that's not for here/now13:20
rick_h_sorry, the conflictux stuff to its own module13:20
hatchyeah....because really I could see use for databinding with no conflict13:21
rick_h_hatch: 100%13:21
rick_h_hatch: did your relations tab headings make it in? /me isn't seeing it in a sync'd with trunk13:23
hatchrick_h_: Huw was working on the styling and I was sure I mentioned to him to add it so I didn't13:25
rick_h_hatch: ah, gotcha13:25
hatcheither I didn't or he forgot13:25
hatch:)13:25
hatchI will throw it in in the unit branch I'm working on now13:26
rick_h_all good, just was looking around and noticed it13:26
rick_h_don't have huw's changes yet so we'll see13:26
hatchoh ok, they should just be in trunk now13:26
rick_h_gary_poster: review/qa. One bug in there during qa13:48
gary_posterack thanks rick_h_ will look soon13:48
hatchthere is a hot air balloon above my house and my docs do not like it in their airspace13:52
rick_h_have to run up to Sec of State, brb14:04
hatchis the unit status bar supposed to be very thick now?14:06
bacgary_poster: are we 1:1 now?14:07
hatchin the inspector14:07
hatchit's the same height as the inputs now14:07
gary_posterbac yeah almost14:07
rick_h_back14:19
hatchfwereade_: hey have you had a chance to think about the relation error reporting?14:22
hatchI'm working on aggregating unit errors and boy would it be nice to not have to do all of this computing on it :D14:22
fwereade_hatch, ha, not really14:25
fwereade_hatch, I'm basically as far as I was when we talked14:25
hatchhaha alright - anything I can do to help move it forward?14:25
fwereade_hatch, well, IIRC, where we were was: setting some relation-and-unit-specific error state on a suitable hook failure, and resetting it when the hook completes properly, is all we really need; and that isn't actually very hard to do14:27
hatchright, basically now I have a unit agent_state in 'error' and a agent_state_info in 'hook failed: db-relation-changed'14:28
hatchit would be really nice if there was an additional object with a proper data set in it14:28
hatchagent_state_data - which could be an object of relevent data14:29
hatchlike - what it's related to, the actual interface name14:29
hatchright now I am parsing the agent_state_info and then have to do a few queries to our local db to figure out what relation that is14:30
fwereade_hatch, ok, that basically works for me I think14:31
fwereade_hatch, I'm not even sure how you'd do that without the relation id available14:31
hatchcan a unit be related to multiple other units on the same interface name?14:31
hatchcould this `hook failed: db-relation-changed` relate to two different services ..... theoretically14:32
fwereade_if your charm declares a foo relation, you can be in N foo relations with N other services, and in each of those you might be related to M units14:32
hatchright that's what I thought14:32
fwereade_hatch, yes it could14:32
fwereade_hatch, so you need relation id in there14:32
hatchso there is no way - atm - for me to really determine what the relation is14:32
fwereade_hatch, correct14:33
* hatch scraps current branch14:33
hatch:P14:33
fwereade_hatch, hell, sorry14:33
hatchhaha it's ok14:33
hatchwe can't have everything we want :)14:33
hatchso is it possible to include the relation id when the hook fails like that?14:33
hatchI haven't spent much time in core14:34
hatchpossible/logical14:34
fwereade_hatch, yeah, that info should be available trivially, just a mo14:34
fwereade_hatch, worker/uniter/modes.go:33714:35
fwereade_hatch, I would support improving the error message there14:35
fwereade_hatch, it actually looks like it's just "relation-changed", not even "db-relation-changed"14:36
fwereade_hatch, but I would be a little reluctant to recommend that anyone depend upon its precise form14:36
hatchyeah exactly :)14:36
fwereade_hatch, and would be happier with the ability to add agent-state-data in SetStatus et al14:37
hatchright that also allows us the ability to modify/expand on that data object as time goes on14:39
hatchwithout modifying the api14:39
gary_posterhey rick_h_, thank you for review and qa.  I can't dupe problem you reported: qa is good for me.  what browser?14:54
rick_h_gary_poster: chrome14:54
rick_h_gary_poster: will load it back up and check others14:54
gary_posterrick_h_, wfm on chrome.  maybe a quick hangout so I can see if you are doing something different than I am doing?14:55
hatchgary_poster: when you have a second we should have a real quick chat about the relation status stuff14:55
rick_h_gary_poster: sure thing, 14:55
gary_posterhatch, ack.  have a call in 5, but hopefully will be short14:55
hatchok no problem, ping when avail14:55
gary_posterrick_h_, real fast in https://plus.google.com/hangouts/_/919c85491c5ef7fdad3aa7c1cf805253ecec4fd2 ?14:56
gary_postertiny.cc/guichat looks dead again  :-/\14:57
rick_h_booo google14:58
rick_h_gary_poster: so in poking at it more, it's only once I set the background model to be the same value as the current value from the user15:03
rick_h_gary_poster: so I think it's something to do with clearing away the conflict marker/hiding the save button15:03
gary_posterah! ok thanks will try again when I get off call15:03
rick_h_gary_poster: rgr15:03
=== frankban_ is now known as frankban
gary_posterhatch, https://plus.google.com/hangouts/_/6817b2266c6d0aab3f53875991c2086f7f573f75 ?15:21
hatchgary_poster: omw15:21
hatchfwereade_: still around?15:23
hatchmind joining gary and I in this chat? ^15:24
fwereade_hatch, yes, more or less15:25
fwereade_hatch, 2 mins15:26
rick_h_bac: couple of comments on the ga branch15:39
bacthanks for the review rick_h.   do you mean just fetch the current ip address and stuff it in there?  i just wanted to mimic the output from charmworld b/c i used it all of the time.15:39
rick_h_bac: yea, on the server side I didn't know if there's an easy 'whatismyup' it could pull15:40
rick_h_bac: but that's wishful thinking. Just now it's *wrong* for some use cases, but I understand15:40
rick_h_bac: the second comment on the conditional though I think would be nice for less error/etc in the console 15:40
=== frankban_ is now known as frankban
bacrick_h_: yeah, getting the ip is easy.  i'll make that change.  it may still be wrong if there are multiple interfaces15:43
rick_h_bac: k, I think most virtualmachines would have one though? I guess on local devs with wireless/wired it would be off?15:43
bacjujugui: review of changes to the charm? https://code.launchpad.net/~bac/charms/precise/juju-gui/ga-key/+merge/18529415:44
bacrick_h_: what is the second comment you referred?15:45
rick_h_bac: while doing a quick QA I wondered if we could just skip the GA loading in dev with a hard coded GA code of devel (vs brad)15:45
bacrick_h_: on, nm, i see it now15:45
bacrick_h_: oh, i didn't mean to have 'brad' be included in the diff.  that was a mistake.15:46
bacrick_h_: since useAnalytics is false, that ga_key can be ''15:46
rick_h_bac: looking15:47
* bac busted for not reviewing my diff15:47
rick_h_bac: so I'm confused. why would it not be if (window.ga_id && juju_config.useAnalytics) ?15:49
rick_h_vs the check being down in window['ga-disable-...?15:49
rick_h_bac: so down in the bottom where we have the snippet to add the ga script, we could just not add it if useAnalytics is false?15:51
bacrick_h_: we could probably just get rid of useAnalytics and treat the existence of the key as the sentinel15:53
bacrick_h_: the old style was from when the key was hard-code and we just wanted to disable it15:54
rick_h_bac: k, gotcha15:54
gary_posterguichat call in 2 https://plus.google.com/hangouts/_/6817b2266c6d0aab3f53875991c2086f7f573f7515:58
benjijujugui ^^16:00
benjijuju-gui, even16:00
gary_posterthanks :-)16:00
hatchbcsaller:16:15
hatchblarg16:15
hatchbranch16:15
hatchahhh16:15
bcsalleryeah16:15
TheMuegary_poster: ping16:29
gary_posterHey TheMue . Thank you for being willing to help us out!16:30
TheMuegary_poster: sure, i filed a bug at https://bugs.launchpad.net/juju-core/+bug/122456816:30
_mup_Bug #1224568: Improve hook error reporting <juju-core:In Progress by themue> <https://launchpad.net/bugs/1224568>16:30
gary_posterTheMue, looks good, thank you.  May I add (6) make sure that we can correlate relation error identifiers with AllWatcher relation info?  I think we are using the tag as the AllWatcher relation identifier, in which case your plan sounds like exactly what we need, but I'd like to verify that16:32
gary_posterhatch, frankban ^^^ in case you have any input16:33
TheMuegary_poster: yep16:33
hatchthanks I'll take a look in a bit16:33
hatchTheMue: what timezone are you in?16:33
gary_posterCool, added.16:35
frankbangary_poster: with (6) it seems a good plan16:35
gary_postercool thanks frankban 16:35
TheMuehatch: european16:39
hatchalright just curious :)16:40
TheMuehatch: why?16:40
hatchso I don't send you a message and expect a response this afternoon haha16:40
TheMuehatch: hehe, yep16:41
hatch:)16:42
hatchgary_poster: any specific card you want me to get onto now?16:43
hatchTheMue: when you land this branch can you link it to the ticket so I can see the diff - I'll need to update our go sandbox to mirror the new format16:53
TheMuehatch: yes, will do16:58
TheMuehatch: will be several smaller branches16:58
hatchalright no problem thanks16:59
benjinew review up: https://code.launchpad.net/~benji/juju-gui/charmworld-api-3/+merge/18532517:13
rick_h_benji: heading to reitveld?17:14
rick_h_benji: looked at anyway, thanks for the new tests17:23
gary_posterhatch, bug 1223864? or move expose from config to destroy section, as (I think?) we discussed17:27
_mup_Bug #1223864: Conflict dropdown on constraints fields never shows in IE10 <ie10> <juju-gui:New> <https://launchpad.net/bugs/1223864>17:27
hatchgary_poster: sounds good17:28
gary_posterhey rick_h_ I still can't dupe the qa issues.  could we hangout one more time and have you walk me through it from the beginning (including loading up the gui)?  The only thing I can see is if you are overwriting existing config values when you change the model, in which case the behavior you describe is correct.17:36
rick_h_gary_poster: sure thing17:42
gary_posterrick_h_, https://plus.google.com/hangouts/_/7a006e48281d727daf2a759297722b00e3d9070d17:42
gary_posterthank you17:42
rick_h_gary_poster: cool, so then you need a second set of eyeballs right? hatch bcsaller17:55
gary_posteryeah thanks rick_h_ 17:56
gary_posterhttps://codereview.appspot.com/13340047/17:56
gary_posterjujugui, could someone give me a second review on that branch above ^^^ ?18:00
bcsallerchecking it now18:00
gary_posterthanks bcsaller18:02
hatchsorry didn't notice the pings18:02
bacrick_h_: on further review, i'm unsure how to get the local ip address in js.18:04
rick_h_bac: cool, nvm then. 18:04
rick_h_bac: thanks for trying. 18:05
bacrick_h_: +1 on your QA technique.  my script predates the 'deploy' target and i never updated18:10
rick_h_bac: cool, yea I like how easy it's gotten18:10
* hatch feels left out18:13
hatchwhat's this technique?18:13
bacso last night my fitbit told me i was short on my walking goal a distance about the same as a round-trip to ben & jerry's.18:13
hatchlol18:13
bachatch: just to use 'make deploy' with the charm instead of using a stone-age script18:14
rick_h_hatch: see https://code.launchpad.net/~bac/charms/precise/juju-gui/ga-key/+merge/18529418:14
rick_h_hatch: I made a comment on the other QA method that I got from frankban's branches18:14
hatchohh gotcha18:14
hatchpossibly update the hacking docs with that?18:14
benjirick_h_: thanks for the review (late reply because I was at lunch)18:16
benjirick_h_: darn, lbox was waiting on my google credentials this whole time, that's why no rietveld was forthcoming18:17
rick_h_benji: ah, gotcha. 18:17
rick_h_benji: well cool and thanks for the tests all the same18:17
benjirick_h_: ooh, thanks for the config catch; I was testing against a local charmworld18:17
rick_h_benji: yea, do the same all the time (well sometimes)18:18
benjiwe probably need a lint step to yell if the config changes, that seems like a common issue18:19
rick_h_yea, if only a 'are you sure you mean to change this' warning. but fortunately it usually sticks out in review18:22
bacrick_h_, hatch: should be noted 'make deploy' requires the env be bootstrapped already18:27
rick_h_bac: ah true. Missed that step in the notes18:27
hatchrick_h_: where were your zindex rules? I want to make sure I follow them :)18:53
rick_h_hatch: top of stylesheet.less18:53
hatchgot em thcx18:53
rick_h_hatch: if you would have thuoght to look in another place let me know, I went with what made more sense to my brain18:53
rick_h_hatch: but not married to the location18:53
hatchmakes as much sense as any I suppose - we just haven't held docs in code before18:54
hatchdo we have a place for 'dev rules' ?18:54
rick_h_somewhat, it might go into 'style guide' though that's more actual styule18:55
rick_h_but partly why I went with the code. I don't always look at the docs when I start going at something18:55
hatchI never do18:57
hatchlol18:57
hatchok that's not true18:57
rick_h_heh, you just did18:57
hatchI am restructuring all of that work you did on the expose button18:57
hatch:(18:58
rick_h_ok, running away. Have to prepare for the wife's birthday tomorrow. 18:58
hatchOOoo18:58
rick_h_hatch: works for me :P18:58
hatchenjoy!18:58
hatchhaha, it just needs to be moved - nothing you did wrong18:58
hatch2H on CSS and markup - now lets go see if IE will ruin my day19:10
hatchlooks like we can keep our IE10 support for another day19:12
hatch:P19:12
bacrick_h_: since your review i've pushed a new branch and the old diff is gone.  what did you mean by "#16 double space in there"19:52
hatchjujugui lf a review and QA (IE, Chrome) https://codereview.appspot.com/13684043/ review notes added20:05
hatchis there anyone still around with an IE VM?20:06
hatchI'd like to get this branch landed before EOD so that Huw can merge the styling and markup changes20:07
gary_posterhatch, I think you can claim the IE qa20:10
hatchalright I'll accept that :)20:10
gary_posterhatch, I don't think rick_h_ is around, and I'm becoming very skeptical of the idea of a boolean conflict20:12
gary_posterhatch, you up for talking it through with me?20:12
hatchsure thing20:12
gary_posterthx.  uh...20:12
gary_posterhatch https://plus.google.com/hangouts/_/30b2e694404bf3e9f509dc3a15c182908c78023120:13
jcastroman, the search autocomplete is pretty awesome20:18
jcastroI can see now why ale was so adamant about getting it in asap20:18
gary_poster:-)20:19
gary_posterthanks jcastro 20:19
jcastrorick_h_: if you've got time before EOD I have some questions as far as what I can get away with offlining with the GUI for saturday20:20
jcastrohttps://jujucharms.com/precise/mongodb-HEAD/20:26
jcastrothis work for anyone?20:26
jcastrothe mongodb guys just reported it to me20:26
jcastrogary_poster: ^^^20:30
jcastrosorry but the mongodb community manager has the finger on the tweet button and I need a sane URL20:30
gary_posterjcastro, https://jujucharms.com/precise/mongodb/20:30
jcastrohuh20:30
jcastroI tried that20:30
jcastrook ...20:30
* jcastro doesn't look a gift horse in the mouth20:31
gary_poster:-)20:31
jcastrohaving laura cz. as our mongo contact = awesome.20:31
gary_posterheh20:32
gary_posterbut yeah20:32
gary_posterwe now support better URLS, but the apache redirects are now broken :-(20:32
gary_posterhey sinzui, where/how do I tweak the jujucharms.com apache redirects?20:33
gary_posterhatch, in return for crushing my dreams and spirit, I mean helping me out to figure out what to do, I'm reviewing and qa'ing your branch. ;-)  code LGTM so far...20:39
hatchhahaha20:39
hatchthanks20:39
gary_posterhatch qa good.  fire when ready20:43
gary_posterthanks!20:43
hatchawesome thanks20:43
* sinzui looks20:51
benjigary_poster: is this what you're looking for? http://bazaar.launchpad.net/~juju-jitsu/charms/precise/charmworld/trunk/view/head:/scripts/configure-apache220:53
hatchjujugui somehow one of the branches landed today has a jslint error20:57
hatchhmm it was bac's branch20:58
gary_posterbenji, no, but thank you20:58
gary_posterhatch, may I have one more call with you?  can't be for long, but need to talk through20:58
hatchbenji: curious as to why your branch didn't go through lbox?20:58
hatchand brads for that matter20:58
hatchsure20:58
hatchcalling20:59
gary_posterhatch don't see you.  https://plus.google.com/hangouts/_/2c5782db2cd9582405c9a6208c501f47f9663063 ?20:59
benjihatch: mine went through lbox, just not rietveld.  I went to lunch while lbox was asking me for google credentials, so that part timed out and by the time I got back it had already been reviewed in LP.  I then used lbox to land it.21:00
hatchoh ok21:00
hatchI'm just trying to figure out why brads didn't get linted before landing21:00
rick_h_jcastro: what's up?21:10
rick_h_gary_poster: ruh roh...21:10
jcastrorick_h_: hey so, there's no interwebs at OLF as you know.21:11
jcastrobut I was thinking ...21:11
rick_h_jcastro: rgr21:11
jcastrodeploy in the hotel room with everything I need21:11
jcastrocheck everything21:11
jcastrothen walk downstairs ... should work?21:11
rick_h_jcastro: issue is that charmworld needs a bunch of time to ingest a ton of data 21:12
rick_h_jcastro: as that needs to be offline with you in your lxc environment21:12
jcastronot to live deploy21:12
jcastroI mean have it running and "laid out"21:12
jcastrobut show example commands21:12
jcastroand then be like "this is what it looks like"21:12
rick_h_so not going to touch the browser?21:12
jcastroright21:12
jcastrobut running ...21:12
rick_h_if you're careful what you do I guess yea21:13
jcastrolike, the gui doesn't flip out in X secs/minutes if it doesn't hit something right?21:13
rick_h_jcastro: right21:13
jcastrorick_h_: ok, so .... screenshots are probably better?21:13
jcastroI want to show awesome stuff, but not risk showing something broken21:13
rick_h_so if you can do 4 lxc's I'd think an lxc environment with charmworld, elasticsearch, mongodb, and the gui would be awesome21:13
jcastroyeah I thought about that21:14
rick_h_can't we shut down and reboot and get them abck up now?21:14
jcastrobut the thing is that workload is interesting to the people in this channel21:14
jcastroand no one else, heh21:14
rick_h_right, but you can stick them off the screen I guess21:14
jcastroyeah I can reboot, and I know it works, but man, I am too scared21:14
rick_h_hah, yea. yea. You should be good if you setup on wifi, deploy/etc. and then go offline21:14
jcastro"don't worry man, reboot your laptop, all 25 cassandra nodes will come up, trust me."21:15
rick_h_just don't touch anything that'll hit the charmworld, so you might lose icons, no browser, etc. 21:15
rick_h_search, autocomplete, etc21:15
rick_h_jcastro: but yea, try out what you want to do. Set it up, flip the wifi switch, try it out21:17
rick_h_if you hit something I can see if I can help work around it for you21:17
hatchrick_h_:  jcastro we should create a virtual box image that can be used for demos like this21:27
hatchwouldn't be too much work either I wouldn't think21:27
jcastroyeah21:27
jcastroI am debating how much to push the GUIs intent21:28
jcastroLike, I really want to say "You know what ... Opsworks looks awesome ... run your own, and _fuck that_."21:28
hatchlol21:32
hatchany status update on the node charm?21:32
hatchI'm going to write my own pretty soon21:32
jcastrowaiting to sign the deal with joyent last I heard? m_3 might have a better status21:33
m_3siik21:42
m_3hatch: just do it21:42
hatchugh trunk is so crazy broken right now21:43
hatchguess I won't be merging this in tonight21:44
hatchsounds like a plan21:58
gary_posterhatch, you can just back out the offending branch if that makes sense22:24
gary_posteryou don't need to be the one who has to fix it22:24
gary_posterthat said...tests were passing for me in my branch when I merged trunk, except for 1, IIRC22:24
hatchwas that after brads google analytics branch?22:25
hatchsorry I am trying to also get ready to go out for supper atm :)22:25
gary_posterhatch, np, if it's EoD it's EoD! :-)22:26
huwshimiMorning22:27
gary_posterhey huwshimi.  guichat is dead.  I added a video chat link to the calendar.  we can use that22:28
huwshimigary_poster: Ah cool. I'll join in a sec22:28
gary_postercool22:28
hatchhuwshimi: my branch (https://codereview.appspot.com/13684043/) moves around some markup and css22:29
hatchso you should keep that in mind if you're doing that sort of stuff today22:29
huwshimihatch: No problems, I'll take a look22:29
hatchI can't land it because of trunk but if you're working in those areas you might want to merge it in22:30
hatchand now i'm off for real this time22:30
hatch:)22:31
gary_posterhatch ttyl22:31
hatchcya22:31
huwshimigary_poster: Can I do anything to help unbreak trunk?23:12
gary_posterhuwshimi, is it blocking you?23:13
huwshimigary_poster: It's not, I'm just aware it's getting late over there for you :)23:13
gary_posterhuwshimi, :-) ok thank you.  how about I'll check in with you when I stop.  If I haven't merged my branch (likely) then I'll ask you to do the unbreaking?23:16
huwshimigary_poster: Sure!23:16
gary_posterthank you!23:17

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