/srv/irclogs.ubuntu.com/2013/11/27/#juju-gui.txt

=== gary_poster is now known as gary_poster|away
bacjujugui: i'm looking at huw's problem12:22
benjik12:22
=== gary_poster|away is now known as gary_poster
gary_posterthanks bac13:01
hatchmorning all14:05
rick_h_morning14:05
hatchgary_poster https://twitter.com/clarler/status/405565740700274688 schedule wise looks like I'm available 14:08
rick_h_woot, hatch on tv14:09
hatchlol14:10
hatchlike a baws14:10
gary_poster:-) hatch will lookin a sec; on call14:10
hatchbac are you planning on fixing huws branch or are we just leaving it to him?14:20
hatchI could also take over if needed14:20
bachatch: if i had time this afternoon i was going to look at it14:21
hatchcool, lemme know if you run out of time14:21
* frankban run the charm tests in ~25mins \o/ , but the legacy server header test fails :-/14:34
gary_posterhatch cool about roundtable.  +1, for you and whoever wants to show up.14:48
hatcheggcellent, now I just need to figure out what to talk about14:49
hatchjujugui call in 1015:50
gary_posterty15:50
* rick_h_ runs 15:51
gary_posterhey frankban, I'm about to qa the quickstart change to support the authtokens.  Could you take a quick look over the diff to make sure it looks alright before I do so?  http://paste.ubuntu.com/6484972/15:51
frankbangary_poster: sure15:51
gary_posterty15:51
gary_postermaybe I should tell the user if we have a token or not...they will see effect, but maybe that will ease surprise?15:54
gary_posters/ease/reduce/15:54
frankbangary_poster: can we stay for five minutes after the call? I have a comment on the diff and an unrelated question 15:55
gary_posterfrankban, sure15:55
frankbangary_poster: thank you15:55
Makyojujugui call in 215:58
gary_posterbenji yoo hoo?16:01
benjigary_poster: coming16:01
bacgary_poster: i have no commercial vouchers available for private projects on LP16:11
gary_posterbac, darn, but thanks.  rick_h_ ^^^ wgrant maybe?16:13
rick_h_gary_poster: yea, will have to catch him one night16:13
bacif only there were an asynchronous messaging system...16:13
Makyofrankban, Would like to go over tasks when you get a chance.  Can make all the cards and see about starting a small thing, but I'm out thurs/fri, don't want to leave too much hanging.16:24
gary_posterfrankban, https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj016:41
gary_posterMakyo, frankban would be great if frankban can work on it on Thurs/Fri16:41
MakyoYeah16:41
gary_postersounds like that's what you mean16:41
gary_postercool16:41
frankbangary_poster, Makyo: sounds good, I can start prototyping something16:47
gary_postercool16:48
frankbangary_poster or Makyo: call?16:48
gary_posterfrankban,  is ready for you :-)16:48
frankbancool joining16:48
gary_posterhttps://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj016:48
hatchbenji any idea how I do a multi check conditional in bash? https://gist.github.com/hatched/e7d882ee100f52ca1eb0 this is throwing an error16:50
* benji looks16:51
hatchline 62: syntax error in conditional expression: unexpected token `;'16:52
hatchthat;s the error for that line16:52
benjihatch: I don't see a problem with that code, what error is it generating?16:53
hatch2013-11-27 16:45:18 INFO juju.worker.uniter context.go:255 HOOK /var/lib/juju/agents/unit-failtesterp-0/charm/hooks/install: line 62: syntax error in conditional expression: unexpected token `;'16:53
hatchthe conditional in that gist is the line in question16:53
hatchoh shoot I know what it is16:54
hatchmissing the space after the final True16:54
hatchdamn I hate bash16:54
hatchthanks for the help :)16:56
benjihatch: heh, I just noticed the same thing, and yes, shell scripting is nasty16:57
* benji pines for MPE job control language16:58
frankbanMakyo: available when you want17:01
Makyofrankban, https://plus.google.com/hangouts/_/7acpir7e3t3ummvqu4iun0imac?hl=en17:01
frankbanguihelp: I need two reviews for https://codereview.appspot.com/34130043 , no rush, thank you!17:40
Makyofrankban, I'll take one.17:40
gary_posterfrankban, I'll take a no-qa one :-)17:40
gary_posteryay 120 -> 40 !17:41
frankban:-)17:41
frankbanthank you both17:41
gary_postermm, fresh Thanksgivikkuh latke.  day early, but too good to pass up.17:47
gary_posterfrankban, am I right in understanding that you removed test_local_release because TestBuiltinServerLocalRelease uses the local release for all of its tests?17:52
frankbangary_poster: yes, that scenario is already tested there17:53
gary_posterok cool.17:53
gary_posterhatch, Makyo I might be late for our calls.  Is that alright?  I'd png you when I was available.17:56
Makyogary_poster, sounds good17:56
gary_posterty17:56
gary_posterfrankban, "LGTM with trivial.  Thank you!"17:57
frankbangary_poster: great thanks!17:58
hatchgary_poster no rush, take your time18:04
gary_posterthanks hatch18:04
gary_posterok, afk for awhile18:15
hatchjujugui looking for a really quick review https://codereview.appspot.com/34120044/19:01
benjihatch: looking19:02
hatchthanks19:02
hatchbenji thanks, and oops I forgot to commit the tests, will run them and submit19:09
benjicool19:09
hatchbenji interesting bug I am guessing because the element is never focused because it's generated on the fly19:26
benjiyeah, I figure it will be something like that (assuming we ever try to fix it given its importance and the likelyhood that we're ripping that code out anyway)19:27
rick_h_ok, how do we build this into the gui? http://david.li/waves/19:35
rick_h_benji: when we get back I'd like to see if you and I could sit down and plot out the best way to setup the new project https://github.com/mitechie/jenkins-github-lander19:36
rick_h_benji: my brain is fried for the day, but if you get a chance to peek appreciate it. The makefile setps to setup and such are a bit of a mess19:36
benjirick_h_: sounds good19:36
rick_h_gary_poster: ^ pushed up to a person branch a working start that does what the thing did yesterday with a couple of tests (far from complete) and docs. Calling that a day. Will have notes for the email for monday. 19:37
gary_posterrick_h_, fantastic!  thanks and happy Thanksgiving19:37
gary_posterjujugui, anyone object to me making a bzr release and putting it in the ~juju-gui charm (not the official ~juju-gui-charmers one)?  I want to make it available for testing quickstart19:41
benjisounds fine to me19:41
gary_posterk19:44
BradCrittendengary_poster: a-ok.19:44
gary_posterthx19:44
=== BradCrittenden is now known as bac
bacbenji: your new random service name + 'make deploy' in the charm make for some fun19:46
benjibac: hmm, maybe there should be an optional service name which __main__ would pass19:47
bacjujugui: please don't look at my review just yet.  i'm doing QA and writing instructions19:47
bacbenji: that'd be nice.19:48
benjibac: I'll make a card19:48
bacgary_poster: flurries in CH.  you got anything?19:52
gary_posterbac. just rain20:07
Makyojujugui https://codereview.appspot.com/34260044/ quick quickstart reviews/qa, please! 20:11
bacMakyo: i'll do it as soon as i can20:12
Makyobac, cool, thanks.20:12
hatchgary_poster just FYI a large data transfer down a websocket will block because the sockets are sent in order. Other have gotten around this by splitting up the large files but....no20:18
hatch:)20:18
gary_posterhatch, great data, thanks :-)20:18
gary_posterand I'll be ready for call soon20:18
hatchgary_poster sure thing, whenever you're ready20:27
bacMakyo: code reviewed.  can't do QA until i get my current run sorted out.20:29
Makyobac, sounds good.20:29
Makyobac, overloaded var; qaing frankban's branch, but will fix in a few.20:33
MakyoHah, roommate (who works for a grocery store) said his friend was raving about this weird cloud product called 'Juju'... :)20:39
hatchw00t w00t20:40
hatchwe are famous!20:40
gary_posterheh20:41
hatchrick_h_ you still around?20:41
gary_posterhatch  zero rush but https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.s6mt389qadeobe2a9tm9d8nm4020:41
rick_h_hatch: what's up?20:41
hatchrick_h_ any experience writing hooks for apache2 and haproxy?20:42
rick_h_hatch: no, would just have to check out what things like charmworld does, or wordpress20:42
hatchok I'll check there20:45
hatchoh netflix you silly silly people https://www.evernote.com/shard/s219/sh/cd7f6ae5-8cdc-4282-ada4-d97e4e695ebd/99a3294013dea17dfb24c22e80cf8db221:12
bachatch: i'm not going to have time to debug huw's branch21:12
hatchbac ok no problem21:13
hatchbac so shall I get him to do it or does it look like ther eis going to be a lot of work for me to do?21:13
hatchjust trying to decide what to start next?21:14
bachatch: the error is in new js code that he wrote. so, i figure he can figure it out but help would be appreciated.21:15
bachatch: i didn't dig in enough to even know what he's trying to accomplish21:15
hatchok then maybe I'll wait until he gets back and assist with it21:15
hatchhe should be starting pretty soon I think21:15
bachatch: it may be as simple as he's looking for the wrong selector.  or, he may need to be defensive about checking for null if the selector isn't there21:16
hatchalright cool thanks21:17
hatchhave a good one21:17
bacjujugui: i'm running locally and did a 'make deploy' followed by a set of juju-gui-source to use lp:juju-gui.  the latter gets stuck and never recovers.  i'm a bit perplexed as to why.  i've been chasing this for a couple of hours trying to QA my branch.  thoughts/21:18
bacs///?/21:18
bacwow, how'd that happen?21:18
hatchlol21:18
rick_h_bac: check the /var/log/juju/unit..... log 21:18
bacjust tried to change the / to a ?21:18
rick_h_bac: that should have info if it failed to change source21:18
bacrick_h_: yeah i have but don't see anything good21:19
rick_h_bac: is the hook change event in there?21:19
rick_h_errr config change hook fire in there21:19
bacrick_h_: i've taken a new tack and just made a new release and stuffed it in the charm21:19
bacrick_h_: no err.  it does say it is preparing a change event and then goes away21:19
gary_posterjujugui small but cool quickstart review request: adds last step of autologin/authtoken work.  https://codereview.appspot.com/3442004321:19
rick_h_bac: machine log?21:20
gary_posterMakyo, call now?21:20
Makyogary_poster, sure, same as on cal?21:20
gary_posterMakyo yes21:21
bacrick_h_: stuffing the release into the charm seems to be the fastest way to deploy.  it'll have to remain a mystery21:21
bacjujugui: small charm review and QA: https://codereview.appspot.com/34270044/21:28
gary_posterbac on it.21:33
bacgary_poster: i should rephrase "small review, pita QA"21:34
gary_posterheh21:34
gary_posternp21:34
* gary_poster wishes local envs worked for him21:34
hazmatgary_poster, what's the issue?21:46
gary_posterhazmat, same one that I sent out over a month ago (I think you replied then).  still not working.  Tim could not dupe.  was told to go talk to Serge and lxc folks, and haven't gotten around to it21:47
gary_posterbac LGTM and QA good, no change21:49
bacgary_poster: great21:50
hazmatgary_poster, another option is using manual provider with ubuntu-cloud lxc containers. 21:54
hazmatneeds trunk juju-core21:54
hazmatalbeit assuming you have a working lxc21:55
gary_posterhazmat, ah right.  can help with some use cases, yes.  thanks.  will put on list to try.  I need to actually verify I have lxc working on this machine.  done lxc stuff on another21:55
bacMakyo: qa was fine.22:12
* bac eod. bye.22:12
gary_posterbye bac.  have fun22:16
gary_posterMakyo, stepping away, but will return later to see how review/qa went22:17
Makyogary_poster, review okay, QAing now. frankban's branch doesn't work for me, but unrelated to his changes, so qaing yours now.22:17
gary_postercool ty22:17
huwshimiNow that I have internet again...22:27
huwshimiMorning22:27
hatchhey huw22:42
hatchhuwshimi if you need help getting your branch up and running lemme know23:02
hatchI should be around for a while yet23:02
huwshimihatch: Yeah, trying to get it working at the moment...23:03
huwshimiAh, fixed.23:21
hatchgreat23:26
hatchhuwshimi I see in your email you added checks just for the tests - that's what we call a 'code smell' :)23:27
hatchadding code simply for a test is sometimes not a good thing23:28
hatchjust keep that in mind :)23:28
huwshimihatch: I realise, but we seem to do that a bunch for nodes that don't exist in tests.23:29
hatchyeah it's always a close call between requiring it for a test or making a check23:29
hatchwe have the utils.makeContainer('id') method for just this purpose fyi23:29
hatchI have no idea what you're doing though so it could be a lot easier to just check in the app :)23:31
huwshimihatch: Just proposing this branch. Would you mind having a look and see if you think I need to clean it up?23:31
hatchsure23:31
huwshimihatch: https://codereview.appspot.com/3452004323:33
hatchcool on it23:33
huwshimithanks!23:35
hatchhuwshimi which bundle/charm did you use for testing?23:39
huwshimihatch: Most of them should have content long enough to scroll..23:40
huwshimihatch: You could look at the Readme tab on the Liferay charm23:41
hatchcool almost done23:41
hatchhuwshimi done and qa done23:44
hatchlittle more work required - maybe I shouldn't have LGTM'd :P 23:45
hatchI'm hoping that the 'jumping' mentioned in the QA happens for you as well23:45
huwshimiThanks I'll take a look23:49
huwshimihatch: Not sure if I can stop that jump23:55
hatchhuwshimi well you just add top and padding, can you maybe transition them with the header transition?23:56
hatchat the same 0.2s?23:56
huwshimiBut then instead of a jump you'll have the text sliding around... isn't that worse for trying to read it?23:56
huwshimihatch: The text would jump up and then slide down...23:57

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