rick_h_ | hatch: have the chrome dev summit videos not been put on youtube? I wanted to catch up but :( | 01:08 |
---|---|---|
hatch | rick_h_: not sure I haven't seen anything yet | 01:12 |
rick_h_ | hatch: yea, found the 'live stream' day one which is an 8hr video | 01:13 |
bac | hi frankban, i qa'd matt's branch and landed it. i did note some issue but wasn't sure whether they were blockers for the branch. it moves us forward. | 12:35 |
bac | s/issue/issues/ | 12:35 |
frankban | bac: yes I saw that, thank you! we have cards for setting up ssh keys and the envs.yaml file. Matt's branch is just the first step, so I guess that's ok. time for two reviews? | 12:37 |
bac | frankban: sure | 12:37 |
frankban | bac: thanks. quick one: https://codereview.appspot.com/30810043. more complex one: https://codereview.appspot.com/30760043 | 12:38 |
marcoceppi | Good morning everyone o/ I'm curious how the sandbox mode/simulator mode works in the gui. I'd like to leverage portions of it for a personal project (maintaining fake juju status in javascript), is it relatively pluggable or is it highly atuned to the gui? | 12:40 |
rick_h_ | marcoceppi: hmm, so it'd be big. It kind of work together with the db setup, the JS models. | 12:45 |
rick_h_ | marcoceppi: we could probably get a hangout to walk through it sometime. | 12:45 |
marcoceppi | rick_h_: cool | 12:45 |
bac | frankban: make check shows incomplete test coverage | 12:50 |
bac | frankban: on your first review | 12:50 |
frankban | bac: that's fixed by the second branch in review | 12:50 |
bac | frankban: ok | 12:50 |
bac | frankban: does quickstart attempt to open the user's default browser or firefox? my default is chromium (as set in 'System Info') but QS opens FF. | 13:07 |
frankban | bac: it uses webbrowser.open, so it should be the default browser, for some definition of "default" | 13:07 |
bac | huh | 13:08 |
frankban | bac: http://docs.python.org/2/library/webbrowser.html#webbrowser.open | 13:08 |
=== gary_poster|away is now known as gary_poster | ||
gary_poster | hi everybody | 13:15 |
frankban | morning gary_poster | 13:16 |
gary_poster | :-) | 13:16 |
frankban | guihelp: anyone available for another review of https://codereview.appspot.com/30760043 (quickstart)? | 13:16 |
gary_poster | I will after mail skim if no one else takes it | 13:17 |
bac | frankban: i was just about to do it. or are you asking for a second? | 13:17 |
frankban | bac: I was asking for a second. thank you | 13:17 |
* bac continues | 13:17 | |
frankban | thanks gary_poster | 13:17 |
gary_poster | welcome | 13:17 |
rick_h_ | I can look frankban | 13:25 |
frankban | rick_h_: cool thanks | 13:27 |
frankban | gary_poster: ^^^ | 13:27 |
gary_poster | ok cool thanks rick_h_ and frankban | 13:27 |
rick_h_ | the coffee it brewed and it's friday! | 13:28 |
gary_poster | :-) yay | 13:28 |
gary_poster | ok email is not exactly tamed, but the wild animals are at least identified | 13:33 |
rick_h_ | shoot to kill! | 13:33 |
gary_poster | heh | 13:34 |
* frankban biab | 13:42 | |
bac | rick_h_: you are doing frankban's review? i'm starting QA so you can skip that part | 13:43 |
rick_h_ | bac: yes, thanks | 13:44 |
benji | why are there paths that start with "./" all over the charm Makefile? | 14:14 |
hatch | save the dots | 14:16 |
rick_h_ | they look cool! | 14:16 |
bac | hatch: ha! | 14:18 |
hatch | :D | 14:20 |
benji | pfft | 14:21 |
hatch | the worst part of tests is fixing them all | 14:22 |
frankban | rick_h_: thanks for the review. I'd like charms.py to be replaced (or part of) a future bundles/charms library that the dpeloyer, charmworld and quickstart can share. But for the time being, I'd prefer for quickstart validation to avoid connecting to the network | 14:27 |
antdillon | rick_h_, Hey Rick, you about? | 14:27 |
rick_h_ | frankban: ok, that's kind of what I was thinking as well. Maybe just a bug to mark integration that needs to occur? | 14:27 |
bac | rick_h_: i am not going to do frankban's QA step 3 as it will take too long with my connection. have a go at it if you want. | 14:27 |
rick_h_ | antdillon: yep | 14:27 |
rick_h_ | bac: ok, will try to go do that in a minute then. | 14:28 |
antdillon | rick_h_, Do you know if there is more api calls to manage.jujucharms.com/api/3/? | 14:28 |
rick_h_ | antdillon: http://charmworld.readthedocs.org/en/latest/api.html | 14:28 |
antdillon | rick_h_, Awesome thanks | 14:28 |
rick_h_ | antdillon: regenerating the docs as they've not been updated for a couple of weeks, so maybe refresh in 5 | 14:29 |
rick_h_ | antdillon: let me know if you've got any specific questions/api stuff you're looking for | 14:29 |
antdillon | rick_h_, Cool thanks | 14:29 |
frankban | rick_h_: sounds good. re changing the name of the check_charm_url utility. sounds good, maybe print_charm_url? | 14:30 |
antdillon | rick_h_, Just wanna pull a specific charms details in a call | 14:30 |
rick_h_ | frankban: yea, I mean maybe that's just me. I didn't have a great alternative. I just was confused for a minute. | 14:31 |
frankban | rick_h_: ok, I'll make the changes, and also add a XXX in charms.py e.g. reuse code (whenever we have reusable code) | 14:32 |
bac | frankban: review and most QA done | 14:32 |
rick_h_ | antdillon: ok, so that's just https://manage.jujucharms.com/api/3/charm/precise/mysql | 14:32 |
antdillon | rick_h_, Yep got it, thank man, the docs are col | 14:32 |
antdillon | cool* | 14:32 |
rick_h_ | antdillon: awesome, api ftw | 14:32 |
frankban | bac: thanks | 14:33 |
rick_h_ | bac: so I just need to do #3 to complete qa? | 14:35 |
bac | rick_h_: that is correct | 14:35 |
rick_h_ | frankban: ok, #3 is good, qa is all ok | 14:40 |
frankban | rick_h_: great, thank you! | 14:40 |
hatch | uuuuuuugh someone else wana fix these tests for me? Free coffee available in your kitchen if you do! | 14:42 |
rick_h_ | hatch: oh come on, it's an oppertunity. I'll let you take it | 14:42 |
rick_h_ | you broke it you bought it :P | 14:42 |
frankban | rick_h_: how much time is required in your opinion to produce that kind of python charms/bundles validation library? the number of projects that can use that is very high: the guiserver, proof, quickstart, charmworld... | 14:43 |
rick_h_ | frankban: well most of it's written, so it's a matter of refactoring it out into a library and getting it back into all the tools. Maybe a week if all goes well? | 14:44 |
frankban | juju-deployer... | 14:44 |
rick_h_ | Maybe that's optimistic | 14:44 |
hatch | haha darn, well I tried | 14:44 |
rick_h_ | to pull the library out is definitely ok in under a week, it's the work to get it into the each of those tools (and there's a few as you note) and some we don't control | 14:44 |
rick_h_ | frankban: but yea, I think we all want to see this happen. The proof stuff, validation, etc. It's a hefy chunk we keep replicating. | 14:45 |
rick_h_ | and if those rules change it's going to suck for us | 14:45 |
frankban | rick_h_: yeah, maybe two weeks/one developer. yes, I don;t expect charms to change so often, but bundles definitely can | 14:46 |
rick_h_ | frankban: yea, but validation can get better. And with two stable releases coming I figure there will be some shake up. I'm not sure what that will end up being yet though. | 14:47 |
rick_h_ | I'm still curious if charms will be like py3. People try to write a charm that just works on each LTS and submit it to both paths, or do two different charms and sync changes when required. | 14:47 |
frankban | rick_h_: describe_gui_charm_url... how does it sound? | 14:49 |
rick_h_ | frankban: hmmm, I think check is ok now that I look at it more. Sorry. I mean it's printing, some validation, and logging. I guess if I had hit the utils.py file first maybe it wouldn't have seemed so out of place | 14:51 |
frankban | rick_h_: from the charm author perspective, I don't think precise and trusty have too many differences (compared to py2/py3). E.g. we need to port the gui charm, and I guess we won't require a second code base for that | 14:52 |
frankban | rick_h_: great, I'll leave check | 14:52 |
hatch | bac: if you wouldn't mind, could you fire open Safari and let me know what version you have? | 15:10 |
bac | hatch: that's a very personal question. the answer of which is Version 7.0 (9537.71) | 15:11 |
bac | (i think that is a Star Date) | 15:11 |
hatch | thanks for opening up to me, Starfleet thanks you | 15:11 |
bac | anything else? shoe size? bagel topping preference? | 15:12 |
frankban | :-) | 15:14 |
hatch | lol nope that's it, I just needed to know what the most recent safari was on mavericks | 15:15 |
hatch | I don't have mavericks installed yet | 15:15 |
bac | frankban: are you going to get to bug 1252295 today? if not i'd like to take it. | 15:17 |
_mup_ | Bug #1252295: guiserver bundle deployment error is empty <juju-gui (Juju Charms Collection):Triaged> <https://launchpad.net/bugs/1252295> | 15:17 |
frankban | bac: the guiserver part of that bug is fixed: we now return something like "no further details available". I think hazmat is working on the deployer part (prevent the deployer to return exceptions without messages) | 15:19 |
bac | frankban: ah, ok. | 15:20 |
frankban | bac: FWIW, bug 1251420 could be interesting | 15:23 |
_mup_ | Bug #1251420: reporting an error from the environment isn't formatted well <juju-gui (Juju Charms Collection):Triaged> <https://launchpad.net/bugs/1251420> | 15:23 |
gary_poster | jujugui call in 10 | 15:50 |
gary_poster | jujugui call in 2 | 15:58 |
gary_poster | hatch, yo yo yo | 16:00 |
marcoceppi | gary_poster: got a few mins for a g+? | 16:35 |
gary_poster | marcoceppi, on call will ping | 16:36 |
marcoceppi | gary_poster: ack, thanks! | 16:36 |
frankban | benji: still lbox problems? | 17:08 |
benji | frankban: yep: "error: Failed to run "bzr push": exit status 3" | 17:09 |
benji | frankban: this is the right branch for the charm trunk, correct? bzr+ssh://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/ | 17:09 |
frankban | benji: yes | 17:10 |
frankban | benji: is this lbox propose? anything useful from -debug? | 17:11 |
benji | yep, "lbox propose"; I'll do a -debug run now | 17:11 |
benji | 2013/11/22 11:11:44 Branch URL inferred: lp:~benji/charms/use-known-pip | 17:12 |
benji | that doesn't look right | 17:12 |
benji | oh: bzr: ERROR: Permission denied: "~benji/charms/use-known-pip/": : Project 'charms' does not exist. | 17:12 |
frankban | benji: yes so lbox pushes the branch for you, and that's not the right location. | 17:14 |
benji | frankban: any hints as to a good location? | 17:15 |
frankban | benji: I use this entries in my ~/.bazaar/locations.conf: http://pastebin.ubuntu.com/6459507/ you can adjust them for your needs | 17:16 |
benji | frankban: ah! thanks; I bet I lost a similar entry in my bzr decimation | 17:17 |
frankban | benji: I guess lbox infers the location only if not set, so a quick fix could also be a push --remember, but the location file is a cool thing (I really hope git will have something similar) and automates things | 17:19 |
benji | yep, it is nice | 17:19 |
benji | grrr, lbox, it is OK if the first line of a commit message is over 50 characters! | 17:20 |
benji | it's like someone created an evil AI that loves haiku | 17:21 |
frankban | :-) | 17:21 |
benji | frankban: do you have time to look at the fruit of my lbox, er labor? https://codereview.appspot.com/31000043/ | 17:33 |
frankban | benji: sure | 17:33 |
gary_poster | marcoceppi, hey. ready. you have a hangout or do you want me to make one? | 17:40 |
marcoceppi | in a meeting gary_poster, will ping when I'm avail again :) | 17:40 |
hatch | jcastro: can you take a look at my ghost charm and let me know if there is anything I need to do to get it approved? Or should I just push it to my name and then we can 'upgrade' it at that point? | 17:40 |
gary_poster | marcoceppi, :-) k | 17:40 |
hatch | so...many....tabs | 17:42 |
marcoceppi | gary_poster: okay! https://plus.google.com/hangouts/_/76cpj7q9igbirg4g5ov1ifeups?hl=en | 17:45 |
jcastro | hatch, push it to your name, then file a bug against charms asking something like "I am ready for review!" and then assign the ~charmers team to the bug | 17:45 |
jcastro | that'll put you in the queue | 17:45 |
hatch | jcastro: ok cool, so should I blog about it once it's up or wait for charmers? | 17:47 |
jcastro | I prefer to wait for charmers because the namespace is cleaner rather than ~jorge/blah | 17:47 |
jcastro | but up to you | 17:47 |
jcastro | it it works might as well blog it, post it on their forum | 17:48 |
jcastro | and say something like "as soon as I pass peer review it'll just be `juju deploy ghost`, but in the meantime, GO DEPLOY THE WORLD." or something | 17:48 |
hatch | haha | 17:48 |
rick_h_ | hatch: there was a guy at CHC working with ghost and I mentioned your charm in the wings | 17:49 |
rick_h_ | hatch: kind of cool, he's a ghost fan | 17:49 |
hatch | jcastro: ok cool I'll make sure it proofs perfect this weekend and create a bug for it so that the review can be quick | 17:50 |
hatch | rick_h_: oh cool - I would like some other users input into what additional features I should add | 17:50 |
gary_poster | jujugui, charm review request #1 of in-progress effort: https://codereview.appspot.com/31020043 | 18:09 |
frankban | gary_poster: taking a look | 18:12 |
gary_poster | thanks frankban | 18:12 |
gary_poster | this is a follow-on to the branch frankban is reviewing, but I won't ask for a review until the first one is done: https://codereview.appspot.com/30980044 . In retrospect, maybe I shouldn't have made the review request. | 18:17 |
gary_poster | I mean, in rietveld | 18:17 |
bac | hey rick_h_ you know a particular bundle that induces the error we discussed? | 18:19 |
rick_h_ | bac: hmm, I think it was fixed. It was due to a bundle have constraints defined as cpu-cores='' | 18:19 |
rick_h_ | so the '' blew it up | 18:19 |
rick_h_ | since it needed to be an int | 18:20 |
bac | rick_h_: ok, cool, i'll sabotage one of mine | 18:20 |
frankban | jujugui, jcastro: new alpha version (0.4.2) of juju quickstart released in the quickstart beta ppa. It includes: support for setting up juju-core packages, local provider support, ability to reuse an existing environment/juju-gui, incrementing bundle deployments count, customized gui charm url validation. | 18:30 |
rick_h_ | frankban: woot! I was just looking to use the lxc support this morning | 18:30 |
gary_poster | awesome! thanks frankban | 18:30 |
hazmat | frankban, nice | 18:31 |
bac | in case anyone else needs it: sudo add-apt-repository ppa:juju-gui/quickstart-beta | 18:32 |
gary_poster | I'll copy it over to juju/stable | 18:32 |
frankban | thanks bac | 18:32 |
jcastro | hey frankban | 18:39 |
jcastro | are you going to announce on the list? | 18:39 |
jcastro | I am working on the quickstart /bundle docs this afternoon | 18:39 |
jcastro | and was going to announce them too | 18:39 |
frankban | jcastro: EOD for me, could you please give it a try and include it in your announcement? | 18:40 |
benji | frankban: thanks for the review | 18:40 |
frankban | benji: welcome | 18:40 |
jcastro | ok! | 18:40 |
frankban | thanks jcastro | 18:40 |
* gary_poster intends to take a late lunch and then realizes he has worked all the way through to next meeting. Happens more often than it should. :-/ | 18:51 | |
jcastro | gary_poster, yeah I do that sometimes, sucks. | 18:51 |
jcastro | (I am eating lunch now and it's 2 oclock, heh) | 18:51 |
gary_poster | :-) | 18:51 |
gary_poster | yeah I may need to do that in next mtg. :-) ...stepping away to kitchen briefly... | 18:52 |
frankban | gary_poster: review done, thank you and have a nice weekend all | 19:05 |
gary_poster | thanks frankban! | 19:06 |
gary_poster | have a great weekend | 19:06 |
hatch | crazy that anandtech still doesn't have a new rMBP 15" review | 19:24 |
hatch | 188 unread blogposts....who wants to help me read? | 19:27 |
hatch | I'm not sure if anyone else is a Digital Blasphemy member but this one is just awesome http://digitalblasphemy.com/preview.shtml?i=traveller2k131 | 19:29 |
hatch | ok down to 157, and done fooding | 19:44 |
jcastro | oh dang, forgot to ask frankban | 19:47 |
jcastro | gary_poster, is the new release of quickstart in your PPA something we should copy over to the juju PPA? | 19:47 |
gary_poster | jcastro, already done. | 19:48 |
jcastro | oh well, that covers that! | 19:48 |
gary_poster | :-) | 19:48 |
bac | has anyone installed the juju-quickstart from the new beta ppa? the package says 0.4.2 but --version say 0.4.0 | 20:02 |
bac | s/say/says/ | 20:02 |
bac | s/say/says/ | 20:02 |
bac | s/n say/n says/ | 20:03 |
bac | s/saysss/says/ | 20:03 |
bac | gary_poster: ^^ | 20:09 |
gary_poster | bac, I have not. and lol at the IRC substitutions. doesn't sound too bad. Maybe shoot a note to peeps and frankban about it? | 20:10 |
gary_poster | I bet it is a release checklist thing | 20:11 |
bac | gary_poster: well, i know he updated the version file in his latest branch. so i fear the PPA just didn't get everything he expected. | 20:11 |
gary_poster | bac, oh. :-/ | 20:12 |
gary_poster | bac, you feel like kicking the machinery, or shall I investigate? | 20:12 |
bac | in fact it probably doesn't have my deployment counter b/c i know i set it at 0.4.1 | 20:12 |
gary_poster | :-( | 20:12 |
gary_poster | 0.4.2-1 | 20:12 |
gary_poster | ? | 20:12 |
bac | gary_poster: i'll go poke at it. | 20:12 |
gary_poster | thank you | 20:12 |
bac | gary_poster: false alarm. i had an old qs install in my /usr/local/lib | 20:18 |
bac | PPA is a-ok | 20:18 |
gary_poster | bac, oh, great | 20:19 |
gary_poster | thank you for investigating, bac | 20:19 |
hatch | man I love it when tests fail but the real world works just fine | 20:20 |
hatch | it just brightens up my day! | 20:20 |
gary_poster | ;-) | 20:21 |
rick_h_ | trade you, you can hack this up in shell and ruby together and intertwined :) | 20:21 |
gary_poster | heh, I wondered about that when I skimmed the code. heh and :-/ | 20:22 |
hatch | RUby? You're implementing our CI in RUBY? | 20:23 |
* hatch storms away | 20:23 | |
hatch | :P | 20:23 |
rick_h_ | yea, and there's a long running fork that's submitted 6 pull requests with no merges back in. So now I've forked the fork and updating it to work wheee | 20:23 |
rick_h_ | hatch: :) | 20:23 |
rick_h_ | why not, it's one of the few lang's I don't have in my github profile | 20:23 |
rick_h_ | COMPLETED | 20:23 |
hatch | lol | 20:23 |
hatch | Github thinks I have Ruby in my Ghost charm, does that count? | 20:24 |
rick_h_ | hah | 20:24 |
rick_h_ | and curses for comments not being part of JSON so this whole config file has not one comment on wtf a 'repo_to_pull_id_param' is | 20:25 |
hatch | there are really no other options? | 20:25 |
rick_h_ | write one | 20:25 |
hatch | I actually have no idea what your doing....so SURE, it shouldn't take that long | 20:26 |
rick_h_ | lol | 20:26 |
gary_poster | yeah. that was my worst case for this :-/ | 20:26 |
rick_h_ | github has an api right? and there's only 7 python libraries that claim to work with it. | 20:27 |
gary_poster | heh | 20:27 |
hatch | well there are always node ones too :) | 20:27 |
rick_h_ | yea, because I've got 99 problems but npm not working aint one | 20:27 |
hatch | lol | 20:27 |
* gary_poster laughs/cries | 20:28 | |
rick_h_ | I'd keep quite on the node for a while. I've been burned three times lately and I'm a bit cranky | 20:28 |
hatch | hey that's the package managers fault | 20:28 |
hatch | they should use juju | 20:28 |
rick_h_ | and what package did you want me to use? :P | 20:28 |
hatch | actually if it makes you feel any better, if npm is down we can't install the GUI anyways | 20:29 |
hatch | so really, it's like an early failure system | 20:29 |
rick_h_ | "node and npm, node and npm, go together like a fish and water ...you can't have one without the other" | 20:29 |
gary_poster | can too | 20:29 |
gary_poster | from charm/packaged bits | 20:29 |
rick_h_ | yea, as long as it's the current release | 20:29 |
rick_h_ | trigger config-changed and you're done for | 20:29 |
hatch | shh I'm trying to make a point here | 20:29 |
hatch | there is no room for logic here | 20:30 |
gary_poster | lol | 20:30 |
rick_h_ | sorry, forgot it's not the canadian way | 20:30 |
rick_h_ | :P | 20:30 |
rick_h_ | I think it gets too cold for logic up there | 20:30 |
hatch | and it's not even "cold" yet | 20:30 |
hatch | only -20 | 20:30 |
gary_poster | uh-oh, the Michigan attacks Canada | 20:30 |
rick_h_ | oh come on, you know that canada is the 51st state, we've just not bothered to claim it yet. | 20:31 |
gary_poster | ooh, ruby has made rick aaaaangry! | 20:32 |
hatch | we aren't worried, you couldn't the handle universal health care and a stable banking system - many would go crazy | 20:33 |
gary_poster | lol. there we go, fight! fight! fight! | 20:33 |
hatch | hahaha | 20:33 |
rick_h_ | don't make me pull out my trump card | 20:33 |
rick_h_ | now's not a good time...I'll go all Mayor on you | 20:33 |
gary_poster | yeah, low blow, I dunno | 20:34 |
rick_h_ | things must just run themselves because that mayor isn't running much... | 20:34 |
gary_poster | :-) | 20:34 |
hatch | You're a little to close to Detroit to be bringing things down to the city management level | 20:34 |
hatch | :D | 20:34 |
gary_poster | lol | 20:34 |
rick_h_ | hey, we've got the auto industry to blame. That mayor is all on the voters | 20:35 |
hatch | have you ever been to Toronto? Everyone there likes to party | 20:35 |
hatch | lol | 20:35 |
rick_h_ | there we go, canada could be our second most populated state | 20:35 |
rick_h_ | http://www.wolframalpha.com/input/?i=population+of+canada+vs+population+of+california+vs+population+of+texas | 20:36 |
hatch | and the only one not in the red? | 20:36 |
hatch | lol | 20:36 |
gary_poster | Texas is red a different way | 20:36 |
hatch | haha oh right | 20:37 |
rick_h_ | well, I got the script to run once manually from the cli on the jenkins server. Going to call success and run away. | 20:38 |
gary_poster | heh | 20:38 |
gary_poster | have a great weekend rick_h_ | 20:38 |
rick_h_ | you guys as well! | 20:39 |
hatch | haha | 20:39 |
hatch | have a good one | 20:39 |
rick_h_ | even candadians | 20:39 |
hatch | cya | 20:39 |
hatch | I think I managed to break every inspector overview test... | 20:40 |
hatch | that must be some kind of record | 20:40 |
gary_poster | heh | 20:41 |
hatch | rick_h_: oh hey if you pop back, get your buddy there to file any feature requests on the ghost gh repo if he has any | 20:45 |
bac | gentlemens i bid you adieu and anon. | 21:12 |
gary_poster | bye bac. have a great weekend | 21:13 |
bac | will do | 21:13 |
hatch | cya bac | 21:17 |
gary_poster | have a great weekend all | 22:27 |
=== gary_poster is now known as gary_poster|away | ||
hatch | you too gary_poster|away cya | 22:28 |
* hatch last person in the office *turns the tunes way up* | 22:44 | |
hatch | jujugui if anyone feels like they want to do a deep and detailed qa this weekend https://codereview.appspot.com/31190043/ :) | 23:19 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!