=== gary_poster|away is now known as gary_poster | ||
hatch | morning all | 13:31 |
---|---|---|
gary_poster | morning | 13:37 |
hatch | https://codereview.appspot.com/31190043/ need some reviews/qa :) | 13:39 |
hatch | shhh don't everyone jump up at once | 14:01 |
rick_h_ | it looks hard :P | 14:02 |
hatch | lol | 14:02 |
hatch | I installed mavericks this weekend | 14:03 |
hatch | and it's a mess | 14:03 |
gary_poster | how so? haven't seen any issues in casual & occasional usage | 14:03 |
hatch | all of the fonts were bold and blurry | 14:03 |
hatch | took me a while to get all of the settings changed to make them look closer to normal | 14:04 |
hatch | every time my NAS goes to sleep syslogd and webdavfs_agent use 100% cpu | 14:04 |
hatch | until I kill it | 14:04 |
hatch | 5GB of hd space went somewhere from before/after the update | 14:04 |
gary_poster | frankban, thank you for the great review. you up for a follow-on review of the changes I made? start is https://codereview.appspot.com/31290043/diff2/30001:40001/server/guiserver/auth.py?column_width=80 | 14:05 |
hatch | the silver lining is that I can actually use two monitors now....so small price to pay? haha | 14:05 |
gary_poster | :-) | 14:05 |
gary_poster | weird symptoms though | 14:06 |
frankban | gary_poster: sure | 14:06 |
gary_poster | thank you | 14:06 |
hatch | yeah all of my issues appear to be happening with others too | 14:06 |
* hatch is reviewing huws branch | 14:08 | |
frankban | gary_poster: done and LGTMed :-) | 14:15 |
gary_poster | frankban, awesome thanks :-) | 14:15 |
hatch | when python is used as a webserver, do people spin up x instances of the server where x == the # of cores on the machine and then load balance between them? | 14:15 |
gary_poster | yes | 14:16 |
hatch | cool, I was doing some python vs node comparisons this weekend - they are pretty similar in their uses | 14:17 |
gary_poster | yeah, makes sense | 14:18 |
hatch | there are some node concurrency libs but I coudln't really believe that they would be better at load balancing than a real balancer | 14:18 |
rick_h_ | that stuff seems to be a case of people going "Well all I need is X and look at how fast it is" | 14:20 |
rick_h_ | then they hit "Well, what if one of them doesn't respond, how do I cut it out and reroute, and what if...and what if" | 14:20 |
rick_h_ | and then they've rewritten the real balancer | 14:21 |
hatch | lol | 14:21 |
hatch | its funny cuz it's true | 14:21 |
benji | frankban: have you seen an error like this before? http://paste.ubuntu.com/6474019/ | 14:23 |
frankban | benji: no, trying locally | 14:24 |
hatch | http://www.winnipegfreepress.com/sports/curling/burgundy-to-cover-curling-trials-233270481.html | 14:26 |
hatch | rofl | 14:26 |
gary_poster | hatch, thanks for reviewing huw's branch. you landing it for him too? | 14:28 |
hatch | yup! | 14:29 |
gary_poster | thank you | 14:29 |
hatch | no problem | 14:30 |
hatch | the other part of my python vs node research is that node is 'faster' but untill ES6 starts to land python has some nicer features | 14:31 |
hatch | so it's basically a toss up | 14:31 |
gary_poster | except for lib comparison? | 14:32 |
rick_h_ | I'd say that's true of the base features of the language, but not true when you get into full ecosystem and library maturity. but ok... | 14:32 |
gary_poster | where Python wins? | 14:32 |
hatch | oh yeah the maturity if the ecosystem python wins hands down | 14:32 |
hatch | it has like 15 years on node | 14:32 |
hatch | haha | 14:32 |
rick_h_ | well even a bit of a culture on testing and such as well. Unless it's gotten better the culture was a lot more 'wing-it' which sucked for db libaries and such | 14:33 |
rick_h_ | but anyway, glad you're peeking around | 14:33 |
benji | frankban: I think I figured it out; that message is a cryptic way of saying that it can't find the file name you gave it (which it does only after bootstrapping the environment to add the impression that it has something to do with juju) | 14:33 |
hatch | yeah node is getting closer - but people still haven't found the best way to do some things | 14:33 |
hatch | I think that's also an issue on the language not having some basic functionality | 14:34 |
rick_h_ | hatch: did you see the paypal node blog post friday? | 14:34 |
hatch | hmm there was a new one? | 14:34 |
hatch | linky | 14:34 |
rick_h_ | hatch: yea, where they talked about their soon to open-source mega-express framework | 14:34 |
hatch | ohh nope did not see that one | 14:35 |
hatch | it's not on their blog | 14:35 |
hatch | ? | 14:35 |
rick_h_ | think I saw it referenced on twitter, looking | 14:35 |
hatch | https://news.ycombinator.com/item?id=6785160 | 14:36 |
hatch | kraken? | 14:36 |
rick_h_ | oh sorry, paypal | 14:36 |
rick_h_ | yea, they had a blog post about kraken and such. It wasn't out yet | 14:36 |
hatch | ugh I hate hacker news | 14:36 |
hatch | sorry for linking that | 14:36 |
hatch | "directory layout is cool" | 14:37 |
hatch | it's like the yahoo answers of the tech world | 14:37 |
hatch | but cool I'll have to look into this | 14:37 |
hatch | although I'd be worried some of my paypal payments would get held for no reason if I used this.... :P | 14:38 |
benji | frankban: I take that back. I don't know why it is doing that. | 14:44 |
frankban | benji: uhm... ok, tests are running here. what command are you using to run the tests? | 14:45 |
benji | frankban: juju-test --timeout=120m -v -e ec2 --upload-tools tests/20-functional.test | 14:45 |
frankban | benji: according to juju-test help, the test to execute must be relative to the tests/ directory | 14:48 |
benji | frankban: I've tried both | 14:49 |
benji | this was my earlier attempt juju-test --timeout=120m -v -e ec2 --upload-tools 20-functional.test | 14:49 |
gary_poster | benji, +1 on filing bugs, and pursuing if necessary, but from a "get 'r' dun" perspective, is there a huge benefit to skipping the unit tests? They run in seconds | 14:50 |
gary_poster | also | 14:51 |
gary_poster | that skips 00-setup | 14:51 |
gary_poster | maybe that's the point? | 14:51 |
gary_poster | 'cause that would normally be required | 14:51 |
benji | gary_poster: I think you are assuming I know something that I don't actually know. I'm trying to reproduce a bug using the given instructions (http://pastebin.ubuntu.com/6254966/) | 14:52 |
gary_poster | benji, ah, interesting. frankban, if full test suite passes, is that pastebin something worth pursuing, given line 1? | 14:54 |
frankban | benji: heh, I think the first line in that paste is not the actual command, but it's part of the output. | 14:54 |
frankban | benji, gary_poster : "make ftest JUJU_ENV=ec2" is the command we usually run for starting functional tests. so I believe that ambiguous juju-test message does not refer to the test file itself, but to the interpreter it requires (tests/.venv/bin/python) | 14:56 |
benji | <sigh> | 14:57 |
gary_poster | that makes sense | 14:57 |
gary_poster | otherwise how would the tests even pass | 14:57 |
frankban | (and that file is created by "make", "make ftest" etc...) | 14:57 |
gary_poster | I mean start | 14:57 |
gary_poster | since the setup dependencies were not there | 14:58 |
hatch | still no takers on my branch eh? It's not THAT bad :P https://codereview.appspot.com/31190043/ | 15:05 |
rick_h_ | hatch: I'll peek in a bit. | 15:06 |
hatch | saweet | 15:06 |
hatch | I'm already merging it into other branches....soooo yeah | 15:06 |
hatch | :D | 15:06 |
hatch | ugh itunes won't play music from my NAS anymore...guess I know what I"m doing tonight...installing Ubuntu on metal | 15:07 |
hatch | alejandraobregon is luca in today? | 15:26 |
hatch | rick_h_ am I supposed to act on these PR tests? | 15:45 |
rick_h_ | hatch: not at all | 15:45 |
hatch | cool | 15:45 |
rick_h_ | hatch: unsubscribe for a bit or something. It's going to be noisy | 15:45 |
hatch | I'm just wondering if I can send the canonical gh stuff to my canonical email instad of my personal one | 15:45 |
hatch | will have to look into that | 15:46 |
gary_poster | jujugui call in 10 | 15:50 |
hatch | jujugui call now | 16:01 |
gary_poster | bah thanks | 16:01 |
hatch | :-) | 16:01 |
alejandraobregon | hatch: hi! Luca is on holiday this week... | 16:08 |
hatch | ahh alright - I actually think I came up with a workaround so I don't think I need to change any design stuff yet | 16:09 |
hatch | if it turns out to not be the case I'll ping you :) | 16:09 |
gary_poster | hatch, you ready for me to do follow-on review or should I wait for follow-up? | 17:05 |
hatch | sorry I didn't see that rick_h_ had reviewd | 17:06 |
hatch | looking | 17:06 |
hatch | gary_poster yeah you can do a follow-on nothing in here requires large changes | 17:08 |
hatch | thanks | 17:08 |
gary_poster | cool, on it | 17:08 |
hatch | gary_poster http://jsbin.com/uVepAdO/1/edit?html,css,js,output unit list header (click on it to view it when it's open) | 17:12 |
hatch | inotherwords flexbox is awesome | 17:13 |
gary_poster | hatch, :-) was playing with it. very cool. lemme play a bit more, 1 sec | 17:13 |
hatch | rick_h_ your QA issue is the card I'm working on right now just FYI | 17:14 |
rick_h_ | hatch: k, figured but taking down my notes as I go | 17:14 |
rick_h_ | working on getting live env up now | 17:15 |
hatch | :) cool | 17:15 |
hatch | gary_poster there is an issue when the unit number gets larger bleeding into the description - but I"m sure I'm just missing an option or something | 17:16 |
gary_poster | hatch, cool, yeah. the other thing I was looking at was the odd indent of the second line but I see it was because of the space at the beginning of the description. Once that is gone it is AOK. Very cool | 17:17 |
gary_poster | and the number has to be pretty big to have that behavior | 17:18 |
gary_poster | bigger than we are likely to have to worry about | 17:18 |
hatch | yeah true true | 17:18 |
gary_poster | but it would still be nice to address | 17:18 |
gary_poster | .desc padding-left: 10px; is a hack that fixes the issue I was after | 17:20 |
hatch | gary_poster http://jsbin.com/uVepAdO/2/edit?html,css,js,output at 20,000 it's still a-ok so using padding fixes it | 17:20 |
hatch | hah :D | 17:20 |
gary_poster | :-) | 17:20 |
hatch | this is so cool, where was flex box 5 years ago | 17:20 |
gary_poster | heh | 17:21 |
rick_h_ | hatch: ERROR cannot get latest charm revision: charm not found: cs:precise/failtester | 17:21 |
rick_h_ | hatch: nvm, I'm silly. it's not reviewed so can't use the short name | 17:22 |
hatch | rick_h_ try cs:~hatch/precise/failtester | 17:22 |
hatch | :) | 17:22 |
rick_h_ | curse you copy/paste instructions that don't work and make me think | 17:22 |
hatch | haha sorry | 17:22 |
rick_h_ | it's monday for goodness sake! | 17:22 |
hatch | my first cup of coffee was really...really bad | 17:23 |
hatch | damn kcup quality control | 17:23 |
rick_h_ | kcup boooooo | 17:23 |
hatch | mondays | 17:23 |
rick_h_ | curse of the coffee seas | 17:23 |
hatch | lol - I want one of those things that'll grind and drip | 17:23 |
hatch | maybe for christmas | 17:23 |
hatch | the auto complete on google for 'grind' are a little disturbing | 17:25 |
gary_poster | hatch, you have a "we should talk but tentatively LGTM" from me :-) | 17:38 |
hatch | :D ok looking | 17:40 |
hatch | rick_h_ I don't understand your comment ""upgrade service is not one of the category types allowed now correct?"" | 17:47 |
hatch | https://codereview.appspot.com/31190043/diff/1/app/views/viewlets/service-overview.js | 17:47 |
rick_h_ | hatch: so the code there pulls a nested object | 17:47 |
rick_h_ | hatch: and it gets it from the list of category types, which are updated to be one of the 4, error, landscape, etc | 17:48 |
hatch | ohh that's only for 'unit' | 17:48 |
hatch | 'service' types are untouched | 17:48 |
rick_h_ | and the upgrade-server is a part of that ntested object for services | 17:48 |
jcastro | heya hatch | 17:48 |
jcastro | what's the word on the ghost charm? | 17:48 |
rick_h_ | ok, that's where I was wanting some clarificaiton, I wasn't seeing the service vs unit split/clarity | 17:48 |
hatch | rick_h_ yeah that took me a bit to understand as well, Matt repurposed the d3 code to add that in | 17:48 |
rick_h_ | hatch: and the code didn't read in a way that it was clear that there wasn't just a missed series of changes | 17:48 |
hatch | I think later we will probably separate it out | 17:48 |
hatch | jcastro this weekend I updated the GH repo so that it passed proof | 17:49 |
jcastro | rawk | 17:49 |
hatch | jcastro tonight/@lunch I'll push it to LP | 17:49 |
hatch | jcastro I would like some help with the apache/haproxy interfaces though | 17:50 |
hatch | not sure who to talk to about that | 17:50 |
jcastro | marco is on holiday, the world is over. | 17:50 |
hatch | lol | 17:50 |
hatch | rick_h_ can you think of a way to comment that which would clarify it? | 17:50 |
hatch | it's kind of a single cog in a large machine heh | 17:50 |
rick_h_ | hatch: I just don't see how this works. You canged status.category to status.categoryType, surely in the case of a service calling categoryName there needs to be a change from status.category to status.categoryType? | 17:53 |
rick_h_ | hatch: I just don't see how this can not change if it serves two masters. | 17:53 |
hatch | :) magic! | 17:54 |
hatch | after I finish commenting we can hangout | 17:54 |
rick_h_ | hatch: k | 17:54 |
rick_h_ | bah, and the other github plugin to jenkins blows up jenkins because the precise version is too old, wheeeee | 17:55 |
hatch | gary_poster want to chat now? | 17:58 |
hatch | https://plus.google.com/hangouts/_/7ecpj24cevvl2eg0vg8jjmjojo?hl=en | 17:58 |
gary_poster | hatch, sure. finishing reading your notes | 17:58 |
hatch | join whenever, rick_h_ feel free to join as well | 17:59 |
hatch | jcastro you should update your video to lay the services out in the shape of a heart lol, its close already | 18:23 |
jcastro | hatch, that should be in the bundle itself I think, heh | 18:24 |
jcastro | a bundle easter egg | 18:24 |
hatch | haha | 18:24 |
hatch | I REALLY want 'to' in bundles | 18:25 |
jcastro | You can add them by hand | 18:25 |
hatch | would be awesome for a ghost bundle | 18:25 |
hatch | oh? | 18:25 |
jcastro | yeah | 18:25 |
jcastro | just go into the yaml | 18:25 |
jcastro | and do | 18:25 |
jcastro | to:0 under the services | 18:25 |
hatch | oh very cool | 18:26 |
hatch | how did I not know that :) | 18:26 |
jcastro | it didn't work until just recently | 18:26 |
jcastro | but hazmat fixed the bug | 18:26 |
jcastro | hazmat, the latest juju-deployer in the PPA is from 3 October though | 18:27 |
jcastro | hazmat, should I ping someone to update that? | 18:27 |
jcastro | hatch, in the meantime to: '0' works | 18:28 |
hatch | cool - now once I get some help on apache/haproxy there could be some very cool bundles | 18:28 |
hatch | I'm sure I could figure it out myself but who has the time! ;) | 18:28 |
gary_poster | $ bzt diff | wc -l | 18:32 |
gary_poster | 666 | 18:32 |
gary_poster | uh oh | 18:32 |
hatch | haha | 18:33 |
hatch | I've had a couple of those | 18:33 |
hatch | jujugui apress is having a sale next Monday, all ebooks $15 just FYI | 18:35 |
gary_poster | cool. similarly, pragprog.com is 50% off with couplon turkey2013 | 18:36 |
gary_poster | coupon | 18:36 |
gary_poster | between now and next Monday | 18:37 |
hatch | is that a real site? :) | 18:38 |
hatch | no worky | 18:39 |
gary_poster | http://pragprog.com/ ? | 18:39 |
gary_poster | wfm | 18:39 |
hatch | oh there it goes | 18:39 |
hatch | broken internets | 18:39 |
rick_h_ | the tubes are freezing up due to the cold :P | 18:46 |
hatch | haha it's not even cold right now - I was carrying drywall out to the trailer this weekend in my bare feet | 18:46 |
* hatch is renovating his bathroom | 18:47 | |
hatch | gary_poster some cool books here thx - because I didn't have enough to read....hah | 18:48 |
gary_poster | hatch I know the feeling :-) | 18:48 |
* hatch is lunching | 19:03 | |
rick_h_ | hatch: http://r.bmark.us/u/9c050ffb1e690c | 19:18 |
rick_h_ | jujugui anyone seen where machines fail to come up with the only log the machine.log not able to communicate to the state server? | 19:34 |
rick_h_ | that's on a real ec2 env fyi | 19:34 |
gary_poster | not I | 19:34 |
hatch | rick_h_ is that bookie? | 20:01 |
rick_h_ | hatch: just r.bmark.us, some code to come out of bookie/breadability, etc | 20:02 |
hatch | cool | 20:03 |
hatch | way faster than the original page, but unfortunately missing the image | 20:03 |
rick_h_ | hatch: so it's a small heroku service to just do readable parsing/caching in redis | 20:03 |
hatch | the image in the middle of the article I mean | 20:03 |
rick_h_ | hatch: ah, yea. Missed that. | 20:04 |
hatch | I clicked through because the copy didn't really make sense without it heh | 20:04 |
rick_h_ | cool | 20:04 |
hatch | Do you manage the code which does the scraping? | 20:04 |
hatch | or is that another lib | 20:05 |
rick_h_ | yea, it's a port of readability js https://github.com/mitechie/breadability | 20:05 |
hatch | ahh cool | 20:05 |
hatch | looks like there is an issue :) | 20:06 |
rick_h_ | yea, there's a few | 20:06 |
rick_h_ | it's tough to get right, needs more <3 | 20:06 |
hatch | I'd imagine the scraping is very difficult | 20:06 |
hatch | rick_h_ but re that article - When I bought my phone I thought the first thing i'd do is install raw android - but after the 4.3 update I actually like sense heh | 20:09 |
hatch | I love how IE10 uses totally different strings for it's flexbox stuff | 20:41 |
hatch | -ms-flex-align vs align-content | 20:42 |
hatch | jujugui anyone still around who has an IE vm? | 20:54 |
gary_poster | hatch, I do. what do you need | 20:54 |
* gary_poster fires up | 20:54 | |
benji | I refuse to answer that question on the basis that it might distract my from finishing this darn branch. | 20:54 |
hatch | gary_poster just proposing the unit header branch now, and it needs a QA In ie :) | 20:54 |
hatch | benji haha | 20:54 |
gary_poster | :-) k | 20:55 |
hatch | I feel sorry for people who have to support old ie | 20:55 |
hatch | gary_poster https://codereview.appspot.com/32260044/ thanks | 21:00 |
gary_poster | welcome | 21:00 |
hatch | gary_poster just added some reviewer notes | 21:02 |
gary_poster | ack | 21:02 |
hatch | jcastro hey is the charm school thing available to anyone? even outside the company? | 21:05 |
gary_poster | hatch we expect IE behavior to be different--chevron goes to next line? | 21:10 |
hatch | gary_poster yes - but I wasn't able to get anything to require being truncated in IE locally | 21:11 |
gary_poster | Landscape does | 21:11 |
gary_poster | for me | 21:11 |
gary_poster | "...restarted" | 21:11 |
hatch | yeah that's still under the max length here :/ | 21:11 |
hatch | maybe I can zoom in | 21:12 |
hatch | but yes IE 'should' wrap because of it's wako support of flexbox | 21:12 |
gary_poster | ok | 21:12 |
gary_poster | looks great on chrome and ff | 21:12 |
gary_poster | and ok on ie | 21:12 |
gary_poster | ==win! under the circumstances :-) | 21:13 |
hatch | I'm hoping on IE11 it 'just works' | 21:13 |
gary_poster | We all hafta dream | 21:13 |
gary_poster | hatch, LGTM with trivial and QA OK. Very cool! Thank you. | 21:14 |
hatch | great thanks | 21:15 |
hatch | only one more card now until I can work on the scaling issues | 21:19 |
hatch | that took longer than expected hah | 21:20 |
jcastro | hatch, yep, as long as it's more than one dude we'll charm school anybody | 21:22 |
hatch | jcastro cool I'll send an email to a couple guys I know who might be interested | 21:23 |
hatch | great idea btw | 21:23 |
jcastro | yeah! | 21:23 |
hatch | jcastro I'm wondering if someone should put together a blog post that compares/contrasts docker/juju | 21:42 |
hatch | it seems that a lot of people don't understand the difference | 21:42 |
hatch | hehe http://kotaku.com/xbox-one-has-the-best-error-messages-1471381751 | 21:46 |
hatch | rick_h_ I can't even come up with a non 'wtf' reply to your link+dm issue lol | 21:56 |
huwshimi | Morning | 21:58 |
hatch | morning huwshimi | 21:58 |
gary_poster | hey huwshimi. nice animations. :-) | 21:58 |
hatch | +1 | 21:58 |
huwshimi | gary_poster: Hey! Thanks :) | 21:59 |
rick_h_ | hatch: :P | 22:00 |
rick_h_ | hatch: welcome to twitter in 2013 | 22:00 |
hatch | lol | 22:00 |
hatch | "let them eat cake! Take away their forks!" | 22:00 |
rick_h_ | I love you're like "what is this issue that everyone's been whining about for months" | 22:00 |
hatch | lol proof I don't use twitter enough | 22:00 |
hatch | seriously though that company is so stupid | 22:01 |
hatch | everything with them is huge knee jerk reactions | 22:02 |
hatch | get what you pay for? | 22:04 |
hatch | :) | 22:04 |
hatch | huwshimi did you have a chance to go through the changes to the dropdown code? Did it make sense? | 22:04 |
huwshimi | hatch: I did. Thanks so much for doing that refactor! | 22:05 |
hatch | haha np, there was a lot of discussion around it lol | 22:05 |
huwshimi | hatch: I hope things weren't too painful with what I'd already written. | 22:06 |
hatch | oh no not at all | 22:06 |
hatch | there really wasn't anything 'wrong' with what you had | 22:06 |
hatch | huwshimi today I added some flexbox stuff to the inspector that you might be interested in https://codereview.appspot.com/32260044 (or to tell me I did it totally stupidly) :D | 22:10 |
huwshimi | hatch: Nice! | 22:11 |
hatch | my first real foray into flexbox - it's really awesome | 22:12 |
* gary_poster has authtoken working in GUI, with a bit more polish than intended. | 22:15 | |
hatch | nice! | 22:15 |
gary_poster | The UX as I envisioned it was a bit too opaque | 22:15 |
gary_poster | now I need to add tests for the changes I made and can propose | 22:15 |
hatch | darn tests | 22:15 |
gary_poster | :-) | 22:15 |
gary_poster | all the stuff before was tested. It's only the "oops, I wouldn't want to use that myself; what now?" stuff. :-) | 22:16 |
hatch | haha | 22:16 |
gary_poster | and, sadly, local environments still don't work on my desktop | 22:17 |
gary_poster | I better get my ec2 reimbursement together | 22:17 |
gary_poster | before Dec 5 | 22:17 |
hatch | what's wrong with the local stuff? | 22:17 |
gary_poster | it's never worked on my desktop. asked for help; Tim et al didn't know. :-/ | 22:17 |
hatch | is Dec 5 the '1 year of ec2 expenses' deadline? lol | 22:17 |
gary_poster | works on my laptop | 22:17 |
gary_poster | I think 5 days after the month is the deadline to get it in the next months paycheck | 22:18 |
hatch | ohh | 22:18 |
gary_poster | so Dec 5 for Dec paycheck, Jan 5 for Jan paycheck, and so on | 22:18 |
hatch | huwshimi I have come across a bug with your animation code....but I don't know how to reproduce it | 22:44 |
hatch | it appears.....that if I load the gui then switch away to another tab, when I come back after it's fully loaded, the sidebar hasn't animated out | 22:44 |
huwshimi | hatch: hmmm... | 22:47 |
hatch | I'm filing a bug now, I was able to reproduce it | 22:47 |
huwshimi | hatch: Which browser? | 22:48 |
hatch | chrome | 22:48 |
hatch | https://bugs.launchpad.net/juju-gui/+bug/1254903 | 22:49 |
_mup_ | Bug #1254903: Sidebar does not open if GUI is loaded in the background <juju-gui:New> <https://launchpad.net/bugs/1254903> | 22:49 |
hatch | it's probably low priority but I figure it might be fresh in your head to do now (if you have time of course) | 22:49 |
huwshimi | hatch: I can't reproduce it here :( | 22:51 |
hatch | hmm | 22:51 |
hatch | maybe it's an OSX Chrome bug | 22:51 |
hatch | I'm in OSX right now.. | 22:51 |
hatch | I'll fire up my laptop | 22:51 |
hatch | if it's osx only then maybe it's hyper low | 22:51 |
hatch | lol | 22:51 |
gary_poster | :-/ still not great | 22:52 |
gary_poster | though this particular bug isn't too bad. hatch if you click on the tab then can you get out of the hosed state? | 22:52 |
hatch | yep, it just acts like it's /minimized but the tab is grey not orange | 22:53 |
hatch | I can reproduce on Ubuntu 12.10 Chrome 30.0 | 22:55 |
hatch | just added that to the bug... | 22:56 |
hatch | oh wait | 22:56 |
gary_poster | maybe back later | 22:57 |
gary_poster | have a good night | 22:57 |
=== gary_poster is now known as gary_poster|away | ||
hatch | you too | 22:57 |
hatch | huwshimi I just updated the bug | 22:57 |
hatch | it doesn't actually act normal | 22:57 |
hatch | it 'thinks' it's open | 22:57 |
hatch | so clicking it 'closes it' heh | 22:57 |
huwshimi | hatch: Oh, so is it a state bug that is being made obvious by the animation? | 22:58 |
hatch | well no the url shows that it should be open | 22:58 |
hatch | then clicking it changes the url to /minimized | 22:59 |
hatch | which changes the tab to orange but stays closed | 22:59 |
hatch | then clicking it again triggers the animation | 22:59 |
hatch | huwshimi it open happens in chrome | 23:00 |
hatch | so I'm guessing it's a chrome bug | 23:00 |
huwshimi | hatch: Right, but it might not be updating a class or something... | 23:00 |
rick_h_ | hatch: which bug is this? | 23:01 |
huwshimi | or maybe clicking it changes the state so that it starts working again | 23:01 |
hatch | https://bugs.launchpad.net/juju-gui/+bug/1254903 | 23:01 |
_mup_ | Bug #1254903: Sidebar does not open if GUI is loaded in the background <juju-gui:New> <https://launchpad.net/bugs/1254903> | 23:01 |
rick_h_ | hatch: oh hmm, hey did we make sure we can skip animations for tests or anything? | 23:01 |
rick_h_ | I know it's a bit late in the game now, but I'm a bit nervous that there's some awesome side effects of this that we'll 'discover' | 23:02 |
huwshimi | rick_h_: I'd be interested to see if you can reproduce that bug... I can't here. | 23:02 |
rick_h_ | checking | 23:02 |
huwshimi | rick_h_: It's just change positions of elements. I'm not sure if it could break a test | 23:03 |
huwshimi | *changing | 23:03 |
hatch | huwshimi what class should I be looking for when it's minimized? | 23:03 |
huwshimi | hatch: The tab should have a parent of #subapp-browser-min | 23:04 |
huwshimi | hatch: Otherwise the parent should be #subapp-browser | 23:04 |
huwshimi | or grandparent or something up the chain | 23:04 |
hatch | ok well both are in the DOM | 23:04 |
hatch | min shows hidden="true" | 23:05 |
hatch | and subapp-browser doesn't have that | 23:05 |
hatch | when I click again, the hidden="true" moves to the subapp-browser | 23:05 |
hatch | I can demo it in a hangout if you like | 23:05 |
huwshimi | hatch: No that makes sense. So it's doing the correct thing, it's probably the animation then. | 23:06 |
hatch | I wonder how hard it would be to create a repro to see if this is indeed a chrome bug | 23:06 |
huwshimi | hatch: So it's starting on the first keyframe but not animating to the second by the sounds of things... | 23:06 |
hatch | I can demo if you like | 23:06 |
rick_h_ | hatch: huwshimi can't dupe here. | 23:07 |
hatch | hmm, and I could dupe on two computers | 23:07 |
huwshimi | rick_h_: Could it be something like this: http://blog.wercker.com/2013/10/02/Chrome-css-animation.html | 23:07 |
huwshimi | oops, hatch: ^ | 23:07 |
huwshimi | rick_h_: Yeah, I think we should just blame hatch | 23:08 |
hatch | lol incoming call | 23:08 |
hatch | https://plus.google.com/hangouts/_/lite/72cpjbdnhbd1meck125vvcnibk?hl=en | 23:08 |
huwshimi | hatch: "That last one was especially annoying since it turned out it was when wercker was not running in the foreground. Focussing on the wercker tab would cause the CPU load to drop to a normal level." | 23:09 |
hatch | hmm that one didn't work | 23:09 |
huwshimi | hatch: You'll have to use my canonical account | 23:09 |
hatch | https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj0?authuser=1 | 23:09 |
hatch | ^ huwshimi | 23:10 |
huwshimi | coming | 23:10 |
huwshimi | hatch: Oh, have to install a plugin | 23:10 |
huwshimi | one sec | 23:10 |
hatch | haha oh right, first time | 23:10 |
hatch | :) | 23:10 |
rick_h_ | tried cache clearing as well without luck | 23:11 |
hatch | feel free to join the demo call | 23:11 |
rick_h_ | sorry, dinner with the boy right now so not available | 23:12 |
rick_h_ | just wanted to make sure we weren't blaming browser.js :P | 23:12 |
hatch | haha maybe we ARE! | 23:25 |
huwshimi | Gonna mark this bug as "invalid: only reproducible within hatch's house" | 23:26 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!