/srv/irclogs.ubuntu.com/2013/02/15/#juju-gui.txt

=== _mup__ is now known as _mup_
* bac has new super-secure pypi password12:23
* frankban remembers to change his pypi password, thanks to bac12:44
* gary_poster too13:09
bacMakyo: 2nd review done14:17
benjigary_poster: is the API sketch in http://paste.ubuntu.com/1628460/ still applicable?14:57
gary_posterbenji, yes, though I've given better links lately. Getting...14:58
benjiI ask because it does not take a command-centric approach, but invents an entirely new vocabulary.14:58
gary_posterbenji  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
gary_posterbenji, the Client object is for commands14:59
gary_posterbenji, the new vocabulary is for Roger's other responsibility: letting Juju components talk/listen among themselves15:00
benjiah, ok15:00
benjigary_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
gary_posterover 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 later15:01
gary_posterbenji, call on juju gui15:01
benjik15:02
bacgary_poster: may i join?15:03
gary_posterof course bac15:03
* bac getting headset15:03
hazmati swear every day this week i get some internal interupt juju support request15:04
hazmattoday its gui with local provider15:04
hazmattherve, when you said you had gui working with local provider was that including being able to deploy services with the gui?15:05
hazmatso the issue is that the local provider storage is read only, which prevents deploys in the gui15:09
thervehazmat, I didn't try that15:13
gary_posterjujugui call in 215:28
gary_postergoodspud call15:30
Makyolp:~makyo/juju-gui/ghost-minors-1125506-1124414 15:33
Makyohttps://codereview.appspot.com/7317046/15:33
bacbenji, gary_poster: when were we going to talk?16:02
gary_posterbac soon will ping16:02
bacgary_poster: ok, if not let me know so i can grab lunch16:02
gary_posterack bac16:03
* bac grabs a healthy snack16:03
gary_posterbac go get lunch.  we'll talk in 56 minutes bac & benji?16:04
bacgary_poster: ok16:14
hazmatjujugui  .. sprint confirmed, ticket booking begins, email to follow17:23
gary_posterawesome17:23
bachazmat: to ATL17:23
hazmatbac, yes17:31
hazmatas soon as the call is done i'll fire off some emails17:31
gary_posterbcsaller, 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 deal18:20
bcsallergary_poster: let me read over what you wrote and then we can do a call when you have time18:28
gary_postercool thanks bcsaller 18:28
gary_posterbcsaller can talk whenever you are avail.  no rush19:07
bcsallergary_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
gary_posterbcsaller, sounds good, talk to you then19:18
benjibac: so, how are we going to get a foothold on this project?19:27
bacbenji: pick a cmd and forge on, learn from mistakes, repeat smarterly19:28
benjibac: sounds good; pairing?19:28
bacbenji: sure.  we can get the ball rolling today and then i'll run with it monday19:29
bacbenji: g+?19:29
benjisure19:29
gary_posterhazmat do you know offhand if we can stuff annotations in the .json files for improv?20:48
hazmatgary_poster, we can of course, drop me a sample.json via email and i'll take care of it20:49
hazmatgary_poster, at the moment its not done, but its trivial to do so20:49
gary_posterawesome thanks hazmat.20:50
Makyohazmat, gary_poster, this is re: testing landscape annotations.20:50
hazmatack21:05
hazmatgary_poster, Makyo do we have such a sample?21:06
gary_posterhazmat, no, we would throw one together by tacking an "annotations" key in to the objects21:06
hazmatgary_poster, sure but we know all the keys from therve's email on the subject?21:07
hazmatie. they haven't changed form/value?21:07
gary_posteryes hazmat21:07
gary_posterno hazmat21:07
hazmatokay.. i'll add it in.. no need for the sample21:07
gary_posterthank you21:08
hazmattherve, 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 think21:17
* gary_poster hopes he isn't part of allowing handoff hell yet again :-)21:19
gary_posterthat's what we're working with on the gui side hazmat, fwiw.  progress already made.  hope it doesn't change too much.21:20
gary_posterprobably small waste, but don't like egregious waste21:20
hazmatgary_poster, its a simple url change, not a big deal21:20
gary_postercool21:20
hazmateffectively i'm thinking we attach env=$env_uuid to all links outbound from the gui21:21
hazmatthe sample has that environment=env0 in the annotation.. but that seems a bit silly, unless landscape wants to ref a different id for the env21:22
gary_posterhazmat, ah I see.  ok, simple, yeah.  let us know how it works out. :-)21:23
gary_posterthanks again21:23
hazmatnp21:23
* hazmat pauses to book flights21:28
MakyoOh yeah, got those all booked.  Do we have a page to drop that info/with hotel info/etc?21:32
thervehazmat, sorry I don't get everything21:35
thervehazmat, by env you mean the Landscape env name?21:36
hazmattherve, yes the ones in the annotation urls21:40
hazmattherve, was wondering if the gui could just add the env uuid there or if its a value that landscape is generating21:41
thervehazmat, afaiu if it should be opaque to the gui21:41
therveI don't really see the point of it manipulating the value21:41
thervealthough using the uuid could be a good idea, we would do in on the landscape side21:41
hazmatgary_poster, are there any US folks around on monday?21:51
benjihazmat: I believe Gary and Brad will be.21:54
hazmatbenji, thanks21:54
hazmattherve, yeah.. that would save repeating that value quite alot in the annotations21:55
thervehazmat, you mean, if landscape doesn't put it by default?21:55
hazmattherve, right.. the gui can just append env=$uuid to query string urls21:56
thervehum...21:56
thervehazmat, yeah I don't want to put too much logic on the gui side though21:57
thervebut I guess we could do something like the alert url, where you can concat stuff to build the final URL21:57
hazmattherve, the gui is already complex, this is simple ;-)21:57
hazmattherve, sure if you want to keep a landscape specific value for the env instead of the uuid that would also work21:58
thervehazmat, it's not so much about complexity rather than control21:58
therveto not have to change things in 2 places if landscape needs a change21:59
hazmattherve, thats the nature of integration and apis..21:59
hazmatwe're already constructing the url in other places as you noted22:00
* Makyo dogwalkinates22:00
hazmattherve, i'm happy to deferring to whatever you'd prefer22:01
hazmatlost network connectivy, just wanted to make a suggestion that would keep things short22:02
hazmatand remove redundancy from the annotations22:02
thervehazmat, let me think about it, I'll send an email22:02
hazmatack22:03
gary_posterhazmat, benji, not I22:06
gary_posterbac will be22:06

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!