/srv/irclogs.ubuntu.com/2015/01/25/#juju-dev.txt

=== Spads_ is now known as Spads
=== bodie__ is now known as bodie_
=== mup_ is now known as mup
=== anthonyf is now known as Guest90478
=== meetingology` is now known as meetingology
=== Tribaal_ is now known as Tribaal
wallyworld_axw: hey, you seen this issue before? I'm trying to land dimiter's fix to unblock trunk10:56
wallyworld_Extant directories unknown:10:56
wallyworld_ labix.org/v2/mgo10:56
axwerrm10:56
axwnope10:56
wallyworld_seems a bot scripting issue10:56
axwwe shouldn't be using labix.org/v2/mgo anymore10:56
wallyworld_yup10:56
axwit's not snuck into dependencies.tsv is it?10:56
wallyworld_hmm, don't think so, i'll check10:57
axwnope10:57
wallyworld_i'll have to grep the release tools scripts, i think they print that message10:57
wallyworld_i really want to get our stuff landed, otherwise will need to make a feature branch, but if i do it off trunk now, local lxc will be broken until the fix lands10:58
=== kadams54 is now known as kadams54-away
anastasiamacmenn0: waigani:ping21:14
menn0anastasiamac: pong21:32
menn0anastasiamac: pong (again)21:57
menn0waigani: just reviewed the FindId/RemoveId change.22:09
waiganimenn0: thanks22:09
anastasiamacmenn0: sorry - day off... kids running around... m missing ur answers :D22:17
anastasiamacmenn0: do u know what tim changed in facade creation/calling?22:17
anastasiamacmenn0: related bug 141402722:18
mupBug #1414027: Increment client facade or return agent version in login <juju-core:Triaged> <https://launchpad.net/bugs/1414027>22:18
menn0anastasiamac: not off the top of my head22:18
anastasiamacmenn0: i think its smth in dependencies...22:18
menn0anastasiamac: but I can try and figure it out if it's urgent22:18
anastasiamacmenn0: is tim not around today?22:18
menn0anastasiamac: he's back tomorrow22:18
menn0anastasiamac: has the day off today22:19
anastasiamacmenn0: k. i'll ask tomorrow22:19
anastasiamacmenn0: it's not urgent 4 me :D22:19
menn0anastasiamac: ok22:19
anastasiamacmenn0: but hazmat has troubles calling all new clients22:19
anastasiamacmenn0: he can see them but when he calls them he get errors22:20
menn0anastasiamac: ok. i'll try and have a look and see if this is related to Tim's changes22:20
menn0anastasiamac: stop working on your day off!22:20
anastasiamacmenn0: m nt working :P m talking to u :D22:20
menn0anastasiamac: that's work :)22:20
anastasiamacmenn0: just an fyi, all feature tests run fine22:20
anastasiamacmenn0: they all work when u do a full go install ../... from juju root...22:21
anastasiamacmenn0: only new facades are affected22:21
anastasiamacmenn0: axw saw the same behaviour on his environment (i did too)22:21
anastasiamacmenn0: and had to re-build from scratch...22:21
anastasiamacmenn0: as per above :D22:22
menn0anastasiamac: ok. good to know.22:23
anastasiamacmenn0: thnx! that's what at least 2 cases of scotch now?.. :D22:26
menn0anastasiamac: umm... I haven't done anything yet :)22:26
menn0anastasiamac: but yes please :-p22:26
anastasiamacmenn0: 1 for CI blockers (well for unblocking them) and 2nd in anticipation :D22:28
menn0anastasiamac: i'll see what I can do. onyx is in crunch mode at the moment, trying to get MESS working before the mid-cycle sprint.22:29
anastasiamacmenn0: understood... tanzanite's storage has a new client for sprint demo too...22:30
anastasiamacmenn0: just making sure that there will b no hiccups for us either :D22:30
menn0anastasiamac: I will look at that bug later today22:31
menn0anastasiamac: I even just assigned to me and everythign22:31
anastasiamacmenn0: thnx :D but really m happy to wait til tim and tomorrow22:33
menn0anastasiamac: well I'll see if I can figure it out but if it turns into an epic then Tim can look tomorrow.22:34
anastasiamacmenn0: thnx :D22:37
hazmatmenn0, dropped a note on 141402722:57
menn0hazmat: just read it. I'm pretty sure that the idea is if the client knows about /environment/:uuid/api it should use that.22:59
menn0hazmat: I believe that much of the API available at the root is going to go away at some point22:59
hazmatmenn0, you mean it should keep trying different endpoints till one works?22:59
hazmatie. does a 1.20.x env respond to that?23:00
menn0hazmat: not sure. thumper has a clearer picture of how this is supposed to work.23:00
* menn0 checks the code23:01
hazmatmenn0, i'm writing an app/client that's autonegotiating from 1.18 to  1.23.. on facades that esasy23:01
hazmatchanging the endpoint is something else23:01
* hazmat waits up for the australian open to start23:04
menn0hazmat: ok, I'm not exactly sure what the plan is. thumper will know more and he's back tomorrow.23:11
menn0hazmat: AFAIK the existing env-specific functionality will continue to work from the root for some time (but probably only for the initial environment I guess)23:12
menn0hazmat: actually, thinking some more, the env functionality at root should work for any env, as long as you log in to that env.23:15
hazmatmenn0, sure but without telling clients that they may talking to a multi-state server env, they don't know till they Info() on the env to get version.. it should go to the jenv so they know where to connect,  connecting to root only works for legacy envs (env 0) in multi-tenant state servers, and connecting to multiple state servers. juju should add some info to disambiguate that in the client side info it stores, and avoid apps having to hit multiple endp23:16
hazmatoints (legacy, multi-style) to discover the correct one.23:16
hazmatmenn0, you login by username, password, not by env uuid23:16
hazmaton the root23:16
menn0hazmat: i'm also pretty sure you can specific the environment tag at login time23:17
* menn0 checks23:17
menn0hazmat: actually, it's before login23:18
menn0hazmat: api.Info has a EnvironTag field. if you set that you get a connection for that envrionment23:19
menn0hazmat: if it's blank you get the initial / state server env23:19
hazmatre before login, was there it weaved together prior to login in the src?23:21
menn0never mind23:21
menn0i've just looked further23:21
menn0setting that field changed the endpoint URL used by the juju API client23:22
menn0it's not passed through23:22
* menn0 goes to check what 1.20 does with that...23:22
menn0and 1.18 even23:23
hazmats/was/where23:23
hazmatk23:23
hazmatmenn0, so yeah.. client api compatibility across multiple versions is the question.. i think dropping it in the jenv is appropriate. a multi-tenant root would be useful, but there's no initial request outside of login which would be a fine place to pass as older clients would ignore, but its also keeping compatibility with the older endpoints23:25
hazmatwhich is nice, but given apps want to be compatible with both, also enshrines it as endpoint for a little while... and of course assuming it does take env-uuid as a param on login23:26
hazmatenviron tag is already in the jenv23:27
hazmatat least for 1.23.. dunno how far back that goes.23:27
menn0still digging... code for 1.18 was in LP so it's a little harder to check23:28
hazmati'm currently doing conditional on that.. but that's a rather subtle distinction which would be better explicit, given the variety of tooling around the api.23:28
hazmatie. juju client compatibility is great but there are other api clients that need to do the same dance..  at least  js and multiple py clients wrt to common usage... so some explicit delineation and email/doc would be useful..23:31
menn0hazmat: i've almost got this figured out. bear with me.23:34
menn0hazmat: so environ-uuid has been in the .jenv since 1.20.0. 1.18 didn't have it.23:35
menn0hazmat: and if you have environ-uuid in the .jenv then you should be using a endpoint of /environment/:uuid/api23:36
menn0hazmat: otherwise you use the root23:36
menn0hazmat: basically, if the jenv includes environ-uuid then the server supports it23:37
menn0hazmat: thumper will probably have some more detail but the above is what I'm seeing from the code at various points in time23:38
menn0hazmat: agreed that this needs to be documented better23:38
hazmatmenn0, cool, thanks.. although i'll note.. not all clients use jenv (landscape and gui both collect info via forms)23:38
menn0hazmat: ok. and that's been fine up until now because there's only been one env and if you connect at root you just get the initial env.23:40
hazmatfor client-side py tools that should handle things, thanks again23:40
menn0hazmat: those forms are going to need to take the env uuid as well i guess23:40
menn0hazmat: i'll follow up with thumper tomorrow and confirm all this and get the docs updated23:41

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