=== _mup__ is now known as _mup_ [12:23] * bac has new super-secure pypi password [12:44] * frankban remembers to change his pypi password, thanks to bac [13:09] * gary_poster too [14:17] Makyo: 2nd review done [14:57] gary_poster: is the API sketch in http://paste.ubuntu.com/1628460/ still applicable? [14:58] benji, yes, though I've given better links lately. Getting... [14:58] I ask because it does not take a command-centric approach, but invents an entirely new vocabulary. [14:59] benji http://bazaar.launchpad.net/~rogpeppe/juju-core/212-api-doc/view/head:/doc/draft/api.txt (comments or concerns about the doc can go here, btw: https://codereview.appspot.com/7314085/diff/1/doc/draft/api.txt) [14:59] benji, the Client object is for commands [15:00] benji, the new vocabulary is for Roger's other responsibility: letting Juju components talk/listen among themselves [15:00] ah, ok [15:01] gary_poster: nope, no ok (i.e., I do not understand): is https://codereview.appspot.com/7314085/diff/1/doc/draft/api.txt about "commands" or this internal juju-juju communication? [15:01] over the same socket, but on the different objects benji. As described now, we will never talk to anything other than the Admin object (only to log in) and the Client object. I've proposed subdividing the Client object into read-only and mutation but that might be for later [15:01] benji, call on juju gui [15:02] k [15:03] gary_poster: may i join? [15:03] of course bac [15:03] * bac getting headset [15:04] i swear every day this week i get some internal interupt juju support request [15:04] today its gui with local provider [15:05] therve, when you said you had gui working with local provider was that including being able to deploy services with the gui? [15:09] so the issue is that the local provider storage is read only, which prevents deploys in the gui [15:13] hazmat, I didn't try that [15:28] jujugui call in 2 [15:30] goodspud call [15:33] lp:~makyo/juju-gui/ghost-minors-1125506-1124414 [15:33] https://codereview.appspot.com/7317046/ [16:02] benji, gary_poster: when were we going to talk? [16:02] bac soon will ping [16:02] gary_poster: ok, if not let me know so i can grab lunch [16:03] ack bac [16:03] * bac grabs a healthy snack [16:04] bac go get lunch. we'll talk in 56 minutes bac & benji? [16:14] gary_poster: ok [17:23] jujugui .. sprint confirmed, ticket booking begins, email to follow [17:23] awesome [17:23] hazmat: to ATL [17:31] bac, yes [17:31] as soon as the call is done i'll fire off some emails [18:20] bcsaller, finally finished my first review. I have so many questions that I'd like to maybe talk about it, and I'd like to see the changes you make before landing, but I don't think it will be a big deal [18:28] gary_poster: let me read over what you wrote and then we can do a call when you have time [18:28] cool thanks bcsaller [19:07] bcsaller can talk whenever you are avail. no rush [19:18] gary_poster: I'm attempting to resolve a few minors that came from looking at your review, maybe we can wait till our scheduled call in 40 minutes? [19:18] bcsaller, sounds good, talk to you then [19:27] bac: so, how are we going to get a foothold on this project? [19:28] benji: pick a cmd and forge on, learn from mistakes, repeat smarterly [19:28] bac: sounds good; pairing? [19:29] benji: sure. we can get the ball rolling today and then i'll run with it monday [19:29] benji: g+? [19:29] sure [20:48] hazmat do you know offhand if we can stuff annotations in the .json files for improv? [20:49] gary_poster, we can of course, drop me a sample.json via email and i'll take care of it [20:49] gary_poster, at the moment its not done, but its trivial to do so [20:50] awesome thanks hazmat. [20:50] hazmat, gary_poster, this is re: testing landscape annotations. [21:05] ack [21:06] gary_poster, Makyo do we have such a sample? [21:06] hazmat, no, we would throw one together by tacking an "annotations" key in to the objects [21:07] gary_poster, sure but we know all the keys from therve's email on the subject? [21:07] ie. they haven't changed form/value? [21:07] yes hazmat [21:07] no hazmat [21:07] okay.. i'll add it in.. no need for the sample [21:08] thank you [21:17] therve, not sure if this is up to date, (looking at thread from ls annotations january) i don't think we need to be passing the env around in annotations, it would be the env uuid, and we'd be attaching that as param to all links i think [21:19] * gary_poster hopes he isn't part of allowing handoff hell yet again :-) [21:20] that's what we're working with on the gui side hazmat, fwiw. progress already made. hope it doesn't change too much. [21:20] probably small waste, but don't like egregious waste [21:20] gary_poster, its a simple url change, not a big deal [21:20] cool [21:21] effectively i'm thinking we attach env=$env_uuid to all links outbound from the gui [21:22] the sample has that environment=env0 in the annotation.. but that seems a bit silly, unless landscape wants to ref a different id for the env [21:23] hazmat, ah I see. ok, simple, yeah. let us know how it works out. :-) [21:23] thanks again [21:23] np [21:28] * hazmat pauses to book flights [21:32] Oh yeah, got those all booked. Do we have a page to drop that info/with hotel info/etc? [21:35] hazmat, sorry I don't get everything [21:36] hazmat, by env you mean the Landscape env name? [21:40] therve, yes the ones in the annotation urls [21:41] therve, was wondering if the gui could just add the env uuid there or if its a value that landscape is generating [21:41] hazmat, afaiu if it should be opaque to the gui [21:41] I don't really see the point of it manipulating the value [21:41] although using the uuid could be a good idea, we would do in on the landscape side [21:51] gary_poster, are there any US folks around on monday? [21:54] hazmat: I believe Gary and Brad will be. [21:54] benji, thanks [21:55] therve, yeah.. that would save repeating that value quite alot in the annotations [21:55] hazmat, you mean, if landscape doesn't put it by default? [21:56] therve, right.. the gui can just append env=$uuid to query string urls [21:56] hum... [21:57] hazmat, yeah I don't want to put too much logic on the gui side though [21:57] but I guess we could do something like the alert url, where you can concat stuff to build the final URL [21:57] therve, the gui is already complex, this is simple ;-) [21:58] therve, sure if you want to keep a landscape specific value for the env instead of the uuid that would also work [21:58] hazmat, it's not so much about complexity rather than control [21:59] to not have to change things in 2 places if landscape needs a change [21:59] therve, thats the nature of integration and apis.. [22:00] we're already constructing the url in other places as you noted [22:00] * Makyo dogwalkinates [22:01] therve, i'm happy to deferring to whatever you'd prefer [22:02] lost network connectivy, just wanted to make a suggestion that would keep things short [22:02] and remove redundancy from the annotations [22:02] hazmat, let me think about it, I'll send an email [22:03] ack [22:06] hazmat, benji, not I [22:06] bac will be