[00:36] gary_poster, bcsaller I proposed but never merged. Want to just include my change, bcsaller ? [00:36] Fell asleep :( [00:37] I'll keep QAing tonight, though [00:37] Makyo: I haven't seen it yet, but am happy to look. I've been working through test failures here [00:38] bcsaller, Okay, https://codereview.appspot.com/14430061/ lp:~makyo/juju-gui/coords - gary_poster QA' [00:38] qa'd for initial rendering of services with annotations [00:38] (though if your branch gets that incidentally, nvm) [00:39] that was a good find, I have tried to remove the whole hasBeenPositioned case and leave the annotations in place to know that (but haven't been able to convince the tests of this yet) [02:15] huwshimi: ping [02:16] rick_h_: Hey [02:16] huwshimi: hey, can you dump out what services is in your code sample? [02:16] rick_h_: Sure, one sec. [02:16] huwshimi: the token doesn't take a model, it has to be model.getAttrs() [02:17] huwshimi: but not sure that a bundle.get('services') will be real charm models either. Looking at that now [02:18] rick_h_: services is Object {haproxy: Object, mediawiki: Object, memcached: Object, mysql: Object} [02:18] huwshimi: right, so services isn't really good info. [02:18] https://manage.jujucharms.com/api/3/bundle/~bac/wiki/wiki/ [02:18] huwshimi: check out that bundle and note that the services block is kind of wishy-washy. It might be a branch, it might be a charm store url, it might be a charm name/series [02:19] huwshimi: so when you request a bundle from charmworld, it'll provide the charm model details in the charm_metadata key [02:20] huwshimi: you have to look in there. It doesn't appear it's in the bundle model atm, but it's used in the view and such it looks like [02:20] huwshimi: bzr grep "charm_metadata" [02:22] rick_h_: OK thanks, I'll take a look [02:24] huwshimi: ok, put notes into the email reply. Let me know if you need more help. [02:25] rick_h_: Thanks rick, much appreciated. [12:09] * gary_poster starts review & qa of Ben's branch. Departure in T-26 minutes [12:09] s/in/at/ [12:11] rick_h_: i'm about half-way done with your review. must relocate right now but will get right on it. [12:13] bac: rgr, thanks! [12:26] quick hack idea: http://jujugui.wordpress.com/2013/10/15/if-you-want-to-run-a-custom-gui/ [12:29] nice! [13:37] Hey guys, does anyone have some spare time to review my onboarding branch and help land it under a flag? [13:42] antdillon: sure thing, paste the merge proposal in here. Can you submit with lbox? [13:49] rick_h_: review done. [13:49] bac: ty much sir [13:57] antdillon: make sure to run 'make lint' in your branch [13:58] antdillon: checking out the test run now to make sure I can lbox it up for you [14:00] rick_h_: now that we have 'bzr rvsubmit' we may want to start requiring the use of lbox for charmworld. if we do, we can add a .lbox that does 'make lint'. that'll prevent late stage, landing failures due to lint. [14:00] bac: true, there's a .lbox now just don't think it runs lint [14:00] i'm +2 on it. (me and presuming benji's +1 given previous conversations) [14:00] bac: at least I hope that's why it took forevery for my lbox to submit [14:01] oh, whee, it has been there all along [14:01] antdillon: we're going to have to talk about lint :P [14:02] rick_h_, Sure [14:02] rick_h_: .lbox.check could probably be simplified to 'make check' [14:03] bac: ah, good call [14:03] i*hate* that we do 'apt-get update' twice! [14:03] bac: did we find out if there's any reason? [14:03] something for the charm or something? [14:05] running through the lint fixes since they're pretty easy, just a bunch of them. Things like proper spacing, lines too long, etc. [14:05] antdillon: ^ [14:05] antdillon: will push up to lbox in a sec [14:06] rick_h_: i think they just want to be up-to-date before installing the python-software-properties and then it is required to update again after adding the PPAs. [14:06] rick_h_, Cool thanks [14:06] probably the right thing to do for a clean install but annoying waste of time for devs [14:09] antdillon: do you need the JujuBaseView (horrible name of something to get model binding? [14:10] rick_h_, I dont believe I do, I inherited it from the login view [14:10] * bac off to get flu shot. bbiab with sore arm [14:10] antdillon: coolio [14:10] bac: ouch, yea mine was sore for 2 days :/ [14:29] man, lbox hates me today. Timed out 3 times now [14:32] hey antdillon. did you get the info you wanted about flags? Sorry, was out for a while on a tour of a school for my older son [14:33] gary_poster, Yeah rick_h_ is on it. Thanks [14:33] cool [14:33] antdillon: https://codereview.appspot.com/14700043 is your review url [14:34] antdillon: will review and add notes. We'll need to rope a second person in on it and go through qa and such. [14:37] rick_h_, Great thanks [14:37] rick_h_, I'll wait for a second reviewer and your notes [14:38] antdillon: yep, and I'll add notes for the feature flag in my review notes. [14:38] antdillon: the reason we're flagging it is that we expect more feedback/changes before we 'release' it to the wild? [14:39] rick_h_, The main reason is because its not storing the dismiss value so its showing on each reload which would get annoying quickly [14:39] antdillon: oh, yea that's not good [14:56] antdillon: https://codereview.appspot.com/14700043/ is initial review feedback and one is as not on checking the feature flag. [14:56] loading up to qa now [14:57] antdillon: and I thought you had a test file? I don't see any test code [15:00] antdillon: qa notes added as well [15:00] rick_h_, No test file im afraid [15:00] rick_h_, Thanks [15:01] antdillon: ok, we'll have to work on that. :) [15:01] rick_h_, Sure [15:02] juju-gui, hadoop charm still broken (broken again!) in comingsoon. release blocker. will make critical card asap [15:02] rick_h_, Should I work through your notes and let you know when my branch is updated? [15:02] antdillon: yes please. Let me know if you need to chat on any of them. [15:03] rick_h_, Will do thanks [15:03] antdillon: and once you've got an update I'll pull those down and update this merge proposal. Then we'll get a second review in on the updated branch. [15:04] bcsaller, please ping when you are around [15:04] gary_poster: I'm around [15:35] bcsaller, did a code review as well. lots of small stuff. If you would like me to prepare a branch with the trivials, to save you some time, I would be happy to. [15:35] (well, there was lots of small stuff, but a few less trivial issues as well) [15:39] rick_h_, I didn't use CSS hover for the cross icon because its moving the sprite background so I need to change the class on it. Is there a way to use the sprite with :hover? [15:41] antdillon: hmm, I missed that. I thought it was just for the hand effect. I didn't notice a cross icon. /me qa's it up again [15:41] rick_h_, Cool thanks, thought I was missing something :) [15:42] j u j u g u i call in 18 (me warms up) [15:42] antdillon: any reason we can't just show/hide the panels? and the cross not need to move? [15:43] antdillon: ignore me, I'm confused [15:44] antdillon: ok, so the sprite X icon. Ugh, ok. Let it be then. [15:44] rick_h_, Cool, might be nice in the future to have a hander class for sprite interactions [15:45] handler* [15:48] fwiw, the hadoop issue is with the charm, not the gui. it is the same thing hatch fixed a week or two ago. [15:48] phew, not my fault then :) [15:48] gary_poster: proof needs <3 then? [15:49] gary_poster: or charmworld ingest? [15:49] rick_h_, oh, good point. yeah, proof I guess. the problem is that config option names must not have dots in them [15:49] it breaks CLI and GUI in different ways [15:49] jujugui call in 10 [15:49] gary_poster: cool, maybe hit up marcoceppi witha bug while he's in the proof code right now [15:49] jujugui call in 10 [15:50] !!! [15:50] gary_poster: goes 2 for 2! [15:50] :-) [15:50] while holding a side conversation, impressive [15:50] rick_h_, yeah, thanks will do [15:50] I had it all typed out and everything :( [15:50] well, you knowm, you have to keep focus on what's important :-P [15:50] lol, I see SFO time being spent on... "guitbot2000" [15:50] gary_poster: I think there's a fix for that already [15:51] marcoceppi, there's a fix for the charm that we offered but apparently t has not been merged [15:51] Jeff gave it to Kapil--we did not propose [15:51] marcoceppi, I can try to get that out there [15:51] marcoceppi, but you are saying proof already has that rule? [15:52] i pushed it out to a charm branch [15:52] gary_poster: there are a bunch of fixes to proof from adeuring, let me check if that was one [15:52] thanks marcoceppi [15:52] hazmat, yeah, cool. Could you propose it for merging to official charm? Or do you want me to? [15:53] gary_poster, if you could that would be great [15:53] hazmat, cool will do. [15:54] gary_poster, marcoceppi: the charmtools proofer will _not_ check if dots appear in config option names. But that would be easy to add, of course [15:54] gary_poster: huw asked me on irc last night about priority for new blingy import/export icon work. i wasn't at my computer so i didn't reply. i'd guess it is low but don't know what he's working on. [15:54] adeuring: cool, thanks for the confirmation. I'll just throw that in there [15:54] gary_poster: is that policy anywhere? [15:55] hazmat: ? [15:55] marcoceppi, no. [15:56] marcoceppi, juju docs are the only place they would, most of the rules are encoded as code the core charm implementation, the intent was that proof would remain an independent validation of that implicit spec. [15:56] seems that we need a way to synchronize the charm docs and the proofer somehow... [15:57] i dunno that i find that intent very compelling.. its more bugs than reality.. the alternative is to just go ahead and have proof run a simple go program against the charm [15:57] so that proof is a final word on validity that matches against juju and the store [15:58] jujugui clal in 2 [15:59] or 1 [16:24] bac oops he ran away [16:27] antdillon: very cool, one more comment on your build/browse. Did you verify the QA issue where the mask was acting funny the first two steps? [16:33] rick_h_, Oh no, must have missed that one [16:33] rick_h_, Let me take a look [16:48] rick_h_, Resolved your comment. It's EOD here soon, is there anything you need from me before I head off? [16:48] antdillon: looking [16:49] antdillon: did you see the one "QA Notes:" ? [16:49] antdillon: if you're EOD I'll hit you up in the morning and we can go from there and get this landed tomorrow [16:49] antdillon: thanks for the work on it today. [16:50] bac: welcome back! fyi updated with comments and code https://codereview.appspot.com/14441069/ [16:51] rick_h_, Got the QA Notes one and replied. Np, thanks for your review today. See ya tomorrow. [17:02] frankban, LGTM with a few trivials [17:02] have a nice evening [17:02] gary_poster: thanks! [17:08] gary_poster: re "juju switch". I decided to use that because 1) "juju switch myenv" does not change the environment.yaml[default] and 2) ~/.juju/current-environment seemed to me an implementation detail for the thing exposed by "juju switch" (with no args). I agree it feels fragile, I'd like something like "juju switch --format json"... [17:10] frankban, sounds good. file juju core bug tomorrow? [17:10] gary_poster: ok :-) [17:35] re-reviewed rick_h_. looks great. [17:35] bac: thanks! [17:55] gary_poster, I am filing your BPs etc for the sprint, so if you get a ton of spam, I apologize now! [17:56] :-) ack jcastro thx [19:06] my favorite recent discover is 'pp' in pdb. for the longest time i would import pprint when i needed it. silly me. [19:07] s/discover/discovery [19:25] jujugui, super-duper important sprint activity: update our wordpress blog's background to have new service blocks! Let's do it! [19:26] gary_poster: +1 [19:26] :-) [19:26] gary_poster, haha, yeah! Someone make a pretty openstack deployment! [19:26] :-) [19:30] OK, a prize for the person who guesses this without looking at blog stats: what country has the most page views of the gui blog in the past 30 days? If you win, you have to tell me how the heck you guessed this country. [19:34] uruguay because that's where all the cool people are migrating [19:34] heh. nope. [19:36] is the prize your voice on my answering machine? [19:36] lol, I was thinking more like buying a drink [19:36] * bac relocates to a place with more jojo [21:54] hey Makyo, I am trying to do review/qa of bcsaller's branch now. could you be the other reviewer pls? https://codereview.appspot.com/14695043/ [22:02] oh, thanks [22:03] Morning [22:03] hey huwshimi [22:07] gary_poster: So I think the bundle charm details is a much larger piece of work than I'm going to be able to do... [22:08] huwshimi, oh, really? ok. what's the story? [22:09] gary_poster: Well, it looks like we need to modify API calls and do a bunch of other things to actually get the charm details [22:10] gary_poster: Stuff that I'm not qualified to do :) [22:10] huwshimi, how so? we already have charm_metadata on the bundle [22:10] gary_poster, yeah [22:10] that's everything you need, isn't it? [22:10] thanks Makyo [22:10] * Makyo also, returns from dogwalk. Freezing :| [22:10] :-/ [22:11] gary_poster: I'm not sure, I'm going off what Rick said yesterday [22:11] gary_poster: And he seemed to think we didn't have the pieces in place yet... [22:11] Next winter comes slowly pale meager and old, first trembling with age and then quiv'ring with cold. (Why I still have all my Freshman jury music memorized is beyond me...) [22:12] I think we may be reading different things huwshimi. I'm trying to rush but I can probably do a hangout in 5 [22:13] gary_poster: We can if you like, but this stuff is WAY over my head :( [22:13] * gary_poster had to look up to see Makyo's quote was Purcell [22:13] huwshimi, ok. let's talk [22:13] soon [22:13] ready soon [22:13] gary_poster: OK, let me know :) [22:13] Though Sting did a cover recently, gary_poster :) [22:14] Makyo, heh [22:14] Makyo, I think I heard that album. pseudo early music? my vague memory is "enh" [22:14] gary_poster, yeah. It wasn [22:14] Oop. [22:14] It wasn't awful, but gosh, Sting's mouth is big. [22:15] lol [22:15] Ditto Bowie. Words just come tumbling out, tripping over everything in the way. [22:15] heh, yeah I think I know what you mean. I can remember delivering a song in that manner and being less than pleased when I heard the recording [22:16] And no matter how much Christmas music he sings, he'll always be Feyd Rautha in David Lynch's Dune wailing "Rooooxaaanne!", to me. [22:16] His past is colorful, I guess. [22:17] lol yeah [22:17] bcsaller, "Awesome! LGTM and qa good, with trivial. Thank you!" [22:17] ahh great [22:17] oh wait I forgot to try live instance, 1 sec [22:17] Uh, sorry, reviews. Will get on that. [22:21] bcsaller only one qa problem, and I'm ok with pushing this to another branch (& person) if you wish. it is kind of important, though. In a real environment, I did the following: (1) open GUI. I see GUI service in center. fine. (2) drag GUI. GUI moves around fine. (3) Make mediawiki ghost via drag. Fine. (4) drag ghost. Fine. (5) Deploy ghost. fine. (6) reload GUI. Uh-oh: charms are to bottom right, and en [22:21] vironment is not panned to center. [22:22] hmm [22:22] bcsaller, ok with landing without fix, though fix will be necessary from team before release. Thanks! [22:22] huwshimi, I need to go to dinner. will ping you later. [22:22] gary_poster: Sure [22:23] gary_poster: yeah, I might have to pass that off, doesn't seem too big though, might have to put the centroid recording back in, I didn't see that case in my qa [22:33] gary_poster: huwshimi all good? [22:34] gary_poster: huwshimi the api thing was just adding an ATTR to the bundle model if I follow where you're chatting [22:35] rick_h_: We were going to try and have a call to figure things out... [22:35] huwshimi: k, let me know if there's anything I can do to help. You and I can chat if you want real quick if it unblocks you? [22:36] huwshimi: the card isn't very descriptive so not 100% sure what you're heading on. Is there a wireframe you're working from? [22:37] rick_h_: Nope, making things up as I go :) [22:37] huwshimi: awesome! :) [22:37] huwshimi: invite on the way [22:37] rick_h_: Essentially we need to display a list of charms and have some what to show their config details as well [22:37] *some way [22:37] all the charms config details? [22:37] huwshimi: https://plus.google.com/hangouts/_/lite/2bdee1ff16f92239f6723ec2db0efd4af3bbde88?hl=en [22:39] rick_h_: "You're not allowed to join this video call." [22:39] bah [22:40] rick_h_, huwshimi may I come by? [22:40] huwshimi: try https://plus.google.com/hangouts/_/10fe9c48d75b9ebbe273219cb03f4e53869ce4f7?authuser=1 [22:41] but take off "authuser" [22:41] gary_poster: coolio we're in [22:58] gary_poster, bcsaller will work on centering. [22:58] Awesome thank you Makyo. Making card for you now [22:58] Makyo: I suspect my change about not recording centroid will go back and it can solve that case, I didn't think of it at the time [23:01] rick_h_: Hmmm... still getting "Failed to load charm details. Charm API server did not respond". It appears to be looping through the charms ok: http://paste.ubuntu.com/6242914/ [23:02] gary_poster, bcsaller cool, thanks. Will get on that. LGTM otherwise; I say land. [23:02] (If you haven't already) [23:02] +1 bcsaller [23:02] I think that was a pretty good "on to the next thing" branch [23:03] definitely! thanks bcsaller :-) [23:03] we'll miss you, but you will do awesome things in the new place [23:03] thanks :) [23:04] I'll miss this team [23:13] huwshimi: what call is it making? [23:14] huwshimi: in the network tab, what api call is going out that's failing? [23:14] huwshimi: this is called from the _renderBundleView I assume? [23:16] rick_h_: Yeah, called in the _renderBundleView [23:18] rick_h_: How would I tell which call it is? [23:18] huwshimi: it should be a 404 or something? [23:18] a failed request [23:18] rick_h_: Nothing is 404ing. I only get the error in the gui itself [23:18] (a growl-style notification) [23:19] huwshimi: right, but that error is that the Gui made an API call (which should show in the network tab as the page loads) and failed to get a successsfull response from manage.jujucharms.com [23:19] huwshimi: so some api call m.j.c/api/3/xxxxxxx should show as a failed 404 or something strange in the network tab [23:20] rick_h_: Yeah, I dunno why I'm not getting one... [23:20] huwshimi: hangout again ands screenshare? maybe I can spot what is odd going on there? [23:20] rick_h_: Sure :) [23:20] https://plus.google.com/hangouts/_/10fe9c48d75b9ebbe273219cb03f4e53869ce4f7?authuser=1