[01:08] hatch: have the chrome dev summit videos not been put on youtube? I wanted to catch up but :( [01:12] rick_h_: not sure I haven't seen anything yet [01:13] hatch: yea, found the 'live stream' day one which is an 8hr video [12:35] 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] s/issue/issues/ [12:37] 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] frankban: sure [12:38] bac: thanks. quick one: https://codereview.appspot.com/30810043. more complex one: https://codereview.appspot.com/30760043 [12:40] 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:45] marcoceppi: hmm, so it'd be big. It kind of work together with the db setup, the JS models. [12:45] marcoceppi: we could probably get a hangout to walk through it sometime. [12:45] rick_h_: cool [12:50] frankban: make check shows incomplete test coverage [12:50] frankban: on your first review [12:50] bac: that's fixed by the second branch in review [12:50] frankban: ok [13:07] 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] bac: it uses webbrowser.open, so it should be the default browser, for some definition of "default" [13:08] huh [13:08] bac: http://docs.python.org/2/library/webbrowser.html#webbrowser.open === gary_poster|away is now known as gary_poster [13:15] hi everybody [13:16] morning gary_poster [13:16] :-) [13:16] guihelp: anyone available for another review of https://codereview.appspot.com/30760043 (quickstart)? [13:17] I will after mail skim if no one else takes it [13:17] frankban: i was just about to do it. or are you asking for a second? [13:17] bac: I was asking for a second. thank you [13:17] * bac continues [13:17] thanks gary_poster [13:17] welcome [13:25] I can look frankban [13:27] rick_h_: cool thanks [13:27] gary_poster: ^^^ [13:27] ok cool thanks rick_h_ and frankban [13:28] the coffee it brewed and it's friday! [13:28] :-) yay [13:33] ok email is not exactly tamed, but the wild animals are at least identified [13:33] shoot to kill! [13:34] heh [13:42] * frankban biab [13:43] rick_h_: you are doing frankban's review? i'm starting QA so you can skip that part [13:44] bac: yes, thanks [14:14] why are there paths that start with "./" all over the charm Makefile? [14:16] save the dots [14:16] they look cool! [14:18] hatch: ha! [14:20] :D [14:21] pfft [14:22] the worst part of tests is fixing them all [14:27] 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] rick_h_, Hey Rick, you about? [14:27] 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] 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] antdillon: yep [14:28] bac: ok, will try to go do that in a minute then. [14:28] rick_h_, Do you know if there is more api calls to manage.jujucharms.com/api/3/? [14:28] antdillon: http://charmworld.readthedocs.org/en/latest/api.html [14:28] rick_h_, Awesome thanks [14:29] antdillon: regenerating the docs as they've not been updated for a couple of weeks, so maybe refresh in 5 [14:29] antdillon: let me know if you've got any specific questions/api stuff you're looking for [14:29] rick_h_, Cool thanks [14:30] rick_h_: sounds good. re changing the name of the check_charm_url utility. sounds good, maybe print_charm_url? [14:30] rick_h_, Just wanna pull a specific charms details in a call [14:31] frankban: yea, I mean maybe that's just me. I didn't have a great alternative. I just was confused for a minute. [14:32] 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] frankban: review and most QA done [14:32] antdillon: ok, so that's just https://manage.jujucharms.com/api/3/charm/precise/mysql [14:32] rick_h_, Yep got it, thank man, the docs are col [14:32] cool* [14:32] antdillon: awesome, api ftw [14:33] bac: thanks [14:35] bac: so I just need to do #3 to complete qa? [14:35] rick_h_: that is correct [14:40] frankban: ok, #3 is good, qa is all ok [14:40] rick_h_: great, thank you! [14:42] uuuuuuugh someone else wana fix these tests for me? Free coffee available in your kitchen if you do! [14:42] hatch: oh come on, it's an oppertunity. I'll let you take it [14:42] you broke it you bought it :P [14:43] 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:44] 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] juju-deployer... [14:44] Maybe that's optimistic [14:44] haha darn, well I tried [14:44] 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:45] 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] and if those rules change it's going to suck for us [14:46] rick_h_: yeah, maybe two weeks/one developer. yes, I don;t expect charms to change so often, but bundles definitely can [14:47] 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] 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:49] rick_h_: describe_gui_charm_url... how does it sound? [14:51] 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:52] 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] rick_h_: great, I'll leave check [15:10] bac: if you wouldn't mind, could you fire open Safari and let me know what version you have? [15:11] hatch: that's a very personal question. the answer of which is Version 7.0 (9537.71) [15:11] (i think that is a Star Date) [15:11] thanks for opening up to me, Starfleet thanks you [15:12] anything else? shoe size? bagel topping preference? [15:14] :-) [15:15] lol nope that's it, I just needed to know what the most recent safari was on mavericks [15:15] I don't have mavericks installed yet [15:17] 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 [15:19] 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:20] frankban: ah, ok. [15:23] bac: FWIW, bug 1251420 could be interesting [15:23] <_mup_> Bug #1251420: reporting an error from the environment isn't formatted well [15:50] jujugui call in 10 [15:58] jujugui call in 2 [16:00] hatch, yo yo yo [16:35] gary_poster: got a few mins for a g+? [16:36] marcoceppi, on call will ping [16:36] gary_poster: ack, thanks! [17:08] benji: still lbox problems? [17:09] frankban: yep: "error: Failed to run "bzr push": exit status 3" [17:09] frankban: this is the right branch for the charm trunk, correct? bzr+ssh://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/ [17:10] benji: yes [17:11] benji: is this lbox propose? anything useful from -debug? [17:11] yep, "lbox propose"; I'll do a -debug run now [17:12] 2013/11/22 11:11:44 Branch URL inferred: lp:~benji/charms/use-known-pip [17:12] that doesn't look right [17:12] oh: bzr: ERROR: Permission denied: "~benji/charms/use-known-pip/": : Project 'charms' does not exist. [17:14] benji: yes so lbox pushes the branch for you, and that's not the right location. [17:15] frankban: any hints as to a good location? [17:16] benji: I use this entries in my ~/.bazaar/locations.conf: http://pastebin.ubuntu.com/6459507/ you can adjust them for your needs [17:17] frankban: ah! thanks; I bet I lost a similar entry in my bzr decimation [17:19] 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] yep, it is nice [17:20] grrr, lbox, it is OK if the first line of a commit message is over 50 characters! [17:21] it's like someone created an evil AI that loves haiku [17:21] :-) [17:33] frankban: do you have time to look at the fruit of my lbox, er labor? https://codereview.appspot.com/31000043/ [17:33] benji: sure [17:40] marcoceppi, hey. ready. you have a hangout or do you want me to make one? [17:40] in a meeting gary_poster, will ping when I'm avail again :) [17:40] 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] marcoceppi, :-) k [17:42] so...many....tabs [17:45] gary_poster: okay! https://plus.google.com/hangouts/_/76cpj7q9igbirg4g5ov1ifeups?hl=en [17:45] 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] that'll put you in the queue [17:47] jcastro: ok cool, so should I blog about it once it's up or wait for charmers? [17:47] I prefer to wait for charmers because the namespace is cleaner rather than ~jorge/blah [17:47] but up to you [17:48] it it works might as well blog it, post it on their forum [17:48] 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] haha [17:49] hatch: there was a guy at CHC working with ghost and I mentioned your charm in the wings [17:49] hatch: kind of cool, he's a ghost fan [17:50] 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] rick_h_: oh cool - I would like some other users input into what additional features I should add [18:09] jujugui, charm review request #1 of in-progress effort: https://codereview.appspot.com/31020043 [18:12] gary_poster: taking a look [18:12] thanks frankban [18:17] 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] I mean, in rietveld [18:19] hey rick_h_ you know a particular bundle that induces the error we discussed? [18:19] bac: hmm, I think it was fixed. It was due to a bundle have constraints defined as cpu-cores='' [18:19] so the '' blew it up [18:20] since it needed to be an int [18:20] rick_h_: ok, cool, i'll sabotage one of mine [18:30] 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] frankban: woot! I was just looking to use the lxc support this morning [18:30] awesome! thanks frankban [18:31] frankban, nice [18:32] in case anyone else needs it: sudo add-apt-repository ppa:juju-gui/quickstart-beta [18:32] I'll copy it over to juju/stable [18:32] thanks bac [18:39] hey frankban [18:39] are you going to announce on the list? [18:39] I am working on the quickstart /bundle docs this afternoon [18:39] and was going to announce them too [18:40] jcastro: EOD for me, could you please give it a try and include it in your announcement? [18:40] frankban: thanks for the review [18:40] benji: welcome [18:40] ok! [18:40] thanks jcastro [18:51] * 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] gary_poster, yeah I do that sometimes, sucks. [18:51] (I am eating lunch now and it's 2 oclock, heh) [18:51] :-) [18:52] yeah I may need to do that in next mtg. :-) ...stepping away to kitchen briefly... [19:05] gary_poster: review done, thank you and have a nice weekend all [19:06] thanks frankban! [19:06] have a great weekend [19:24] crazy that anandtech still doesn't have a new rMBP 15" review [19:27] 188 unread blogposts....who wants to help me read? [19:29] 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:44] ok down to 157, and done fooding [19:47] oh dang, forgot to ask frankban [19:47] gary_poster, is the new release of quickstart in your PPA something we should copy over to the juju PPA? [19:48] jcastro, already done. [19:48] oh well, that covers that! [19:48] :-) [20:02] 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] s/say/says/ [20:02] s/say/says/ [20:03] s/n say/n says/ [20:03] s/saysss/says/ [20:09] gary_poster: ^^ [20:10] 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:11] I bet it is a release checklist thing [20:11] 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:12] bac, oh. :-/ [20:12] bac, you feel like kicking the machinery, or shall I investigate? [20:12] in fact it probably doesn't have my deployment counter b/c i know i set it at 0.4.1 [20:12] :-( [20:12] 0.4.2-1 [20:12] ? [20:12] gary_poster: i'll go poke at it. [20:12] thank you [20:18] gary_poster: false alarm. i had an old qs install in my /usr/local/lib [20:18] PPA is a-ok [20:19] bac, oh, great [20:19] thank you for investigating, bac [20:20] man I love it when tests fail but the real world works just fine [20:20] it just brightens up my day! [20:21] ;-) [20:21] trade you, you can hack this up in shell and ruby together and intertwined :) [20:22] heh, I wondered about that when I skimmed the code. heh and :-/ [20:23] RUby? You're implementing our CI in RUBY? [20:23] * hatch storms away [20:23] :P [20:23] 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] hatch: :) [20:23] why not, it's one of the few lang's I don't have in my github profile [20:23] COMPLETED [20:23] lol [20:24] Github thinks I have Ruby in my Ghost charm, does that count? [20:24] hah [20:25] 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] there are really no other options? [20:25] write one [20:26] I actually have no idea what your doing....so SURE, it shouldn't take that long [20:26] lol [20:26] yeah. that was my worst case for this :-/ [20:27] github has an api right? and there's only 7 python libraries that claim to work with it. [20:27] heh [20:27] well there are always node ones too :) [20:27] yea, because I've got 99 problems but npm not working aint one [20:27] lol [20:28] * gary_poster laughs/cries [20:28] 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] hey that's the package managers fault [20:28] they should use juju [20:28] and what package did you want me to use? :P [20:29] actually if it makes you feel any better, if npm is down we can't install the GUI anyways [20:29] so really, it's like an early failure system [20:29] "node and npm, node and npm, go together like a fish and water ...you can't have one without the other" [20:29] can too [20:29] from charm/packaged bits [20:29] yea, as long as it's the current release [20:29] trigger config-changed and you're done for [20:29] shh I'm trying to make a point here [20:30] there is no room for logic here [20:30] lol [20:30] sorry, forgot it's not the canadian way [20:30] :P [20:30] I think it gets too cold for logic up there [20:30] and it's not even "cold" yet [20:30] only -20 [20:30] uh-oh, the Michigan attacks Canada [20:31] oh come on, you know that canada is the 51st state, we've just not bothered to claim it yet. [20:32] ooh, ruby has made rick aaaaangry! [20:33] we aren't worried, you couldn't the handle universal health care and a stable banking system - many would go crazy [20:33] lol. there we go, fight! fight! fight! [20:33] hahaha [20:33] don't make me pull out my trump card [20:33] now's not a good time...I'll go all Mayor on you [20:34] yeah, low blow, I dunno [20:34] things must just run themselves because that mayor isn't running much... [20:34] :-) [20:34] You're a little to close to Detroit to be bringing things down to the city management level [20:34] :D [20:34] lol [20:35] hey, we've got the auto industry to blame. That mayor is all on the voters [20:35] have you ever been to Toronto? Everyone there likes to party [20:35] lol [20:35] there we go, canada could be our second most populated state [20:36] http://www.wolframalpha.com/input/?i=population+of+canada+vs+population+of+california+vs+population+of+texas [20:36] and the only one not in the red? [20:36] lol [20:36] Texas is red a different way [20:37] haha oh right [20:38] 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] heh [20:38] have a great weekend rick_h_ [20:39] you guys as well! [20:39] haha [20:39] have a good one [20:39] even candadians [20:39] cya [20:40] I think I managed to break every inspector overview test... [20:40] that must be some kind of record [20:41] heh [20:45] 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 [21:12] gentlemens i bid you adieu and anon. [21:13] bye bac. have a great weekend [21:13] will do [21:17] cya bac [22:27] have a great weekend all === gary_poster is now known as gary_poster|away [22:28] you too gary_poster|away cya [22:44] * hatch last person in the office *turns the tunes way up* [23:19] jujugui if anyone feels like they want to do a deep and detailed qa this weekend https://codereview.appspot.com/31190043/ :)