/srv/irclogs.ubuntu.com/2013/11/22/#juju-gui.txt

rick_h_hatch: have the chrome dev summit videos not been put on youtube? I wanted to catch up but :(01:08
hatchrick_h_: not sure I haven't seen anything yet01:12
rick_h_hatch: yea, found the 'live stream' day one which is an 8hr video01:13
bachi 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
bacs/issue/issues/12:35
frankbanbac: 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
bacfrankban: sure12:37
frankbanbac: thanks. quick one: https://codereview.appspot.com/30810043. more complex one: https://codereview.appspot.com/3076004312:38
marcoceppiGood 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
marcoceppirick_h_: cool12:45
bacfrankban: make check shows incomplete test coverage12:50
bacfrankban: on your first review12:50
frankbanbac: that's fixed by the second branch in review 12:50
bacfrankban: ok12:50
bacfrankban: 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
frankbanbac: it uses webbrowser.open, so it should be the default browser, for some definition of "default"13:07
bachuh13:08
frankbanbac: http://docs.python.org/2/library/webbrowser.html#webbrowser.open13:08
=== gary_poster|away is now known as gary_poster
gary_posterhi everybody13:15
frankbanmorning gary_poster 13:16
gary_poster:-)13:16
frankbanguihelp: anyone available for another review of https://codereview.appspot.com/30760043 (quickstart)?13:16
gary_posterI will after mail skim if no one else takes it13:17
bacfrankban: i was just about to do it.  or are you asking for a second?13:17
frankbanbac: I was asking for a second. thank you13:17
* bac continues13:17
frankbanthanks gary_poster 13:17
gary_posterwelcome13:17
rick_h_I can look frankban 13:25
frankbanrick_h_: cool thanks13:27
frankbangary_poster: ^^^13:27
gary_posterok cool thanks rick_h_ and frankban 13:27
rick_h_the coffee it brewed and it's friday!13:28
gary_poster:-) yay13:28
gary_posterok email is not exactly tamed, but the wild animals are at least identified13:33
rick_h_shoot to kill!13:33
gary_posterheh13:34
* frankban biab13:42
bacrick_h_: you are doing frankban's review?  i'm starting QA so you can skip that part13:43
rick_h_bac: yes, thanks13:44
benjiwhy are there paths that start with "./" all over the charm Makefile?14:14
hatchsave the dots14:16
rick_h_they look cool!14:16
bachatch: ha!14:18
hatch:D14:20
benjipfft14:21
hatchthe worst part of tests is fixing them all14:22
frankbanrick_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 network14:27
antdillonrick_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
bacrick_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: yep14:27
rick_h_bac: ok, will try to go do that in a minute then. 14:28
antdillonrick_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.html14:28
antdillonrick_h_, Awesome thanks14:28
rick_h_antdillon: regenerating the docs as they've not been updated for a couple of weeks, so maybe refresh in 514:29
rick_h_antdillon: let me know if you've got any specific questions/api stuff you're looking for14:29
antdillonrick_h_, Cool thanks14:29
frankbanrick_h_: sounds good. re changing the name of the check_charm_url utility. sounds good, maybe print_charm_url?14:30
antdillonrick_h_, Just wanna pull a specific charms details in a call14: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
frankbanrick_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
bacfrankban: review and most QA done14:32
rick_h_antdillon: ok, so that's just https://manage.jujucharms.com/api/3/charm/precise/mysql14:32
antdillonrick_h_, Yep got it, thank man, the docs are col14:32
antdilloncool*14:32
rick_h_antdillon: awesome, api ftw14:32
frankbanbac: thanks14:33
rick_h_bac: so I just need to do #3 to complete qa?14:35
bacrick_h_: that is correct14:35
rick_h_frankban: ok, #3 is good, qa is all ok14:40
frankbanrick_h_: great, thank you!14:40
hatchuuuuuuugh 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 it14:42
rick_h_you broke it you bought it :P14:42
frankbanrick_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
frankbanjuju-deployer...14:44
rick_h_Maybe that's optimistic14:44
hatchhaha darn, well I tried14: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 control14: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 us14:45
frankbanrick_h_: yeah, maybe two weeks/one developer. yes, I don;t expect charms to change so often, but bundles definitely can14: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
frankbanrick_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 place14:51
frankbanrick_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 that14:52
frankbanrick_h_: great, I'll leave check14:52
hatchbac: if you wouldn't mind, could you fire open Safari and let me know what version you have?15:10
bachatch: 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
hatchthanks for opening up to me, Starfleet thanks you15:11
bacanything else?  shoe size? bagel topping preference?15:12
frankban:-)15:14
hatchlol nope that's it, I just needed to know what the most recent safari was on mavericks15:15
hatchI don't have mavericks installed yet15:15
bacfrankban: 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
frankbanbac: 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
bacfrankban: ah, ok.15:20
frankbanbac: FWIW, bug 1251420 could be interesting15: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_posterjujugui call in 1015:50
gary_posterjujugui call in 215:58
gary_posterhatch, yo yo yo16:00
marcoceppigary_poster: got a few mins for a g+?16:35
gary_postermarcoceppi, on call will ping16:36
marcoceppigary_poster: ack, thanks!16:36
frankbanbenji: still lbox problems?17:08
benjifrankban: yep: "error: Failed to run "bzr push": exit status 3"17:09
benjifrankban: this is the right branch for the charm trunk, correct? bzr+ssh://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/17:09
frankbanbenji: yes17:10
frankbanbenji: is this lbox propose? anything useful from -debug?17:11
benjiyep, "lbox propose"; I'll do a -debug run now17:11
benji2013/11/22 11:11:44 Branch URL inferred: lp:~benji/charms/use-known-pip17:12
benjithat doesn't look right17:12
benjioh: bzr: ERROR: Permission denied: "~benji/charms/use-known-pip/": : Project 'charms' does not exist.17:12
frankbanbenji: yes so lbox pushes the branch for you, and that's not the right location. 17:14
benjifrankban: any hints as to a good location?17:15
frankbanbenji: I use this entries in my ~/.bazaar/locations.conf: http://pastebin.ubuntu.com/6459507/  you can adjust them for your needs17:16
benjifrankban: ah! thanks; I bet I lost a similar entry in my bzr decimation 17:17
frankbanbenji: 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
benjiyep, it is nice17:19
benjigrrr, lbox, it is OK if the first line of a commit message is over 50 characters!17:20
benjiit's like someone created an evil AI that loves haiku17:21
frankban:-)17:21
benjifrankban: do you have time to look at the fruit of my lbox, er labor?  https://codereview.appspot.com/31000043/17:33
frankbanbenji: sure17:33
gary_postermarcoceppi, hey.  ready.  you have a hangout or do you want me to make one?17:40
marcoceppiin a meeting gary_poster, will ping when I'm avail again :)17:40
hatchjcastro: 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_postermarcoceppi, :-) k17:40
hatchso...many....tabs17:42
marcoceppigary_poster: okay! https://plus.google.com/hangouts/_/76cpj7q9igbirg4g5ov1ifeups?hl=en17:45
jcastrohatch, 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 bug17:45
jcastrothat'll put you in the queue17:45
hatchjcastro: ok cool, so should I blog about it once it's up or wait for charmers?17:47
jcastroI prefer to wait for charmers because the namespace is cleaner rather than ~jorge/blah17:47
jcastrobut up to you17:47
jcastroit it works might as well blog it, post it on their forum17:48
jcastroand 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 something17:48
hatchhaha17:48
rick_h_hatch: there was a guy at CHC working with ghost and I mentioned your charm in the wings17:49
rick_h_hatch: kind of cool, he's a ghost fan17:49
hatchjcastro: ok cool I'll make sure it proofs perfect this weekend and create a bug for it so that the review can be quick17:50
hatchrick_h_: oh cool - I would like some other users input into what additional features I should add17:50
gary_posterjujugui, charm review request #1 of in-progress effort: https://codereview.appspot.com/3102004318:09
frankbangary_poster: taking a look18:12
gary_posterthanks frankban 18:12
gary_posterthis 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_posterI mean, in rietveld18:17
bachey 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 up18:19
rick_h_since it needed to be an int 18:20
bacrick_h_: ok, cool, i'll sabotage one of mine18:20
frankbanjujugui, 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 morning18:30
gary_posterawesome!  thanks frankban 18:30
hazmatfrankban, nice18:31
bacin case anyone else needs it: sudo add-apt-repository ppa:juju-gui/quickstart-beta18:32
gary_posterI'll copy it over to juju/stable18:32
frankbanthanks bac 18:32
jcastrohey frankban18:39
jcastroare you going to announce on the list? 18:39
jcastroI am working on the quickstart /bundle docs this afternoon18:39
jcastroand was going to announce them too18:39
frankbanjcastro: EOD for me, could you please give it a try and include it in your announcement? 18:40
benjifrankban: thanks for the review18:40
frankbanbenji: welcome18:40
jcastrook!18:40
frankbanthanks 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
jcastrogary_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_posteryeah I may need to do that in next mtg. :-) ...stepping away to kitchen briefly...18:52
frankbangary_poster: review done, thank you and have a nice weekend all19:05
gary_posterthanks frankban!19:06
gary_posterhave a great weekend19:06
hatchcrazy that anandtech still doesn't have a new rMBP 15" review19:24
hatch188 unread blogposts....who wants to help me read?19:27
hatchI'm not sure if anyone else is a Digital Blasphemy member but this one is just awesome http://digitalblasphemy.com/preview.shtml?i=traveller2k13119:29
hatchok down to 157, and done fooding19:44
jcastrooh dang, forgot to ask frankban19:47
jcastrogary_poster, is the new release of quickstart in your PPA something we should copy over to the juju PPA?19:47
gary_posterjcastro, already done.  19:48
jcastrooh well, that covers that!19:48
gary_poster:-)19:48
bachas anyone installed the juju-quickstart from the new beta ppa?  the package says 0.4.2 but --version say 0.4.020:02
bacs/say/says/20:02
bacs/say/says/20:02
bacs/n say/n says/20:03
bacs/saysss/says/20:03
bacgary_poster: ^^20:09
gary_posterbac, 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_posterI bet it is a release checklist thing20:11
bacgary_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_posterbac, oh. :-/20:12
gary_posterbac, you feel like kicking the machinery, or shall I investigate?20:12
bacin fact it probably doesn't have my deployment counter b/c i know i set it at 0.4.120:12
gary_poster:-(20:12
gary_poster0.4.2-120:12
gary_poster?20:12
bacgary_poster: i'll go poke at it.20:12
gary_posterthank you20:12
bacgary_poster: false alarm.  i had an old qs install in my /usr/local/lib20:18
bacPPA is a-ok20:18
gary_posterbac, oh, great20:19
gary_posterthank you for investigating, bac20:19
hatchman I love it when tests fail but the real world works just fine20:20
hatchit 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_posterheh, I wondered about that when I skimmed the code. heh and :-/20:22
hatchRUby? You're implementing our CI in RUBY?20:23
* hatch storms away20:23
hatch:P20: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 wheee20: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 profile20:23
rick_h_COMPLETED20:23
hatchlol20:23
hatchGithub thinks I have Ruby in my Ghost charm, does that count?20:24
rick_h_hah20: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' is20:25
hatchthere are really no other options?20:25
rick_h_write one20:25
hatchI actually have no idea what your doing....so SURE, it shouldn't take that long20:26
rick_h_lol20:26
gary_posteryeah.  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_posterheh20:27
hatchwell there are always node ones too :)20:27
rick_h_yea, because I've got 99 problems but npm not working aint one20:27
hatchlol20:27
* gary_poster laughs/cries20: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 cranky20:28
hatchhey that's the package managers fault20:28
hatchthey should use juju20:28
rick_h_and what package did you want me to use? :P20:28
hatchactually if it makes you feel any better, if npm is down we can't install the GUI anyways20:29
hatchso really, it's like an early failure system20: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_postercan too20:29
gary_posterfrom charm/packaged bits20:29
rick_h_yea, as long as it's the current release20:29
rick_h_trigger config-changed and you're done for20:29
hatchshh I'm trying to make a point here20:29
hatchthere is no room for logic here20:30
gary_posterlol20:30
rick_h_sorry, forgot it's not the canadian way20:30
rick_h_:P20:30
rick_h_I think it gets too cold for logic up there20:30
hatchand it's not even "cold" yet20:30
hatchonly -2020:30
gary_posteruh-oh, the Michigan attacks Canada20: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_posterooh, ruby has made rick aaaaangry!20:32
hatchwe aren't worried, you couldn't the handle universal health care and a stable banking system - many would go crazy20:33
gary_posterlol.  there we go, fight! fight! fight!20:33
hatchhahaha20:33
rick_h_don't make me pull out my trump card20:33
rick_h_now's not a good time...I'll go all Mayor on you20:33
gary_posteryeah, low blow, I dunno20:34
rick_h_things must just run themselves because that mayor isn't running much...20:34
gary_poster:-)20:34
hatchYou're a little to close to Detroit to be bringing things down to the city management level20:34
hatch:D20:34
gary_posterlol20:34
rick_h_hey, we've got the auto industry to blame. That mayor is all on the voters 20:35
hatchhave you ever been to Toronto? Everyone there likes to party20:35
hatchlol20:35
rick_h_there we go, canada could be our second most populated state20:35
rick_h_http://www.wolframalpha.com/input/?i=population+of+canada+vs+population+of+california+vs+population+of+texas20:36
hatchand the only one not in the red?20:36
hatchlol20:36
gary_posterTexas is red a different way20:36
hatchhaha oh right20: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_posterheh20:38
gary_posterhave a great weekend rick_h_ 20:38
rick_h_you guys as well!20:39
hatchhaha20:39
hatchhave a good one20:39
rick_h_even candadians20:39
hatchcya20:39
hatchI think I managed to break every inspector overview test...20:40
hatchthat must be some kind of record20:40
gary_posterheh20:41
hatchrick_h_: oh hey if you pop back, get your buddy there to file any feature requests on the ghost gh repo if he has any20:45
bacgentlemens i bid you adieu and anon.21:12
gary_posterbye bac.  have a great weekend21:13
bacwill do21:13
hatchcya bac21:17
gary_posterhave a great weekend all22:27
=== gary_poster is now known as gary_poster|away
hatchyou too gary_poster|away cya22:28
* hatch last person in the office *turns the tunes way up*22:44
hatchjujugui 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!