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

=== rogpeppe1 is now known as rogpeppe
=== gary_poster|away is now known as gary_poster
gary_posterrick_h_, I was going to say thank you to Hannah for her email but wanted to confirm that you can add things on allhands.  could you confirm?13:32
rick_h_gary_poster: looking now13:37
gary_posterty13:37
gary_posterrick_h_, radio silence on the CI email. :-/ worried that it means tl;dr13:39
gary_postersigh13:39
rick_h_gary_poster: yea, I was thinking of doing personal pings to at least curtis/john13:39
gary_posteryeah would be great, thank you13:39
rick_h_gary_poster: objective sheet submitted13:45
gary_posterawesome, thanks rick_h_ 13:46
rick_h_thanks for getting it opened up 13:46
hatchmorning all13:46
rick_h_up early hatch ?13:46
rick_h_are you still 3hrs back?13:47
hatchso on Friday I picked up a digital ocean account, it would be much cheaper to run our CI there than on ec2 - unfortunately it doesn't support Juju until we can use Manual Provisioning in the next version of Juju13:47
rick_h_or two I guess13:47
hatchrick_h_ I'm ALWAYS GMT-613:47
gary_posterand not cranky about it all!13:48
hatchlol13:48
hatch-always-13:48
hatchis that better?13:48
hatch:P13:48
hatchit's 7:48 here13:48
rick_h_hatch: yea, I've got a free open source account on rackspace but no juju there either 13:48
rick_h_hatch: ah, so you're only 1hr back now13:48
gary_posterso much better :-)13:48
hatchif we actually used DST it would make no difference, it's still dark an hour later haha13:49
rick_h_heh13:49
=== BradCrittenden is now known as bac
bachi frankban, using quickstart the gui deploys but then there is a connection refused error when connecting the web socket.  have you seen that before?13:59
frankbanbac: is it different from what you reported yesterday?14:00
bacyes14:00
bacfrankban: it is using --no-browser and isn't the auth-connect problem.  will paste14:00
frankbanbac: cool thanks14:01
bacfrankban: http://pastebin.ubuntu.com/6514606/14:01
baci assume it is the connection to begin deploying the bundle14:01
* bac changes his ec2 connection password now...14:04
rick_h_:)14:04
frankbanbac: it seems so, and it's weird, is this trunk?14:05
bacfrankban: yes14:05
frankbanbac:  what happens if you re-run the same command, reusing the existing env?14:11
* bac tries14:12
bacfrankban: it succeeds.14:12
frankbanbac: :-/14:12
hatchnew Tim Horton k-cups - are almost exactly like in the store14:13
hatchinnnteresting14:13
rick_h_that says a bit about what's in the store14:13
hatchhey gary_poster Happy Birthday!14:13
gary_poster:-) thanks hatch14:13
frankbanbac: so my guess is that, even if juju marked the service as started, the guiserver was not yet up and running...14:14
hatchno problem! Where is the party???14:15
bacyeah, hbd gary14:16
frankbanbac: so a possible solution is that app.connect retries the "juju.connect(api_url)" step several times before giving up14:16
bacfrankban: ok, i'll file a bug.  do you want to try to reproduce?14:16
frankbanbac: I'll do, are you using local or ec2?14:16
bacfrankban: ec214:20
gary_posterthanks bac :-)14:21
hatchrick_h_ while granted it's not the best coffee around, a k-cup is no different than a drip coffee maker albeit with a lot more waste 14:24
benjimarcoceppi: I have a WIP branch with several comments explaining different bits up for your pre-review at https://codereview.appspot.com/36190043/14:40
bacfrankban: you may be interested to know i discovered yesterday that with no default-series set juju now creates precise instances.  in the past it had defaulted to the OS of the local machine, saucy in my case.  this seems like a sane change.14:54
frankbanbac: cool, so we only have to handle explicit default-series14:56
marcoceppibenji: thanks! I've not used lbox, or codereview, so I'll dig in, in a few mins14:59
hatchgary_poster rick_h_ https://github.com/arturadib/shelljs you might find this interesting14:59
gary_postercool15:00
rick_h_hatch: heh, I'm not generally all for reinventing X with Y stuff. Go JS though I guess. Probably helps with the grunt stuff. 15:01
hatchwell it beats trying to write stuff in make/bash :)15:01
rick_h_if you spent as much time finding a way around make to just read part of the book about make, you'd be a jedi master :P15:03
hatchHAH! that doesn't fix it's shortcomings lol15:04
rick_h_yet the clones never seem to get its advantages either15:05
hatchbig money big money no wammy's! Cmon launchpad, third time is the charm!15:05
rick_h_lol15:05
rick_h_<3 that a show that's not been on the air for 20 years is still a know-able reference 15:05
hatchhaha yeah it's awesome15:06
hatchwell the only real advantage to bash/make is that you don't need to install any deps :)15:06
hatchso any clone will never be able to get that advantage, unless we can convince peeps to install node/python by default in the distro hah15:07
rick_h_hatch: that statement shows you don't know enough bash/make if you think that's the only advantage :)15:07
hatchI guess...what's another one?15:08
rick_h_multi pass dependency resolution? INI variable overrides? speed? target globbing?15:08
hatchwell you can do those from any higher level language as well15:09
hatchand for ones which you can't you can always shell out for that one little bit15:09
rick_h_can do vs is implemented in X?15:10
rick_h_the thing is all the clones start out solving the easy stuff that anyone can do. It's the more powerful useful things I get annoyed with the clones for15:10
rick_h_it's not about adding features, just "anything but X" mentality15:10
hatchright, well some things just haven't been implemented in the higher level languages so in those cases it needs to be shelled out for15:11
rick_h_so you have to know shell anyway :)15:11
hatchright, but that doesn't make it enjoyable to write15:11
rick_h_carry on! /me goes back to hacking15:11
hatchjujugui looking for a review and qa https://codereview.appspot.com/36500044/ plz and thx15:13
rick_h_hatch: I can peek at it15:14
hatchthanks15:14
hatchit's a pretty easy review/qa just a little tedious to qa15:15
gary_posterhatch, would it be easy to also do a better job of passing the Juju Core error message through?15:16
gary_postermaybe that's uin the branch already--just going by your cover letter15:16
hatchgary_poster I think we fixed that in a previous branch, let me take a look15:17
gary_postercool ty15:17
hatchgary_poster good catch I'll add this to the branch15:18
hatch^ rick_h_ 15:19
gary_posterthanks hatch15:19
rick_h_hatch: k, will ignore it for a little bit more then15:19
hatchrick_h_ I'll have to spin up an LXC instance to test this, it'll be a bit15:22
rick_h_hatch: rgr15:22
hatchso my router reset (for whatever reason) and assigned all new ips across my network :/15:36
hatchi've probably been h4x0r3d15:37
hatchjujugui call in 1015:50
gary_posterty15:50
frankbangary_poster: do you have time for a pre-imp after the call?15:51
gary_posterfrankban, yes, sounds good15:51
frankbanthanks!15:51
bacfrankban: i filed bug 1257349.  were you able to reproduce?15:57
_mup_Bug #1257349: Quickstart fails if guiserver not up yet <juju-quickstart:Triaged> <https://launchpad.net/bugs/1257349>15:57
gary_posterjujugui call in 115:58
frankbanbac: not tried yet, thanks for filing the bug15:58
bacfrankban: new card too15:58
frankbancool15:59
gary_posterbenji Makyo ping16:00
hatchrick_h_ ok the new feature has been added but you cannot qa it as far as I know :)16:10
rick_h_hatch: ok, so lbox is done, go review16:17
hatchwha?16:17
rick_h_hatch: so was your last comment means that lbox is done and you'd like that reivew?16:18
rick_h_review?16:18
hatchyes :)16:19
hatchthe feature I added will only show up if there is difficulty in deploying a charm, so I'm not sure how to simulate that16:19
rick_h_the name collision feature? 16:19
hatchthat one is easy to do, but now the validation is done in the GUI so the server will never get an invalid name16:20
rick_h_k, will look at it16:20
hatchthanks16:23
benjirick_h_: I looked at the lander a fair bit I only had small, nit-picky thoughts, so either I'm getting soft, or I like it.16:34
rick_h_benji: heh, ok16:35
hatchcd ..16:44
hatchoops16:44
benjirick_h_: would it be sane for me to do the "update browser tests to use sandbox env" card?16:45
rick_h_benji: yes, that should be something we can do purely in the gui functional tests themselves. 16:45
benjirick_h_: cool, what does it mean?16:45
rick_h_benji: it might bread out into a couple of cards, let me know if you want to pre-chat on it16:45
rick_h_benji: hangout?16:46
benjirick_h_: sure; shall I create one?16:46
rick_h_sure thanks16:46
benjirick_h_: or benjiyork.com/chat worked for you, right?16:46
rick_h_benji: rgr16:46
benjirick_h_: hmm, "full"  (not sure how that happened).  Try this: https://apprtc.appspot.com/?r=5643467816:47
benjirick_h_: ok, I'll make a hangout16:48
rick_h_sec, retrying16:48
rick_h_hatch: review inbound17:06
hatchthx17:06
rick_h_man, I miss the recently closed tabs ui in the current chrome dev stuff :(17:10
frankbangary_poster, Makyo: six "envs management" cards created, the ones with my face17:13
Makyofrankban, cool17:14
hatchrick_h_ that dropdown is gone?17:18
Makyohttps://twitter.com/BoredElonMusk/status/407921563011256320/photo/1/large17:19
hatchlol17:20
bacso Makyo is that a real thing with musk?  or is it a baseless (though still funny) joke?17:22
MakyoJust a joke, I'm sure.17:22
hatchbac baseless? PLEASE!17:22
hatch:P17:23
MakyoGiven that his location is set to 'Mars'.17:23
bacyou never know17:23
hatch"Apple falls short of analysts expectations, stocks fall" <--- the problem with the stock market lol17:23
bacnancy did set ronnie's schedule based on her numerologist17:23
hatchgary_poster rick_h_  the email from Maarten looks like a perfect usecase for our 'proposed' subordinate for the GH stuff17:25
rick_h_hatch: well he replied to the email i sent out with the notes of our work in progress stuff. 17:28
rick_h_hatch: so he's aware of that already17:28
hatchoh ok cool - I try to follow emails but there are so many :)17:28
rick_h_yea, know what you mean17:29
hatchman that unit error branch was a 1 line fix and took an hour to qa17:33
hatch:/17:33
hatchlol17:33
hatchI gota speed up that lxc machine somehow17:34
hatchumm17:39
hatchso we don't have any tests for removeUnit in the fakebackend?17:39
hatchhow did THAT happen17:39
hatch:D17:39
MakyoMagic?17:41
hatchhaha17:42
hatchlunchtime!17:48
gary_posterhatch, rick_h_ agreed that it sounds like a good match18:08
hatchjuju is so awesome18:09
hatch:)18:09
gary_posterrick_h_, should I volunteer us to Maarten?18:17
hatchI can also help with that if you guys need a grunt18:17
rick_h_gary_poster: well I was assuming he's looking for more. He replied to the original CI email and asked some questions and curtis jumped in explaining some use of subordinate charms. 18:18
rick_h_gary_poster: I'm hesitent because I'm not sure where it's going. 18:18
gary_posterrick_h_, yeah, I know what you mean, and we want to be done.18:23
rick_h_gary_poster: I guess I'm happy to talk about it, but his email seems very centric on setting up a workflow around java based things, auto creating jobs out of github projects, etc, none of which is in our current scope of stuff. 18:23
gary_posterrt18:24
gary_posterok might reply but will do so carefully. :-) thanks rick_h_ 18:24
rick_h_if who ever does work on it want to see if we have some common ground I'm happy to coordinate that bit18:24
rick_h_gary_poster: rgr18:25
gary_postercool18:25
hatchahh I thought it was just jenkins plugins as subordinates18:26
hatchin the general sense 18:26
rick_h_hatch: maybe I'm reading more into the email. I'm just trying to be careful :)18:27
hatchhaha, yeah don't go get yourself sucked into writing Java18:27
hatch:P18:27
hatchmarcoceppi I would like some assistance writing the http hook for my ghost charm so that I can hook it up to apache for load balancing so that I can release a bundle...any idea who I might talk to?18:37
marcoceppihatch: me?19:05
hatchThat's right, Santa is gona be Canadian! http://gizmodo.com/canadas-getting-ready-to-claim-the-north-pole-as-its-o-147564360819:05
hatchmarcoceppi you want to help? 19:05
hatchI'll take it!19:05
marcoceppihatch: http hook should be pretty straight forward19:05
marcoceppinot sure what you'll nee19:06
hatchwell reading the apache charm it was saying I needed to write some apache directives and set them on relation-change ?19:06
* marcoceppi becomes disgruntled19:07
marcoceppihatch: one sec, let me see19:07
hatchok thanks - I could be totally misunderstanding it too 19:08
hatchhttps://twitter.com/NodeSummit/status/407951852358348800 Go JavaScript!19:19
bacweird, juju-quickstart that has a test that fails depending on what shell you run it in.19:40
hatchrick_h_ any idea if I can set up my work email for the juju project on github somehow?19:41
bacdue to the way the docstring has line-breaks when --help is run19:41
hatchbac that's interesting19:41
bacit was frustrating.  now it is interesting.19:41
hatchyeah I suppose I didn't have to go through the frustrating part :)19:42
rick_h_hatch: http://www.codeography.com/2011/08/05/project-specific-git-author.html ?19:42
hatchrick_h_ well that is for commit messages and such19:43
hatchthe emails come from gmail19:43
hatcher19:43
hatchgithub19:43
hatchat least as I understand how it works..19:43
rick_h_hatch: https://gist.github.com/BPScott/136679019:44
rick_h_hatch: see the link at the top of it19:44
rick_h_hatch: a quick search for github select email per project19:44
hatchoh awesome!19:45
bacand now it is fixed19:45
rick_h_hatch: that's a good point thouogh. I'll add that to the notes/docs 19:45
hatchrick_h_ somewhat in the same light - it might also be worth a note about setting up your 'user' for the juju git repo so that people don't commit using their personal emails and whatnot19:46
hatchcan we remove python support yet? huh huh can we can we can we plz!!!!!!!19:57
rick_h_hatch: huh?19:57
hatchpyjuju sorry19:57
hatch:)19:57
rick_h_oh, heh19:57
bacgary_poster: our call is now?  can you invite my canonical me?20:02
gary_posterbac, finishing up with mramm20:03
hatchjujugui looking for a review/qa https://codereview.appspot.com/36690043/ plz and thx20:04
MakyoOn it.20:05
MakyoAlso, done with appointment.  Looking for a card after review.20:05
hatchthanks Makyo 20:05
hatchmost of it is just tests20:06
hatchand now onto the bug I have been avoiding.....20:15
hatch*grumble grumble grumble*20:15
hatchhey gary_poster  did you end up picking up that js book yesterday?20:29
gary_posterhatch yes20:29
hatchthink it's worth picking up? The descriptions looked a little beginner-ish. I was thinking of picking this up as well http://shop.oreilly.com/product/0636920027713.do20:30
hatchiunno, it's so hard to pick up a js book these days :)20:30
hatchreviews are mixed20:31
gary_posterhatch, I think the functional one is worth picking up.  if you haven't seen someone go deep with functional programming, I think you'll find it fascinating.  The beginning is beginnery but it doesn't stay there, and the whole idea is to introduce a different way of thinking from the ground up.  For the other one you linked, yeah, reviews seem not so good.20:37
hatchcool, picking it up now20:39
hatchnow if i could just remember my oreilly information20:39
Makyohatch, chunk mismatch on your review (doing QA and going by diff for now, just fyi)20:56
hatchpoop ok thanks20:56
hatcharg the issue I'm experiencing is not even possible!20:57
hatchbcsaller you look like you're bored and want to give me some info on databinding ;) 21:08
* bcsaller yawns21:09
bcsallerhow can I help?21:09
hatchwell I have this very odd bug https://bugs.launchpad.net/juju-gui/+bug/125311421:09
_mup_Bug #1253114: Unit details left breakout sometimes empty when opening <juju-gui:Triaged> <https://launchpad.net/bugs/1253114>21:09
hatchas far as I can tell, it's rendering the template, the binding is being setup21:09
hatchbut it's not updating the DOM to the values in the model21:10
hatchit's very intermittent 21:10
* bcsaller looks21:10
hatchso I was wondering if you had any insight into what might be causing it, right now I'm console.log debugging hoping to catch a glimpse of something out of the ordinary21:11
hatchI should mention that the binding IS attached because the DOM will update after the first rendering21:11
bcsallerhatch: this is with the simulator running though, it could be removing the unit, maybe adding it back later?21:11
bcsallerlike if this happens and you turn off the simulator will it ever recover?21:12
hatchif it happens and I just sit watching the unit details page, it will eventually fix itself on the next delta for that unit21:12
bcsallerso this is *only* with the simulator? it could be a fakebackend issue with addUnit then?21:13
bcsallerI'm wary of saying its a databinding issue, not because I think that code is perfect, but because this seems like the type of issue that would show up other places21:14
hatchhmm21:15
hatchit's almost impossible for me to debug on a real env21:15
hatchas it appears to happen when the unit changes states21:15
bcsallerhatch: short of walking through the debugger with you I don't have any additional insights. I'd be tempted to explore if it is a difference in the deltas coming from the fakebackend21:23
hatchbcsaller ok no problem I'll keep stepping through  thanks though :)21:25
hatchok I found something...21:27
hatchwhen it doesn't work the delta passed to the updateDOM doesn't contain the attributes for the unit details21:27
hatchI"m just rambing now feel free to get back to your real work :)21:28
hatchbcsaller found the issue fyi - if we take this path http://bazaar.launchpad.net/~juju-gui/juju-gui/trunk/view/head:/app/views/databinding.js#L173 then the other bindings aren't updated so it's kind of a race condition. if no modelChangeKeys are provided it works as expected.21:49
hatchjust thought you'd be interested to know21:49
hatchnow...how to fix...21:50
* bcsaller checks21:50
bcsallerhatch: should be an || in the condition, no?21:51
hatchbasically instead of mixing it, it's filtering them out21:51
bcsaller== undefined ||21:51
bcsallerhmm, no, maybe not21:52
hatchnope21:52
hatchright21:52
hatch:)21:52
hatchI'll fix it, no problem, just pointing it out21:52
hatchI'm actually surprised that we haven't run into this before heh21:52
hatchso this -could- have happened on a life env but almost never heh21:53
bcsalleryeah, odd21:53
hatchbcsaller my guess is that originally we wanted to update ONLY those keys but later decided that it should mix it in21:54
hatchand it was never updated21:54
hatchbut somehow it just 'worked' for the 99.9% time21:54
huwshimiMorning21:58
hatchmorning huwshimi 21:59
gary_postermorning22:01
hatchhmm - fixed issue, cannot be sure22:16
hatchthat's the crappy thing about race condition bugs hah22:17
hatchgary_poster since you were the other person to experience this issue think you could review/qa? https://codereview.appspot.com/36850043/22:49

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