/srv/irclogs.ubuntu.com/2014/02/13/#juju-gui.txt

Makyohatch, they don't have infrastructure to deal with it, and have to get home somehow :P00:20
Makyorick_h_, http://www.buzzfeed.com/passantino/potentially-catastrophic-ice-storm-bears-down-on-the-south00:20
hatchMakyo they should have stayed home00:20
hatchI remember I was in Toronto once and looked at the news and was like .....nope, staying at the office00:21
Makyohatch, the city would've shut down in a place where the community would have a hard time with a city being shut down.00:21
hatchthat sounds like a recursive problem00:21
Makyohatch, you live in a place with infrastructure to support the city shutting down, right?  You've got the will and the way to stock up on what you need to deal with a snowstorm, whereas the south is lacking both of those.00:22
MakyoNevermind the whole problem of local and county level governments fighting with each other and screwing over departments of transportation, as with what happened in Atlanta.00:23
hatchyeah that's true that's true00:23
hatchive been stranded in a car before, it's no fun, definitely not something you want to do if you can avoid it00:24
hatchnow I'm trying to decide if I should buy this humble bundle https://www.humblebundle.com/weekly00:27
hatchMakyo if you enjoy card games you should check out hearthstone, its actually pretty good00:30
MakyoI've been hearing good things about it, yeah00:30
MakyoOoo, Psychonauts.00:30
hatchIve been playing the beta, I haven't spent any $ yet, but so far it's pretty enjoyable00:30
hatchhaha yeah00:30
hatchheh it just glitched00:34
hatchMakyo think you'll get your branch landed and mine reviewed tonight?00:53
hatchI can never remember when your EOD is :)00:54
Makyohatch, Was an hour ago, but I'm usually around.  QAing your branch now.00:55
hatchohh ok so your EOD is an hour after mine, I can remember that I think :)00:55
hatchthanks00:55
=== arosales_ is now known as arosales
hatchMakyo since your branch landed is there a way I can re-trigger the CI in github?01:49
Makyohatch, yes, hold on.01:50
hatchI know I can if it fails the :shipit: CI....but not the pre-shipit....afaict01:50
Makyohttps://github.com/juju/juju-gui/blob/develop/docs/continuous-integration.rst#helpful-tips01:50
hatchahh I gota log in and do it manually01:51
MakyoThe value you would use is origin/pr/125/merge01:51
MakyoUh, and then add ac omment saying it passes01:51
hatchok sounds good01:52
hatchtoo bad we can't do a "rerun ci" :) comment01:52
rick_h_patches welcome :P01:52
hatchbastard!01:53
hatchlol01:53
hatchmanually triggereded01:53
Makyo:repeat_one:01:54
hatchhaha yup01:54
hatchI gota go shovel my parents driveway soon...ugh it's over 2x as big as mine and mine took 45mins 01:54
hatchso....much....snow01:55
hatchif we could reduce the http requests made in our CI we could probably cut a HUGE amount of time off of it01:55
rick_h_yea01:56
hatchall those 404's and the like01:56
hatchI wonder why it's so slow01:56
hatchsauce labs must be throttling it or something01:56
hatchstackoverflow.... 36M pageviews/day, 334M SQL queries, 23 servers. 01:58
hatch"...strive to maintain 50ms..."01:58
hatchwowzers01:58
rick_h_wow, every pageview averages 10 queries?01:59
rick_h_seems like a lot for that many pageviews. You'd think you could cache/etc some to cut that down01:59
hatchyeah I'm not sure about that02:00
hatchit's very odd that they could do 10 queries per pageview and still get a 50ms response time02:00
hatchheh02:00
hatchhttp://nickcraver.com/blog/2013/11/22/what-it-takes-to-run-stack-overflow/02:00
hatchSQL servers have 384 GB of memory with 1.8TB of SSD storage02:01
hatchHAHAHA02:01
hatch384GB of memory02:01
hatchhow the heck02:01
hatch11IIS webservers....they could probably use half if they used nginx :P02:03
hatch___rick_h_ will ci run with a :shipit: if I had to manually trigger the first test?02:27
hatch___I ask because it doesn't look like it is02:27
rick_h_hatch___: yea, :shipit: runs any time regardless just by you hitting it02:32
hatch___hmm it didn't, I deleted the accepted message in hopes it'll run 02:32
* rick_h_ goes to look02:32
rick_h_the only thing that blocks a shipit is if it has a 'merge request accepted' message already02:32
rick_h_so yea, it's running right now02:33
rick_h_you're good02:33
hatch___it is?02:33
rick_h_yesa02:33
rick_h_http://ci.jujugui.org:8080/job/juju-gui-merge/02:33
rick_h_http://ci.jujugui.org:8080/job/juju-gui-merge/117/02:33
rick_h_origin/125/head02:34
hatch___very odd I don't see it02:34
hatch___http://ci.jujugui.org:8080/job/juju-gui/ in the list on the left02:34
rick_h_right, there's two jobs02:35
rick_h_one for pull request tests02:35
rick_h_and one for the merge/shipit tests02:35
rick_h_they're different because the merge one triggers the api to merge if it succeeds02:35
hatch___ohh shoot then it was probably running already02:35
rick_h_while the test one does not, it's pushing back that the tests passed to the pull request02:35
rick_h_yes02:35
hatch___my bad02:35
rick_h_all good02:35
hatch___I hope it doesn't merge it in twice now02:35
rick_h_it'll handle/blow up itself02:35
rick_h_no, it'll be cool02:36
hatch___oh good good02:36
hatch___phew02:36
hatch___:)02:36
hatch___well now I know02:39
rick_h_yea, everything has the two jobs. normal and -merge02:40
hatch___kind of makes sense02:41
hatch___I just never thought of it before02:41
rick_h_hmm, failing IE tests?02:41
rick_h_hatch___: looks like there's an IE issue with your branch02:41
rick_h_see https://saucelabs.com/jobs/12ebdad9a20a429a96f161842e9157d002:41
rick_h_hatch___: mess with it tomorrow02:41
rick_h_it's way past EOD, get out of here02:42
hatch___yup thanks02:42
hatch___haha i'm playing hearthstone as well02:42
rick_h_k, cool then02:42
hatch___looks like i'm gona lose this match 02:42
rick_h_you can blame me :)02:42
hatch___lol02:43
=== _mup__ is now known as _mup_
frankbanmorning dimitern: I see that the charmsHandler parses the revision file in order to fetch the "bundle" revision (falling back to an obsolete meta.revision). How is the case where no revision file is present handled?10:00
dimiternfrankban, morning, in a meeting - will reply a bit later10:00
frankbandimitern: cool thanks np10:01
frankbanok so I guess the logic is in PrepareLocalCharmUpload. I guess revision is what found in the revision file or 0. Then if revision <= the max revision found in the db for that charm id, revision is set to maxRevision + 110:08
dimiternfrankban, sorry, i'm back11:04
dimiternfrankban, yeah, the revision from the charm is respected, but at the end an unique one is chosen, depending on what charms are in state already11:04
* dimitern wonders when exactly did the revision file got obsolete11:05
frankbanheh, thanks dimitern 11:38
hatchrick_h_ so IE11 gives totally different failures than IE10, so I'm downloading Win 8 with IE10  now14:26
rick_h_hatch: k14:27
hatchit looks like the failures are because the version of YUI we use doesn't consider IE11 IE14:28
rick_h_lovely14:28
hatchhopefully an upgrade will fix that14:28
rick_h_well, that'll be part of updating for IE11 work. 14:28
rick_h_yea14:28
hatchalthough I'm starting to get a little skeptical of YUI's development :/14:29
rick_h_how so?14:29
rick_h_they were updating promises in the last sprint. I figured you'd be happy :)14:29
hatchwell it seems to me that they are focusing more on Pure and other Yahoo projects than YUI14:31
hatchthe progress on YUI has been very slow14:31
hatchmodern.ie is pretty cool....just finished downloading the image...will see if it works :)14:42
hatchI remember the hoops we had to jump through to run various versions of IE14:43
benjihatch: bad news: I think we have to scrap the "drop a directory to deploy a charm" feature.14:44
benjiIt turns out that browser's completely ignore symbolic links when dropping files, which many charms depend on (the GUI charm included)14:45
* hatch ragequits14:45
rick_h_ah crap14:45
hatchbenji what if you symlink a zip file and just drop it? What happens then?14:46
benjihatch: I don't understand the question.  Do you mean create a zip with a symlink in it?14:47
=== rogpeppe2 is now known as rogpeppe
hatchbenji no I mean I'm wondering if it's an issue with the folders or an issue with symlinks in general14:47
hatchso take a .zip of a charm and create a symlink to it14:47
hatchdrag and drop that symlink, what does it do14:48
benjihatch: I don't know what that does, but I don't see how it applies to this situation.14:48
hatchwell in order to file a bug with chrome we need to know if it's actually an issue with chrome or an issue with the zip lib14:49
benjihatch: it is chrome, when a directory containing symlinks is dropped the symlinks are not represented in the datastructures (only real files and directories)14:50
hatchcool the modernie vm's work14:50
hatchohh ok cool, can you file a bug with chrome or would you like me to?14:50
benji(and I just tried dropping a symlink to a zip and it is the same as dropping the zip itself)14:50
hatchohh ok so it's a combination of a directory and symlinks then14:51
benjiwell, not quite; I suspect if you drop a group of files and symlinks only the files will be recognized14:51
benjifor the link-to-zip example the browser never sees the symlink, just the contents of the zip14:52
hatchwould you be able to do a bit of qa'ing and create a detailed bug report?14:52
hatchit's a story I think we would still like so we need them to fix it :) or at least monitor the status of the issue14:53
rick_h_is this true of ff as well benji ?14:54
benjirick_h_: FF doesn't support enough of these new-ish APIs for this to work anyway14:54
benjiIOW, I don't know :)14:54
hatchrick_h_ I am able to reproduce the test failure in Win 8 IE10 so will get on fixing it now14:55
rick_h_benji: ok, then yea. Let's hit the stop button if we cna't support it. 14:55
rick_h_benji: not sense spinning our wheels 14:55
rick_h_benji: but thanks for taking it through and getting it as far as we can. It would be something cool to have. We should note where things stopped for future reference14:55
hatchwait, I get a totally different test failure14:56
hatchugh14:56
hatchdo we know what OS sauce labs IE is running on?14:56
hatcher14:56
hatch7 or 814:56
rick_h_hatch: win7 and ie1014:56
hatchdamn14:56
rick_h_hatch: I think, check the browser.py file in tests14:56
rick_h_frankban: how does this effect your current/recent work? ^14:57
benjihatch: your hunch was a good one; dopping a set of files, one of which is a symlink works (i.e., the symlink is represented in the browser as a file with the contents of the pointed-to file), but when the directory containing those files is dropped, the symlink is missing from the set of files14:57
benjiI'll look to see if this bug has been reported in Chrome yet.14:57
hatchhttps://code.google.com/p/chromium/issues/list14:57
hatchbenji thanks.....this is unfortunate :(14:58
frankbanrick_h_: reading14:59
rick_h_hatch: benji frankban time for a hangout? I want to chat if there's another way to cover the use case15:00
benjirick_h_: sure15:00
rick_h_hop in the staundup hangout please15:00
frankbanrick_h_: sure, just a minute to read15:00
frankbanrick_h_: ready when you want15:02
rick_h_frankban: cool, https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj0?authuser=115:02
rick_h_benji: ^15:03
benjirick_h_: my machine froze, one second15:06
hatch*sigh* IE testing15:18
hatchtest broken....put a debugger.....test fixed15:19
rick_h_if it wasn't for IE you wouldn't be able to be a Web Development Engineer, just "Web Dude" 15:19
hatchlol...actually I'm a Web Master, I've had enough years experience copying files via FTP and doing folder based version control15:20
hatchhaha15:20
rick_h_hatch: if you're getting different errors in win7 and win8 we should try to see if we can make both pass ok15:22
rick_h_maybe add both to CI if they're acting that different for IE10 :/15:22
hatchrick_h_ yep I'm fixing the test in Win 8 right now....it's a stupid failure, the widget positioning is off, so I am changing the assertion to a range15:23
rick_h_ugh, how far off? e.g. does it look right in real life to the user?15:23
hatchwe need to get in touch with someone to find out which version if IE we are supporting15:23
hatch2px-ish, it changes every run15:23
rick_h_k, well right now we're doing IE1015:23
rick_h_that should be regardless of win7 or 815:24
hatchyeah, I'm not even sure you can get IE10 in windows 8 though15:24
hatchI think it auto updates15:24
hatchat least it did for me...15:24
hatchI'm fixing the test regardless....but curious15:25
rick_h_ugh, ok. Yea, I mean the goal is to get IE11 support in, but it's after wrapping up local charms/relation lines, and starting inspector on left 15:25
=== lazyPower_ is now known as lazyPower
makyo_jujugui call in 1015:50
=== makyo_ is now known as Makyo
MakyoMuch better.15:50
hatch:)15:51
rick_h_Makyo: can you run the standup? I've got the IS meeting and will try to hop over before the end15:51
MakyoYep, sure thing.15:53
hatchjujugui call in 215:58
hatchbenji ^15:59
hatchbenji when you get around to filing that chrome bug can you link it plz so I can star it16:08
hatchthat's two chrome bugs in a week....jeesh!16:08
hatchwe are bleeding edge yo!16:08
hatchMakyo I ended up picking this up https://www.humblebundle.com/weekly last night16:09
rick_h_standup over?16:09
hatchrick_h_ yup, redo?16:09
rick_h_hatch: naw, I'll add notes for tomorrow. 16:10
hatchok16:10
hatchrick_h_ did you create the juju-core card in Project 1?16:10
rick_h_hatch: yes, it's realted and we're not tracking any more. It's something we can work on/do16:11
rick_h_so moved it out of tracking and into todo16:11
hatchok it's not supported by juju-core right now 16:11
rick_h_right, the card is to add support16:11
rick_h_to core16:11
hatchok I'll put frankban's head on it16:11
rick_h_well if benji or someone else wants to hack on it they can :)16:12
hatchoh I didn't know anyone else had enough juju-core experience16:13
rick_h_meh, we've got to learn and get more16:13
hatchwe should have a lesson to get more of us up to speed16:13
hatcha 1 week sprint in Hawaii to get us up to speed on it :P16:14
rick_h_heh, feel free to submit it...on a day while I'm away on holiday16:14
hatchlol16:14
jcastro_rick_h_, https://bugs.launchpad.net/juju-gui/+bug/127987316:17
_mup_Bug #1279873: Bundle deploy instructions still refer to beta PPA <juju-gui:New> <https://launchpad.net/bugs/1279873>16:17
jcastro_we need this one fixed before EOM too16:18
rick_h_jcastro_: yea, I'm waiting to see if it's in universe or gets to main 16:18
rick_h_jcastro_: because in trusty the stable ppa isnt' needed any more either right?16:18
rick_h_guess we cna just roll with it16:19
jcastro_correct16:19
jcastro_it won't matter if it's in main/universe for the end user as both are enabled by defaulty16:20
jcastro_just not installed by default16:20
rick_h_k16:20
hatchwow time warner is buying comcast? wow yet another huge hit to net neutrality 16:28
hatchhttp://gizmodo.com/the-40-best-tweets-about-the-comcast-time-warner-cable-1522034013 lol16:34
benjigit help: I tried "git push juju archived-drag-and-drop-directory-to-deploy-charm" to push my branch to the main project but it complained about the branch name (error: src refspec archived-drag-and-drop-directory-to-deploy-charm does not match any.)  Any ideas?16:41
hatchbenji did you create the branch first?16:42
hatchgit push origin current_branch:refs/heads/branch_to_create16:43
hatchthat's what grb runs to create the remote branch 16:43
benjihatch: nope; something like "git branch juju NAME-OF-BRANCH"16:43
hatchbut for you origin will be your fork (likely)16:43
rick_h_origin == juju16:44
hatchyeah that won't create the remote branch16:44
benji"git push juju current_branch:refs/heads/archived-drag-and-drop-directory-to-deploy-charm" results in the same error16:44
rick_h_git push 16:45
hatchgit push <remote-name> <local-branch-name>:<remote-branch-name>16:45
rick_h_git push  <REMOTENAME> <LOCALBRANCHNAME>:<REMOTEBRANCHNAME> 16:45
benjiah, push; trying that16:45
rick_h_benji: ^16:45
hatch:D16:45
rick_h_https://help.github.com/articles/pushing-to-a-remote16:45
hatchapparently the new Windows 7 vm I just started thinks that I have an HP printer16:45
hatchI wonder if I send pictures to it where they will get printed out too16:46
hatchlol16:46
benjithat worked; thanks guys16:46
hatch:) np, git is an odd duck16:46
hatchbut you can pretty much do whatever you want if you know the commands hah16:46
benjiI continue to belive that git has more power than I want.16:48
benjioh, and the UI sucks16:48
hatchtrue....and true16:48
hatchalthough...git is just a db....so it really needs an ORM....like GRB (which I use)16:48
hatcher....ORM-like-tool16:48
rick_h_benji: it gets to be like vim, you find the 11% you use 16:48
hatch11%? That's pretty specific :)16:49
benjiexcept that I like power in my editor16:49
benjihatch: he thinks internally in fractions and converts to percentages for presentation; he uses 1/9th of vim16:49
hatchhaha16:50
hatchrick_h_ u so smart16:51
hatchson-of-a, I get no failures in WIn 7 IE1016:52
hatchwell I'll push my changes and see what happens16:52
rick_h_jujugui machine view run through with UX 8:30am est tomorrow if anyone can/wants to make it17:14
hatchoh balls that's early17:15
hatchwhat's the current time in EST?17:15
rick_h_12:1417:15
hatch(as a international company we should really deal in UTC)17:15
rick_h_yea, we should but I fail17:15
hatchok 7:30 for me, not so bad, I thought it was 2H17:15
rick_h_I've got to learn17:15
hatchhaha so'k I'll let it pass.....this time17:15
rick_h_heh17:15
hatchrick_h_ send me an invite i'll be there17:15
rick_h_hatch: rgr17:16
hatchwth I think sauce labs is broken or something17:31
hatchincredibly frustrating17:31
rick_h_hatch: port forward and run make test-server locally and use the 'start a sauce instance' to debug from sauce?17:33
hatchrick_h_ do you know how I may run our unit tests using sauce labs on IE 10?17:33
hatchheh 17:33
rick_h_hatch: :)17:33
hatchare there docs somewhere?17:33
rick_h_no, they're kind of custom17:33
rick_h_hatch: I use https://chrome.google.com/webstore/detail/sauce-launcher/mmcebionhdleomnkegjcoadpghnmcebl17:33
rick_h_and launch a sauce instance of the right machine type17:34
rick_h_then in that remote window go to http://yourhomeip:8888 (you setup port forward from your router to your dev machine)17:34
rick_h_and you can control the sauce machine and run the tests/debug your local running instance17:34
hatchohh yeah that's not going to work unfortunately....my router is a pos and won't let internal connections no matter what I do17:35
rick_h_hatch: then run it from an ec2 instance?17:35
rick_h_or something17:35
hatchmaybe I can spin up a ec2 instance17:35
rick_h_:)17:35
hatchugh I hate it when tooling gets in my way17:35
rick_h_just remember in ec2 you have to expose the port in the security group17:35
rick_h_time to router upgrade 17:35
hatchit's also the modem17:37
hatchno can do17:37
hatchhmm it's not accepting the juju-gui creds from the wiki17:37
rick_h_hatch: right, what I do is set it up to send all traffic thorugh and put my own router right inside the one given to me17:37
hatchohh I see do a DMZ to your own router17:37
hatchyeah I should do that17:37
rick_h_hatch: yea, I don't trust their routers they make me have17:38
hatchwell mine is definitely a pos17:38
hatchgreat the sauce launcher thing won't accept the creds17:40
* hatch flips desk17:40
rick_h_hmm, sec17:40
hatchit's ok i'm doing it through the actual sauce labs website17:41
hatchnow17:41
rick_h_k17:42
rick_h_yea, seems it doesn't like the password from the extension now17:42
hatchwow ec2 really likes to hide what the username is on these instances hey?17:58
rick_h_ubuntu17:58
rick_h_always ubuntu?17:58
rick_h_or you mean the dns names?17:58
hatchrick_h_ well their documentation says ec2-user18:00
hatchapparently it's different for every distro, but there is no documentation to that effect18:00
hatch*sigh* cannot connect to the ec2 instance18:13
rick_h_hatch: you opened the port?18:15
rick_h_in the security group?18:15
hatchyeah it seems the entire instance crashed18:15
hatchit kicked me out of ssh18:15
hatchand now I can't get back in lol18:16
hatchtoday is not my day haha18:16
hatchok trying with a new image18:17
hatch3rd time is the charm!18:17
rick_h_hatch: you know there's official ubuntu images you should use right?18:19
rick_h_http://cloud-images.ubuntu.com/locator/ec2/18:19
hatchI just picked 12.04 from the list in aws18:19
hatchbut very cool18:20
hatchI didn't know about that list18:20
rick_h_yea, we put out official ones that always have the same setup (thus always ubuntu user) and usch18:20
rick_h_such18:20
hatchok finally got it18:21
rick_h_yay18:21
hatchnow to sauce lab18:21
rick_h_Makyo: you're on the databinding card? Can I stick your head on there?18:21
Makyorick_h_, oh, whoops.  Yeah18:21
rick_h_cool18:21
rick_h_Makyo: is this the last card?18:22
hatchrick_h_ the tests pass....18:22
rick_h_hatch: they don't run that fast :P18:22
hatchit's well past the ones which failed18:22
hatchhmm ok well I guess I can trigger another jenkins run and see if it works18:23
rick_h_hatch: ugh ok. so you've got a {'platform': 'Windows 7', 'version': '10'}, selenium instance passing on your branch?18:23
rick_h_is it somtehing to do with prod vs debug then?18:23
Makyorick_h_, maybe.  There might be a clean-up branch with a few small things, but if needed, we can release the feature flag before then.18:23
rick_h_Makyo: k, I'm going to move it to maint then so I can start adding cards in here today/tomorrow for the machine view/inspector stuff if that's cool?18:24
hatchhmm maybe it's a prod vs debug thing18:24
Makyorick_h_, Yep!18:24
rick_h_Makyo: thanks!18:24
hatchtrying prod18:25
hatchok prod passes as well18:28
hatchrick_h_ https://saucelabs.com/tests/340a9b1951a649a1957f624555cc4254 see 100% IE10 WIn 718:29
hatchI'm out of ideas now18:29
hatch:)18:29
rick_h_hatch: ok, so final thing that'll rely on timing :)18:29
rick_h_hatch: trigger build, watch when it starts the IE tests and outputs the url to the console Starting unit tests against IE18:30
rick_h_hatch: and go to the url on the CI server18:30
rick_h_hatch: http://ci.jujugui.com:8888/test/index.html18:30
hatchwell ok...but why? We already know it fails because you can see in the video that there is the error18:31
* rick_h_ gets the popcorn to watch it18:31
rick_h_hatch: meh, yea I was thinking your's is working but it's not different 18:31
hatchI will though, maybe it will give SOME indication18:32
hatchheh18:32
hatchmaybe it's loading a different version of IE or something18:32
* rick_h_ goes to look at your diff to look for setHTML bits18:33
hatchI 'think' the error is coming from the Y.one().append() 18:37
hatchbut that should be stubbed out18:37
hatchand is everywhere else...18:37
rick_h_'should be stubbed out'?18:38
hatchyeah, like changed for a stub method18:38
rick_h_append shouldn't call setHTML though. Looks like it uses addHTML18:40
hatchyeah that makes more sense doesn't it? hah...that's all I had as far as ideas where setHTML comes from18:40
rick_h_if (node.nextSibling) { // IE errors if refNode is null18:40
rick_h_try to go about that a diff way18:41
rick_h_add it to a diff node, not body18:41
rick_h_isn't there a container or anything we can use? 18:41
hatchwell it shouldn't be doing any of that18:41
hatchI stubbed out Y.one with a new object with an append() function18:41
hatchso it shouldn't be actually 'appending' anything18:42
hatchor one()ing for that matter18:42
rick_h_hmm, worth a debug then I guess. 18:42
rick_h_to trace the call in the test18:42
rick_h_why would you stub out Y.one? I don't follow18:42
hatchshhh watching movie18:43
hatch;018:43
hatch:)18:43
rick_h_heh18:43
hatchapparently it was a schrodinbug the act of watching it caused it to change....and pass18:46
hatchugh18:47
hatchrick_h_ quick hangout I'll explain the Y.one stubbing?18:47
rick_h_hatch: k, yea I've got a couple of ?18:47
hatchhttps://plus.google.com/hangouts/_/76cpi0qie5fr86lftidmcjfgcg?hl=en18:47
* hatch remembers to go shut his ec2 instance down19:12
rick_h_good call19:12
hatchwell now the decision....what router to buy19:13
hatchis there a 'popular' one these days? lke the ol WRT54g19:13
marcoceppirick_h_: hey, so, docstrings, how do I parse them for like documenting methods and classes in python?19:38
marcoceppii found docutils, but it's all like rst2html19:38
rick_h_marcoceppi: hmm http://sphinx-doc.org/ext/autodoc.html ?19:39
rick_h_marcoceppi: there's some other javadoc like tools, but usually just use sphinx for docs and use autodoc19:40
marcoceppiah, this looks like it19:40
rick_h_honestly, I don't tend to generate just api docs of the code apart from the code19:40
rick_h_benji: ^ ?19:40
rick_h_have any magic thing you like for that?19:40
marcoceppiI just don't want to document twice19:41
rick_h_yea19:41
marcoceppionce, in code, run make doc, then you're done19:41
rick_h_I find the two docs tend to be different. API docs aren't usually useful for users in any way. They need more full sentence type material that's structured by use vs by code module19:41
rick_h_and devs are going to be in the code anyway, or at least able to run help(somemodule) in an interpreter19:42
rick_h_but yea, check out autodoc and see if that helps. 19:42
benjiI don't really like structured function/method/class documentation so I don't know much about the options available for Python.  You can't go wrong with sphynx though.19:43
benjiHere are some API documentatin tools: https://wiki.python.org/moin/DocumentationTools#Automatic_Python_API_documentation_generation_tools19:43
marcoceppibenji: rick_h_ thanks19:47
hatchbenji thanks for making that chrome bug...starred 20:24
benjinp20:24
hatchhttp://liquidat.wordpress.com/2014/02/11/first-look-at-cockpit-a-web-based-server-management-interface/20:25
* rick_h_ runs away. Have a good one 20:47
hatchcya rick_h_ 20:48
hatchhey benji  which tests are you referring to with your current branch?20:48
benjihatch: the canvas drop handler tests.  There are some truly contorted tests already (that I'm not touching) but in the course of the last branch I had some nice refactorings and tests that I wanted to capture.20:49
hatchohh ok, so nothing that I've been working on too that's going to cause conflicts?20:49
benjiI don't think so.20:50
hatchcool20:50
Makyojujugui small review/QA: https://github.com/juju/juju-gui/pull/12721:15
rick_h_Makyo: done, what's the test ease with the ATTR? The fact that you can set it on init?21:28
Makyorick_h_, I can ensure that both the menu is displayed and that the relation  the module believes is being shown in the menu matches the relation that is actually shown in the module.21:30
hatchrick_h_ """ Error: the string "makeContainer requires a context in order to track containersto cleanup." was thrown, throw an Error :) """ haha thanks for that part :) 22:07
rick_h_hatch: :)22:12
rick_h_hmm, no huw. Wonder if it's baby time23:34
hatchhmm yeah he is usually in 1.5h ago23:34
rick_h_yea23:35
hatchmaybe we can convince him to do some left panel css work while it's sleeping ;)_23:35
rick_h_hah23:38
hatchso close to being done this refactor!23:38
hatchI wrote integration tests for the util class too23:38
hatchalleviate our concerns of the unit tests being too unity23:39

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