hatch | huwshimi for reals?? | 02:58 |
---|---|---|
hatch | maybe Makyo can reproduce | 02:58 |
hatch | :) | 02:58 |
Makyo | What. | 03:00 |
hatch | hah you fell into my trap! | 03:00 |
Makyo | fired. | 03:00 |
Makyo | Just got back from Chicago :P | 03:00 |
hatch | lol | 03:00 |
hatch | have fun? | 03:00 |
Makyo | Ish? | 03:00 |
hatch | well....thats....good? | 03:00 |
Makyo | James got laryngitis part way through so some of it was spent at urgent care. | 03:00 |
hatch | oh damn, all good now? | 03:01 |
Makyo | Also, the hotel was an atrium (another Hyatt Regency, like October), so there was a lot of heights to deal with. | 03:01 |
Makyo | I know know now that I can ask for a low floor and they'll give it to me :) | 03:01 |
hatch | lol | 03:02 |
Makyo | Yeah, just some antibiotics to clear it i[. | 03:02 |
Makyo | up | 03:02 |
hatch | well that's good | 03:02 |
Makyo | Didn't look at a computer screen all weekend. I'm so proud of myself :) | 03:02 |
hatch | ahh it's awesome when that happens | 03:02 |
Makyo | I brought the MBA and didn't even take it out of its bag. | 03:02 |
hatch | haha nice | 03:03 |
hatch | You should try and reproduce this bug for me https://bugs.launchpad.net/juju-gui/+bug/1254903 :) | 03:04 |
_mup_ | Bug #1254903: Sidebar does not open if GUI is loaded in the background <juju-gui:New> <https://launchpad.net/bugs/1254903> | 03:04 |
Makyo | Sure | 03:04 |
hatch | yussss | 03:04 |
hatch | I need it to happen to at least one other person so it doesn't get closed haha | 03:05 |
Makyo | Pulling. | 03:06 |
hatch | oh you can do it on comingsoon | 03:06 |
hatch | sorry I'll update the bug | 03:07 |
huwshimi | hatch: I wasn't being serious :) | 03:14 |
hatch | these are lies | 03:15 |
hatch | you're all fired up with energy after your holiday | 03:15 |
hatch | :P | 03:15 |
huwshimi | hatch: Hey, I'm just happy I figured out a way to do animations that we haven't been able to do for over six months :) | 03:16 |
hatch | no-no-no we were ABLE to do them, we just didn't want to do all the extra work to get them to work | 03:17 |
hatch | lol | 03:17 |
Makyo | hatch, reproduced, chrome 31 | 03:19 |
huwshimi | hatch: heh, yeah, we can also build a monkey. It'll just take us a little while to get all the different sciences worked out. | 03:19 |
hatch | Makyo YUS VICTORY!!!!! | 03:19 |
hatch | huwshimi those monkeys are a tough breed | 03:20 |
* Makyo sneaks for the night. | 03:20 | |
hatch | cya tomorrow Makyo | 03:20 |
huwshimi | 2 vs 2 then. I guess we need to sort this out then... | 03:21 |
hatch | clearly you're doing it wrong | 03:21 |
hatch | huwshimi so are you going to switch us to sass soon? :) | 03:25 |
huwshimi | hatch: As soon as we upgrade Node :) | 03:25 |
hatch | crap that's my job too isn't it? | 03:26 |
huwshimi | hatch: Well, that's the blocker at this point | 03:26 |
huwshimi | hatch: I think so! | 03:26 |
hatch | well then | 03:26 |
huwshimi | hatch: Have you been following Gary's authtoken branch at all? | 03:28 |
huwshimi | hatch: Just trying to figure out what this means: "One of the things it does is that now the sandbox actually says that its name is "demonstration". This shows us that the notification boxes don't line up properly if they move. Do you have any brilliant ideas on fixes for this, by chance?" | 03:28 |
hatch | huwshimi nope | 03:29 |
huwshimi | :( | 03:29 |
hatch | reading.. | 03:29 |
huwshimi | hatch: Can the environment name change dynamically? | 03:29 |
hatch | ohh I think I know | 03:30 |
hatch | one sec, testing | 03:30 |
huwshimi | Changing the name doesn't seem to break anything anyway | 03:30 |
hatch | ok I was wrong... | 03:31 |
hatch | I thought that he meant the notification when 'something' happens like deploying a bundle | 03:31 |
hatch | but changing the environment name to 'super long environment name' and then making a notification still has the popup thing in the right place | 03:31 |
hatch | and the dropdown 'list' also appears in the right spot... | 03:31 |
hatch | at least this is on comingsoon | 03:32 |
hatch | assuming that his branch landed? | 03:32 |
huwshimi | hatch: It appears to be the same for his branch too. | 03:33 |
huwshimi | hatch: Not to worry I'll ask for clarification | 03:33 |
hatch | yeah...then I have no idea | 03:33 |
huwshimi | hatch: Do you think there's any timeline for upgrading Node? | 03:33 |
huwshimi | hatch: I know it's a "test and see if we can" kind of thing... | 03:33 |
hatch | well...I can probably start on it after my branch tomorrow | 03:34 |
hatch | It's probably important that we get it upgraded | 03:35 |
hatch | there have been lots of changes with the charm so I'll have to coordinate with someone else too probably | 03:35 |
huwshimi | hatch: Is there a specific reason you were thinking about SASS again? | 03:37 |
hatch | source maps | 03:37 |
huwshimi | ah right | 03:38 |
hatch | beyond that, I think it offers some nicer syntax stuff | 03:38 |
hatch | but that's just personal preference | 03:38 |
hatch | I am not sure we will use the more advanced sass stuff | 03:38 |
hatch | but it would be nice to use for things like your animations and whatnot | 03:39 |
frankban | morning bac: could you please take a look at https://codereview.appspot.com/32760043 ? | 12:39 |
bac | sure frankban | 12:42 |
frankban | bac: thanks | 12:43 |
=== gary_poster|away is now known as gary_poster | ||
bac | frankban: the code looks fine, working on qa | 13:16 |
frankban | bac: great thanks | 13:16 |
bac | frankban: following your (excellent) instructions i encountered this: http://paste.ubuntu.com/6479076/ | 13:58 |
bac | not sure why | 13:58 |
frankban | bac: looking | 13:58 |
frankban | bac: what does `which juju` return in the container? | 13:59 |
bac | doh | 13:59 |
bac | my custom go version | 13:59 |
frankban | bac: I see, indeed juju-core was not installed | 14:00 |
bac | nope | 14:00 |
bac | so it uses the existence of 'juju' on the path to decide whether to install the package? | 14:00 |
bac | maybe it should do a 'which juju' and ensure it is /usr/bin/juju | 14:00 |
frankban | bac: yes, the code uses "juju version" or similar. maybe we should make that check with the full path | 14:01 |
bac | frankban: yay, for my non-standard system to break qa | 14:01 |
frankban | :-) updating the branch, so that you can continue the QA after pulling it | 14:02 |
bac | after removing my custom juju and re-running, it did install juju-core and re-used the ec2 env | 14:02 |
frankban | bac: changes pushed, now juju is always used with the full path | 14:07 |
bac | cool | 14:08 |
frankban | bac: delta here: https://codereview.appspot.com/32760043/diff2/1:20001/quickstart/app.py | 14:10 |
bac | frankban: looks good. i | 14:11 |
frankban | cool | 14:11 |
bac | am just waiting for the qa to finish. not going to restart with new patch. | 14:11 |
frankban | bac: sounds good | 14:12 |
frankban | guihelp: I need another review (no QA) for https://codereview.appspot.com/32760043 ? Anyone available? | 14:12 |
* gary_poster on calls | 14:13 | |
rick_h_ | frankban: sure thing, sec | 14:13 |
frankban | rick_h_: thank you | 14:13 |
bac | frankban: it seems it is trying to re-use an existing ec2, even though juju status does not show it started | 14:13 |
bac | frankban: i wonder if that is an effect of it starting with my custom juju? | 14:14 |
bac | frankban: aws console shows no instances running | 14:15 |
bac | frankban: but i get http://paste.ubuntu.com/6479151/ | 14:15 |
bac | frankban: destroying the (non-existent) environment caused quickstart to bootstrap a new one the next time | 14:17 |
frankban | bac: could you please 1) juju destroy-environment -e ec2 2) pull the branch, 3) re-run quickstart. | 14:17 |
frankban | heh\ | 14:17 |
frankban | bac: for some reason juju got confused | 14:18 |
frankban | bac: maybe a bug in your compiled version of juju | 14:19 |
bac | frankban: ok, qa done successfully. | 14:25 |
frankban | bac: thanks a lot | 14:26 |
bac | frankban: np. | 14:26 |
bac | frankban: thanks for the good lxc primer. i may clip that and squirrel it away | 14:27 |
frankban | cool :-) | 14:28 |
frankban | bac: my last discovery was "sudo lxc-ls --fancy", which also returns all the ip addresses in the container's network namespace | 14:31 |
bac | um, --fancy is fancy | 14:32 |
bac | rick_h_: i tried to reproduce 1251420 on friday but had no luck. do you actually see that gigantoid error message in the notifications? | 14:34 |
rick_h_ | bac: yes, the value for "Error" is stuck into a notifcation with the text (something like) Deploy id: X failed. There was an error: <Env Error - Details... | 14:35 |
frankban | bac: using that, it is possible to ssh into an lxc, for instance, with a function like http://pastebin.ubuntu.com/6479241/ | 14:35 |
rick_h_ | frankban: you duped that correct? Wasn't it from having a constraint set to '' ? | 14:35 |
frankban | rick_h_: uhm... not sure if that generated the empty error... | 14:37 |
rick_h_ | frankban: that wasn't the empty error, that was the "casting a EnvError to string is fugly" error | 14:37 |
frankban | rick_h_: to dupe that I suppose we want a bundle with invalid data, e.g. an invalid value for an option, or invalid constraints yeah | 14:38 |
bac | rick_h_: i modified one of my bundles to have an invalid constraint and dragged it locally onto an ec2 canvas. that should've aggravated it, no? | 14:43 |
rick_h_ | bac: in a live env? | 14:43 |
rick_h_ | bac: and did you get anything back? The websocket should show the traffic and that dump is from the websocket connection | 14:44 |
bac | rick_h_: yeah | 14:44 |
bac | rick_h_: i added '''constraints: "cpu-cores=" ''' to mediawiki | 14:44 |
rick_h_ | bac: I mean if you didn't get that, did you get something else? Did it work? | 14:44 |
rick_h_ | bac: that might read as undefined, try "cpu-cores=''" | 14:44 |
bac | rick_h_: istr that it did not work and i did not get the error. will spin up again | 14:44 |
bac | rick_h_: sorry, my eyesight is not good enough to parse that. | 14:45 |
bac | is that =<singlequote><singlequote> | 14:45 |
bac | or =<dq><dq> | 14:45 |
rick_h_ | bac: rgr, set the constraint to an empty string | 14:45 |
rick_h_ | single-single | 14:45 |
rick_h_ | sec, looking for the old rev of jcastro's bundle that did it | 14:46 |
bac | rt | 14:46 |
rick_h_ | http://bazaar.launchpad.net/~jorge/charms/bundles/jenkins/bundle/revision/4 | 14:46 |
rick_h_ | that was erroring, so he removed them entirely | 14:46 |
rick_h_ | bac: so maybe you can try to get his bundle at older revisions and use it | 14:47 |
bac | rick_h_: are you saying r4 was bad in the right way? | 14:51 |
rick_h_ | bac: I believe so | 14:51 |
bac | rick_h_: ok | 14:51 |
rick_h_ | bac: sorry for the vagueness of memory here | 14:51 |
rick_h_ | bac: but I know he fixed them by removing the constraints and it caused the env to blow up with an invalid value | 14:51 |
frankban | rick_h_: your two suggestions are very good, thank you!. Is it ok if I create cards? one refers to pre-existing code (lxc) and the other seems out of scope for this branch (settings.COMMANDS) | 14:51 |
rick_h_ | frankban: rgr | 14:51 |
frankban | rick_h_: cool | 14:51 |
bac | rick_h_: i'll try my changes to local muletrain and see what happens first | 14:51 |
rick_h_ | bac: yea, sorry I bet it's the right constraing | 14:52 |
rick_h_ | I think the cpu-cores is cast to an int before going to the client | 14:52 |
rick_h_ | so maybe that's why it's not failing for you | 14:52 |
rick_h_ | while the branch from jorge has all of them set to empty and not all were valid/cast to a number | 14:52 |
jcastro | sweet, so I get to add constraints back to all my bundles? | 14:58 |
rick_h_ | jcastro: yea, but they can't be those empty ones or it'll error. | 14:58 |
rick_h_ | they have to be legit | 14:58 |
hatch | jcastro the ghost charm has been pushed to lp and a bug created to get it into charmers.....question though...do I need to re-submit it to charmers for every update? | 14:59 |
jcastro | I like the idea of having empty ones, makes it easy to figure out "oh neat, I can tune this" | 14:59 |
hatch | 2 legit to quit | 14:59 |
jcastro | hatch, from then on you'll do a merge proposal and we'll autoget it in the queue | 14:59 |
hatch | ahh gotcha | 14:59 |
jcastro | after X amount of times we can hand it over to you as a maintainer | 14:59 |
jcastro | but at first we like to peer review | 15:00 |
hatch | I noticed that the 'owner' of the charm isn't anywhere in the jujucharms.com listing | 15:00 |
hatch | should we not have a field to contact the owner? | 15:00 |
jcastro | huh, I thought we had the maintainer field showing up? | 15:00 |
hatch | https://jujucharms.com/precise/mongodb-18/ | 15:01 |
hatch | once they get recommended that goes away | 15:01 |
jcastro | ah, maybe that's on purpose then? | 15:02 |
hatch | not sure - I think we should have it though to encourage discussion between the user and developer | 15:02 |
jcastro | I don't remember why we took it off | 15:03 |
jcastro | maybe to encourage people to discuss things like that on the general list instead of email? | 15:03 |
hatch | hmm maybe | 15:04 |
* hatch adds maintainer to the UI, doesn't tell anyone | 15:05 | |
rick_h_ | hatch: stop that, it's a long standing argument | 15:12 |
rick_h_ | the maintainer and the cake is a lie | 15:13 |
hatch | lol - well I can't help it if I'm right | 15:13 |
TheMue | gary_poster: ping | 15:34 |
gary_poster | TheMue, hi | 15:45 |
TheMue | gary_poster: i'm currently working on the migration of the debug-log command from using ssh to the api | 15:49 |
gary_poster | TheMue, ah, cool! | 15:50 |
gary_poster | jujugui call in 10 | 15:50 |
hatch | ooo nice! | 15:50 |
TheMue | gary_poster: here i wanted to ask you what you/the gui expects from the command | 15:50 |
gary_poster | TheMue, great! Thank you. May I ping you after our team call and we can try to have a call? So, maybe in 30 minutes or so? | 15:52 |
TheMue | gary_poster: sure, will be here | 15:52 |
gary_poster | hatch did you ever write your reply about the local charm integration? | 15:52 |
gary_poster | thank you TheMue | 15:52 |
TheMue | gary_poster: yw | 15:52 |
hatch | sorry I'll finish it now | 15:53 |
gary_poster | cool thanks hatch, np. | 15:53 |
gary_poster | jujugui call in 2 | 15:58 |
gary_poster | TheMue, you available now? | 16:40 |
gary_poster | TheMue, https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj0 | 16:40 |
Makyo | frankban, lmk when you have a moment for pre-imp | 16:40 |
hatch | gary_poster would you mind seeing if you can reproduce this bug https://bugs.launchpad.net/juju-gui/+bug/1254903 it's very odd how different people aren't able to | 16:49 |
_mup_ | Bug #1254903: Sidebar does not open if GUI is loaded in the background <juju-gui:Triaged> <https://launchpad.net/bugs/1254903> | 16:49 |
gary_poster | hatch, k on it | 16:50 |
frankban | Makyo: I am available now | 16:54 |
gary_poster | hatch, can't dupe. also, we know from tests that some aspects of the code require that the browser be visible. That seems fairly reasonable to me. Even if we decide we can sufficiently dupe it, I'd be tempted to triage it low at the most. | 16:54 |
Makyo | frankban, https://plus.google.com/hangouts/_/7ecpi5n825fv8nflmovv6gqa4c?hl=en | 16:54 |
rick_h_ | gary_poster: the focus issues are tests around FF and the valueChange YUI event since it only polls when the input has focus. Focus shouldn't be an issue here unless specific browser versions were trying to cut corners by not rendering invisible bits while not looking a the page. | 16:55 |
hatch | gary_poster yeah I am not too concerned with it as far as it being a bug goes - it's just a very curious bug with how not-repeatable it is for some | 16:56 |
rick_h_ | but we can't dupe in a few diff verisons/machines so seems hit/miss/too browser specific atm imo | 16:56 |
gary_poster | rick_h_, hatch, ack | 16:56 |
jcastro | hey frankban | 17:09 |
frankban | jcastro: hi | 17:10 |
jcastro | do we have a bug for allowing passing constraints from quickstart? | 17:10 |
jcastro | like, say I want an all in one ghost bundle | 17:10 |
jcastro | and I want everything on node 0 | 17:10 |
jcastro | but I'm going to want more resources than the standard default | 17:10 |
frankban | jcastro: so those are bootstrap node constraints | 17:11 |
jcastro | right | 17:11 |
hatch | oo that would be very cool | 17:13 |
hatch | juju quickstart ghost all-on-one-superpowermode | 17:13 |
hatch | oh sorry | 17:13 |
hatch | juju quickstart ghost --all-on-one-superpowermode | 17:13 |
hatch | :D | 17:14 |
frankban | jcastro: quickstart does not support that. AFAICT provider constraints are specified in the envs.yaml file. We could add that functionality as part of our environment creation story | 17:14 |
jcastro | yeah because for small bundles, it still sucks to have a node just for orchestration | 17:14 |
jcastro | frankban, ok I'll file a wishlist bug then | 17:14 |
hatch | I'm really happy we are now getting features for the little guy | 17:15 |
jcastro | indeed | 17:15 |
jcastro | the little guy drives adoption | 17:15 |
hatch | that! | 17:15 |
frankban | jcastro: hum, they can also be passed as bootstrap args, so a quick fix could be adding a --constraints arg to quickstart | 17:15 |
jcastro | "you think we did a good job with your ghost blog? wait until you see what we can do with your stuff at your job!" | 17:15 |
hatch | haha | 17:16 |
jcastro | frankban, yeah so maybe just allowing quickstart to pass arguments to deployer underneath? | 17:16 |
jcastro | that also sounds like less work! | 17:16 |
frankban | jcastro: well, to pass argument to juju, not the deployer. I have a question, what is the behavior of a bundle which deploys a service to machine 0 if the local provider is used? | 17:17 |
jcastro | I was not thinking of local | 17:17 |
jcastro | that is a good question | 17:18 |
gary_poster | frankban, hatch, others, I'm planning to add these mock function and method test helpers to the GUI tests. feedback welcome. http://pastebin.ubuntu.com/6480086/ | 17:53 |
hatch | cool I'll take a look in a bit | 17:55 |
hatch | gary_poster I took a look real quick - it looks like you're replicating some of the functionality of http://sinonjs.org/ | 17:57 |
hatch | maybe we want to implement that instead? | 17:59 |
gary_poster | hatch, maybe so. ugh. I just want to get my branch finished. Adding the dependency is more annoying, unfortunately. Didn't you or someone else look into this once before? | 18:00 |
hatch | we did - at the time we didn't have any real need for it :) | 18:01 |
hatch | I'm not saying we shouldn't land your code | 18:01 |
gary_poster | I just have a need in one test | 18:01 |
hatch | just that we should probably look into this type of lib more seriously | 18:01 |
hatch | because it could simplify things I bet | 18:01 |
gary_poster | agreed | 18:01 |
frankban | gary_poster, hatch: +1 those two functions look good, perhaps we could have another card to evaluate introducing a mock library | 18:03 |
gary_poster | good idea +1 | 18:04 |
gary_poster | thanks both | 18:04 |
* hatch looks at the bug list, board, backlog..... | 18:05 | |
* hatch bows head | 18:05 | |
hatch | lol | 18:05 |
gary_poster | :-) | 18:05 |
* Makyo walk dogs. Maybe they'll stop being such brats :T | 18:12 | |
hatch | the relations need to move under the service model like the units did | 18:20 |
gary_poster | not sure I agree | 18:20 |
gary_poster | units are definitely children of services | 18:20 |
gary_poster | relations have two masters | 18:20 |
hatch | hmm | 18:21 |
hatch | good point | 18:21 |
benji | well, that's a new one: "no reachable servers" | 18:22 |
hatch | benji sounds like you are having a bad week | 18:23 |
benji | it might sound that way, but I'm enjoying myself at the moment | 18:23 |
hatch | a service should have reference to the relation object.....that's how it should be done | 18:23 |
hatch | right now they are too separate imho | 18:23 |
gary_poster | sounds like a potentially nice change. | 18:34 |
hatch | jujugui looking for review and qa https://codereview.appspot.com/33130043/ plz and thanks | 18:47 |
hatch | and performance card gets pushed down again....poor performance card | 18:49 |
hatch | forget that, I seem to have found a bug in the real env qa | 19:09 |
hatch | if I'm working on the source in the charm how do I get it to re-build? | 19:12 |
hatch | sorry I'm totally blanking | 19:12 |
rick_h_ | hatch: elaborate please? 'working on the source in the charm' you working on the gui source js from a live env? | 19:19 |
hatch | yeah | 19:19 |
rick_h_ | hatch: you shouold be able to turn on debug mode to get it to use the debug files | 19:19 |
hatch | that's what I thought too | 19:20 |
rick_h_ | then you can edit those and it'll be fresh on each page load | 19:20 |
hatch | well the files in juju-gui-source need to be built somehow | 19:21 |
hatch | and all of my my make commands in there just hang | 19:21 |
rick_h_ | well the charm takes care of the build and getting the uncompressed files into place. They're just in the build-debug vs build-prod | 19:22 |
rick_h_ | and no re-making should be necessary | 19:22 |
hatch | yeah it's definitely not working because I put an alert at the top of app.js and nothing happens on refresh | 19:23 |
hatch | the gui is also not hitting any of my breakpoints | 19:23 |
rick_h_ | which sources is it pull? | 19:23 |
rick_h_ | compressed all.js? | 19:23 |
rick_h_ | or each individual file? | 19:23 |
hatch | the individual files are here under juju-ui | 19:24 |
rick_h_ | hatch: right, but in the network tab of your browser, which files are loading | 19:24 |
hatch | all of them | 19:25 |
rick_h_ | each single file? | 19:25 |
rick_h_ | you're not being clear :P | 19:25 |
hatch | when I try to put a breakpoint in, say the service-relations.js it puts it at the bottom of the file as if it hasn't been parsed | 19:25 |
hatch | yes, every single file | 19:25 |
rick_h_ | ok, so then do a sudo updatedb and locate a filename and make sure you're editing the right copy | 19:26 |
rick_h_ | there's going to be a couple of hits in there | 19:26 |
hatch | /var/lib/juju/agents/unit-juju-gui-0/charm/juju-gui-source/app/app.js | 19:27 |
rick_h_ | hatch: that's the only copy of app.js on the system? | 19:27 |
rick_h_ | there should be a build dir based one | 19:27 |
hatch | well the other one is the release version | 19:27 |
hatch | the build dir based one are just links to that | 19:28 |
rick_h_ | so /var/lib/juju-gui/release/juju-gui-0.13.0/build-debug/juju-ui/app.js is the file ot edit | 19:28 |
hatch | but that's not the branch I'm using | 19:28 |
hatch | oh | 19:28 |
hatch | thats the issue | 19:28 |
rick_h_ | well it's the path in my deploy I've got here that is | 19:28 |
rick_h_ | adjust to your charm/deploy/setup | 19:29 |
hatch | I used a different source directory and the charm serves up the release one on debug | 19:29 |
hatch | well that's an odd issue | 19:29 |
rick_h_ | just keep editing app.js files until you find which one it's using :P | 19:29 |
hatch | that would make sense as to why it's not being parsed | 19:29 |
rick_h_ | then file a bug on the charm after that | 19:29 |
hatch | yeah... this is definitely a bug | 19:30 |
rick_h_ | if it's not acting like you're expecting, but there are only so many files to test which it's serving | 19:30 |
hatch | yeah - it's serving the wrong files | 19:31 |
hatch | well this will be interesting to debug | 19:32 |
hatch | going to grab some lunch | 19:36 |
hatch | bbl | 19:36 |
gary_poster | hatch, not serving wrong files. Supposed to be elsewhere. | 19:40 |
gary_poster | can talk later | 19:40 |
gary_poster | oh reread. may be wrong, yes | 19:47 |
bac | hey gary_poster can we whack the juju-gui trunk series (https://launchpad.net/juju-gui/trunk) or change the charm to not recognize it? | 20:08 |
bac | juju set juju-gui "juju-gui-source=trunk" | 20:08 |
bac | does not do what you might think it does | 20:08 |
bac | and it is very disturbing | 20:08 |
gary_poster | bac, I'm OK with it but would prefer if it were renamed, not removed. | 20:09 |
gary_poster | but I've used it only rarely | 20:09 |
bac | old-stinky-trunk | 20:09 |
gary_poster | so maybe ripping it out until we need it in anger again is reasonable | 20:09 |
gary_poster | lol | 20:09 |
benji | jujugui: I have a charm branch up that adds randomized service names and tweaks the retry decorator at https://codereview.appspot.com/33230043/ | 20:22 |
benji | there are even some nice pre-review comments for whomever looks at it, going fast! | 20:22 |
bac | benji: i'll look at it | 20:22 |
benji | thanks | 20:22 |
hatch | ahoy | 20:26 |
hatch | ahah fixed | 20:27 |
rick_h_ | woot! https://github.com/CanonicalJS/juju-gui/pull/2 and http://ec2-54-196-48-122.compute-1.amazonaws.com:8080/job/juju-gui-merge/15/console | 20:30 |
hatch | lookin goooood | 20:31 |
gary_poster | sweet, rick_h_ ! I like the Juju GUI icon too ;-) | 20:31 |
hatch | oh that's what that is? | 20:31 |
rick_h_ | hah, that's just a default github thing | 20:31 |
hatch | lol | 20:31 |
gary_poster | oh ok | 20:31 |
rick_h_ | but yea, I totally did that. I rock boss :) | 20:31 |
gary_poster | heh | 20:32 |
hatch | rick_h_ are the symbol's necessary? | 20:32 |
hatch | $$merge$$ | 20:32 |
rick_h_ | hatch: the $$? | 20:32 |
hatch | yeah | 20:32 |
rick_h_ | yea, I did that for now since the check is just a 'is $$merge$$ in the comment' | 20:32 |
hatch | ahh | 20:32 |
rick_h_ | but yea, make a pull request, add a $$merge$$, I reload the page on the small web service, and it triggers a merge build, test, and that then does an api merge once it passes | 20:34 |
bac | benji: done | 20:36 |
hatch | rick_h_ so if two branches are in the queue, A and B, and A fails, but B passes, will B get merged in ahead of A? | 20:36 |
benji | bac: cool, thanks | 20:36 |
rick_h_ | hatch: rgr | 20:36 |
rick_h_ | hatch: each runs on it's own, it merges develop (trunk) first and then runs tests | 20:36 |
hatch | coolio | 20:36 |
rick_h_ | hatch: so basically A + trunk fails, but B + trunk works, then B + trunk gets merged and A needs more love | 20:36 |
hatch | awesome | 20:37 |
rick_h_ | and jenkins makes sure only one runs at a time since it had to go through the single juju-gui-merge job, and that doesn't watch for changes, it had to be manually triggered via the web service which is the gatekeeper to develop (trunk) | 20:37 |
hatch | ahh cool, so we require jenkins and a webservice? so two machines? | 20:38 |
rick_h_ | no, right now the goal is to run the webserver on the jenkins machine | 20:38 |
rick_h_ | it can run on port 80/443 or whatever since jenkins runs on 8080 by default | 20:38 |
hatch | ahh ok - so can we run this on prodstack? | 20:39 |
rick_h_ | heh, not any time soon. It needs the latest jenkins and such. It's not charmed, lots of tweaking in the jenkins install to get it to work | 20:39 |
rick_h_ | right now the goal is to run it in ec2 land | 20:39 |
hatch | ahh, well we should be able to run it on a free node | 20:40 |
rick_h_ | :/ slow tests? and eventually we'll have more things in here. Not sure on that | 20:41 |
hatch | hmm that is a good point | 20:43 |
hatch | ec2 is just pretty expensive for an 'always on' server | 20:44 |
rick_h_ | well, not for reserved instances and it'll be a lot cheaper than trying to get everything on prodstack | 20:44 |
rick_h_ | testing automated code pulls from github and such breaks a BUNCH of egress filewall, executing random code, etc rules | 20:45 |
gary_poster | jujugui, I beg your indulgence of two reviews of this monster, with my apologies: https://codereview.appspot.com/33290043 | 20:45 |
Makyo | gary_poster, on it | 20:45 |
gary_poster | it adds authtoken suport | 20:45 |
gary_poster | thank you | 20:45 |
gary_poster | I need one QA as usual | 20:45 |
hatch | rick_h_ yeah I suppose you're right, a reserved medium is $50/mo a lot cheaper than the sysadmin/dev time to put on prodstack :) | 20:46 |
gary_poster | maybe? :-) | 20:46 |
gary_poster | ok biab | 20:46 |
benji | gary_poster: I'd like to review it but don't have enough time left in the day to digest that thing :) | 20:46 |
hatch | haha I suppose that $50 never ends | 20:46 |
gary_poster | benji, understood and thanks for looking | 20:47 |
hatch | oh and I guess that's +bandwidth | 20:48 |
hatch | ok jujugui https://codereview.appspot.com/33130043/ all fixed ready for review/qa | 20:59 |
gary_poster | hatch, ok. trade you reviews? Please? :-) | 21:00 |
hatch | wasn't Makyo doing yours? | 21:00 |
gary_poster | hatch I most definitely need 2 :-P | 21:00 |
hatch | well in that case you got-a-deal | 21:01 |
gary_poster | cool thank you :-) | 21:01 |
hatch | oh jeebz | 21:01 |
gary_poster | lol | 21:01 |
hatch | man that was a horrible deal | 21:01 |
hatch | haha | 21:01 |
gary_poster | lol, pretty much | 21:01 |
* benji shorts hatch's review futures and calls it a day. | 21:05 | |
hatch | darn | 21:06 |
gary_poster | hatch, not your branch (also in comingsoon) but why does deleting units (50 -> 1) cause a notification error, do you know? | 21:25 |
hatch | gary_poster nope, I THINK I created a bug already for it | 21:34 |
hatch | one sec | 21:34 |
hatch | gary_poster https://bugs.launchpad.net/juju-gui/+bug/1253113 | 21:34 |
_mup_ | Bug #1253113: Reducing the number of units in inspector on sandbox throws error <juju-gui:New> <https://launchpad.net/bugs/1253113> | 21:34 |
gary_poster | ack thanks hatch | 21:34 |
hatch | gary_poster http://yuilibrary.com/yui/docs/api/classes/YUI.html#method_getLocation the safari bug in question is fixed but we may want to test to be sure anyways | 21:40 |
gary_poster | ack hatch thanks. you have a safari handy? ;-) | 21:42 |
hatch | haha that I do | 21:43 |
gary_poster | hatch :-) I'll look at it if I still have time after your review | 21:43 |
hatch | authenticated = token === submittedToken; | 21:43 |
hatch | haha i had to read this a few times | 21:44 |
hatch | go javascript! | 21:44 |
gary_poster | lol | 21:44 |
gary_poster | sorry. parens help? | 21:44 |
hatch | it could but that's pretty trivial | 21:44 |
hatch | :) | 21:44 |
hatch | some day we will split all this login business into it's own class so we don't need to instantiate app | 21:48 |
hatch | instantiating app is so slow | 21:48 |
gary_poster | hatch so far LGTM with trivial and QA OK in sandbox. waiting for ec2 (lxc doesn't work for me :-( ) | 21:49 |
hatch | the branches I've been working on have gotten a lot of use out of 'failtester' :D | 21:49 |
gary_poster | actually hatch, I was surprised and pleased that the new giganto test I had that used the mock functions (and instantiated app) was pretty fast | 21:50 |
gary_poster | but I still like that idea so we have less to understand at once | 21:50 |
hatch | oh that's good - it looks like the stubs work out really well here | 21:51 |
gary_poster | yeah I was pleased with how that worked out. I didn't see anything that did what I wanted in that other library, but I was just skimming | 21:52 |
huwshimi | Morning | 22:05 |
hatch | shh he's here | 22:05 |
hatch | oh hi huwshimi | 22:05 |
hatch | lol | 22:05 |
gary_poster | see what you did?! | 22:05 |
hatch | haha woops | 22:05 |
huwshimi | hatch: Hey | 22:05 |
hatch | lol | 22:05 |
huwshimi | :) | 22:05 |
hatch | that was pretty funny | 22:06 |
huwshimi | hatch: I just run away from my problems :) | 22:07 |
hatch | lol | 22:07 |
gary_poster | heh :-) hey huwshimi | 22:07 |
huwshimi | gary_poster: Hey | 22:07 |
gary_poster | huwshimi, did you get my flighty email last night? | 22:08 |
gary_poster | or yesterday morning :-P | 22:08 |
huwshimi | gary_poster: Oh, I did, I completely forgot to ask for clarification | 22:08 |
gary_poster | heh, np | 22:08 |
huwshimi | gary_poster: I wasn't quite sure what the issue was. | 22:08 |
gary_poster | huwshimi, lemme see if I can come up with instructions 1 sec | 22:10 |
huwshimi | thanks | 22:10 |
gary_poster | huwshimi, I came up with instructions, which demonstrated that I was crazy and everything is fine. ;-) nevermind me and sorry for the confusion. Other tasks from spreadsheet ok? | 22:13 |
huwshimi | gary_poster: heh | 22:15 |
huwshimi | gary_poster: Yeah, working on the sticky header on the charm/bundle details panel at the moment | 22:15 |
gary_poster | hatch, so...relation should show as bad only on one side? | 22:16 |
huwshimi | gary_poster: Nearly done, but I have an annoying bug that I'm sorting out | 22:16 |
hatch | gary_poster right | 22:16 |
gary_poster | ok cool hatch. QA OK then. | 22:16 |
gary_poster | thank you! | 22:16 |
hatch | service A hook may be ok but service B hook may have failed | 22:16 |
hatch | awesome thanks | 22:16 |
hatch | gary_poster can you clarify which 'peer relation issues' you are referring to https://codereview.appspot.com/33130043/diff/20001/app/models/models.js | 22:19 |
gary_poster | hatch, I was trying to figure out what bug you were fixing | 22:19 |
gary_poster | seemed like you were addressing the fact that there might not be another side of relationship | 22:20 |
hatch | ohh - well if the first unit created was in error, it would skip over trying to figure out what was wrong because the 'error' check was only after the first unit in error | 22:20 |
hatch | it was a legacy issue, must have been there since day 1 | 22:21 |
gary_poster | ah! | 22:21 |
gary_poster | ok | 22:21 |
hatch | because that's really old code | 22:21 |
gary_poster | cool | 22:21 |
gary_poster | biab to see if I can land :-) | 22:26 |
hatch | so the Dell Precision M3800 is $1200 more expensive in Canada | 22:38 |
hatch | w t f? | 22:38 |
hatch | http://www.dell.com/ca/business/p/precision-m3800-workstation/fs vs http://www.dell.com/us/business/p/precision-m3800-workstation/fs | 22:39 |
hatch | gary_poster your peer relation question got me thinking so I checked it out....looks like there is a small UI bug with them now https://bugs.launchpad.net/juju-gui/+bug/1255336 I'll work on fixing that before moving forward since I have already landed the original branch | 22:44 |
_mup_ | Bug #1255336: Relation status indicator in inspector incorrectly placed on peer relations <juju-gui:New> <https://launchpad.net/bugs/1255336> | 22:44 |
gary_poster | cool thanks hatch | 22:46 |
rick_h_ | hatch: check out joining the Linux foundation if you're buying as well as you get a nice discount | 22:59 |
rick_h_ | hatch: from dell, hp, or lenovo | 22:59 |
hatch | well right now it's a tossup between the dell or apple | 22:59 |
hatch | I don't think lenovo or hp make a comparable laptop | 23:00 |
* gary_poster scoffs at dell | 23:00 | |
hatch | well the dell has the touch screen and is a little cheaper | 23:00 |
hatch | but the apple is probably better | 23:00 |
gary_poster | yeah, I like touch screens :-/ | 23:00 |
hatch | lol | 23:00 |
gary_poster | :-) | 23:00 |
gary_poster | Makyo, thank you again for working on my branch. how is QA going? | 23:01 |
Makyo | gary_poster, real env now. Left it running while I was out of town - go me! | 23:01 |
gary_poster | ugh :-P :-) | 23:02 |
hatch | lol u and real envs | 23:02 |
Makyo | Siiigh. | 23:04 |
hatch | I just had another service-bounce-back issue....but now I cant reproduce it...damn this bug just wont die hah | 23:04 |
Makyo | I was all proud of myself for not looking at my computer during the whole trip, too. | 23:05 |
hatch | jujugui can a peer relation go bad? or is it always good? | 23:05 |
gary_poster | hatch, I think it is has a hook, which means it can go bad | 23:06 |
hatch | ok so I'm going to set any peer relation to have the name 'peer' | 23:06 |
hatch | else there isn't really anywhere to put the status indicator | 23:07 |
hatch | or I suppose it could be the services name | 23:08 |
hatch | that's probably more accurate | 23:08 |
hatch | as it's the role which is peer not the name | 23:08 |
Makyo | gary_poster, LGTM | 23:17 |
gary_poster | awesome, thanks very much Makyo! | 23:17 |
gary_poster | I'm excited to see this in the quickstart | 23:18 |
* Makyo destroys environment >:/ | 23:18 | |
hatch | as am I | 23:18 |
hatch | quickstart is so cool | 23:18 |
gary_poster | agreed | 23:18 |
hatch | was just quoted $425 for my bathroom window | 23:20 |
hatch | jeesh it's barely 2ft square | 23:20 |
hatch | to home depot I guess hah | 23:21 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!