Thursday, 2017-04-27

*** d0ugal has quit IRC00:04
Qimingbriancurtin, imo, removing it sounds the right thing to do since it was deprecated two years ago00:05
mordredbriancurtin: I believe you're using keystoneauth.session for endpoint finding/filtering - but you might find https://review.openstack.org/459405 interesting/worth a read - it's an attempt to capture the total story around service version discovery ... I'd love to know if it's got any holes or issues from your POV00:09
mordred(if you have time - no worry/rush)00:09
mordredbriancurtin: it's also hairy - so it's entirely fine to have the opinion "screw that, I'll just let ksa do its thing thanks" :)00:09
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Fix more nova-net functional tests  https://review.openstack.org/46035600:28
*** hongbin has quit IRC00:33
*** annegentle has joined #openstack-sdks00:36
*** Kuwagata has joined #openstack-sdks00:44
*** hoangcx has joined #openstack-sdks00:45
*** gouthamr has joined #openstack-sdks00:46
*** Kuwagata has quit IRC00:49
*** madorn has quit IRC00:51
*** rajpatel has quit IRC00:53
*** sdague has quit IRC00:58
*** annegentle has quit IRC01:14
*** annegentle has joined #openstack-sdks01:19
*** madorn has joined #openstack-sdks01:19
*** prg3 has quit IRC01:38
*** prg3 has joined #openstack-sdks01:41
*** gouthamr has quit IRC01:42
*** annp has joined #openstack-sdks01:43
*** yanyanhu has joined #openstack-sdks01:50
openstackgerritSteve Martinelli proposed openstack/python-openstackclient master: Fix more nova-net functional tests  https://review.openstack.org/46035602:02
RuiChenI saw a project named "OSCLIENT (CLI)" in the project navigator page https://www.openstack.org/software/project-navigator , that means openstackclient or any other things?02:02
*** annegentle has quit IRC02:06
*** huanxuan has joined #openstack-sdks02:07
*** john-davidge has joined #openstack-sdks02:19
*** rajpatel has joined #openstack-sdks02:22
*** john-davidge has quit IRC02:23
*** reedip has joined #openstack-sdks02:34
*** dave-mccowan has quit IRC02:36
*** dave-mccowan has joined #openstack-sdks02:37
*** dave-mccowan has quit IRC02:37
*** dave-mcc_ has joined #openstack-sdks02:43
*** Kuwagata has joined #openstack-sdks02:45
*** chlong has joined #openstack-sdks02:48
*** Kuwagata has quit IRC02:50
*** reedip has quit IRC02:50
*** gouthamr has joined #openstack-sdks02:51
*** rajpatel has quit IRC03:00
*** reedip has joined #openstack-sdks03:03
openstackgerritMerged openstack/python-openstackclient master: Fix quota functional tests for nova-net  https://review.openstack.org/46033403:03
openstackgerritMerged openstack/python-openstackclient master: Fix server create with nova-net  https://review.openstack.org/46035203:06
*** rajpatel has joined #openstack-sdks03:16
*** rajpatel has quit IRC03:18
*** huanxuan_ has joined #openstack-sdks03:20
*** Kuwagata has joined #openstack-sdks03:22
*** huanxuan has quit IRC03:23
*** dave-mcc_ has quit IRC03:24
*** Kuwagata has quit IRC03:25
*** reedip has quit IRC03:26
*** reedip has joined #openstack-sdks03:32
*** reedip has quit IRC04:01
*** gouthamr has quit IRC04:01
*** reedip has joined #openstack-sdks04:23
*** salv-orlando has joined #openstack-sdks04:51
*** e0ne has joined #openstack-sdks05:49
*** e0ne has quit IRC05:55
*** reedip has quit IRC06:07
*** amotoki has quit IRC06:19
*** amotoki has joined #openstack-sdks06:20
*** d0ugal has joined #openstack-sdks06:44
*** salv-orlando has quit IRC06:48
*** yanyanhu has quit IRC06:51
*** hoangcx_ has joined #openstack-sdks06:52
*** hoangcx has quit IRC06:54
*** salv-orlando has joined #openstack-sdks07:07
*** Serlex has joined #openstack-sdks07:10
*** hoangcx has joined #openstack-sdks07:27
*** hoangcx_ has quit IRC07:28
*** ralonsoh has joined #openstack-sdks07:34
*** jpich has joined #openstack-sdks07:36
*** e0ne has joined #openstack-sdks07:52
*** kuzko has quit IRC07:55
*** hoangcx_ has joined #openstack-sdks07:55
*** kuzko has joined #openstack-sdks07:57
*** hoangcx has quit IRC07:57
*** hoangcx has joined #openstack-sdks08:18
*** hoangcx_ has quit IRC08:20
*** ssbarnea has joined #openstack-sdks08:22
*** amotoki has quit IRC08:22
*** ssbarnea has quit IRC08:28
*** amotoki has joined #openstack-sdks08:32
*** reedip has joined #openstack-sdks08:40
*** annp has quit IRC08:41
*** hoangcx has quit IRC08:42
*** Serlex1 has joined #openstack-sdks08:45
*** d0ugal has quit IRC08:45
*** Serlex has quit IRC08:45
*** e0ne has quit IRC08:48
*** e0ne has joined #openstack-sdks08:49
*** reedip has quit IRC09:02
*** d0ugal has joined #openstack-sdks09:04
*** d0ugal has quit IRC09:04
*** d0ugal has joined #openstack-sdks09:04
*** aarefiev_afk is now known as aarefiev09:16
*** john-davidge has joined #openstack-sdks09:18
*** amotoki has quit IRC09:21
*** amotoki has joined #openstack-sdks09:22
*** amotoki has quit IRC09:23
*** e0ne has quit IRC09:30
*** e0ne has joined #openstack-sdks09:36
*** amotoki has joined #openstack-sdks09:41
*** cdent has joined #openstack-sdks09:43
*** salv-orl_ has joined #openstack-sdks09:49
*** salv-orlando has quit IRC09:52
*** d0ugal has quit IRC10:01
*** d0ugal has joined #openstack-sdks10:08
*** gildub has quit IRC10:10
*** annp has joined #openstack-sdks10:14
*** d0ugal has quit IRC10:21
*** d0ugal has joined #openstack-sdks10:26
*** huanxuan_ has quit IRC10:27
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Start using 'cliff.sphinxext'  https://review.openstack.org/45286110:28
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Convert 'server' commands to use autoprogram  https://review.openstack.org/45845210:28
*** amotoki has quit IRC10:28
*** gildub has joined #openstack-sdks10:34
*** jkilpatr has quit IRC10:38
*** salv-orl_ has quit IRC10:38
*** gildub has quit IRC10:40
*** annp has quit IRC10:42
*** d0ugal has quit IRC10:45
*** sdague has joined #openstack-sdks11:04
*** d0ugal has joined #openstack-sdks11:05
*** jkilpatr has joined #openstack-sdks11:08
*** dave-mccowan has joined #openstack-sdks11:09
*** markvoelker_ has joined #openstack-sdks11:11
*** markvoelker has quit IRC11:12
*** cdent has quit IRC11:30
*** markvoelker_ has quit IRC11:38
openstackgerritSteve Martinelli proposed openstack/python-openstackclient master: Fix more nova-net functional tests  https://review.openstack.org/46035611:40
*** john-davidge has quit IRC11:44
*** john-dav_ has joined #openstack-sdks11:44
*** amotoki has joined #openstack-sdks11:45
*** ssbarnea has joined #openstack-sdks11:48
*** gildub has joined #openstack-sdks11:53
*** amotoki has quit IRC11:58
*** amotoki has joined #openstack-sdks12:01
*** dgonzalez has quit IRC12:02
*** dgonzalez has joined #openstack-sdks12:07
*** annegentle has joined #openstack-sdks12:13
*** amotoki has quit IRC12:19
*** ssbarnea has quit IRC12:21
*** amotoki has joined #openstack-sdks12:25
*** amotoki has quit IRC12:34
*** markvoelker has joined #openstack-sdks12:35
*** annegentle has quit IRC12:41
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery  https://review.openstack.org/45971012:45
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guideline describing a cloud profile document  https://review.openstack.org/45986912:45
openstackgerritMonty Taylor proposed openstack/api-wg master: Add document describing consuming discovery  https://review.openstack.org/45940512:45
mordreddtroyer: thanks for the review - great feedback12:49
mordredand *doh* - how did I mess mention of volumev212:49
mordrededleafe: I'm starting to think that maybe there should be two documents where version discovery one is - one about version discovery, and one about what should go into the service catalog12:50
*** bobh has joined #openstack-sdks12:52
*** amotoki has joined #openstack-sdks12:55
*** cdent has joined #openstack-sdks12:57
edleafemordred: Yeah, I'm not sure about the exact breakdown, but in general there should be a "here's how to set things up correctly as a provider", and a "here's to process things correctly as a consumer"13:02
edleafeMost of the API-WG stuff focuses only on the former13:02
*** d0ugal has quit IRC13:04
*** Guest94155 has quit IRC13:10
openstackgerritMonty Taylor proposed openstack/service-types-authority master: Add an attribute for aliases  https://review.openstack.org/46053913:12
openstackgerritMonty Taylor proposed openstack/service-types-authority master: Add explicit mention of versions  https://review.openstack.org/46054013:12
*** salv-orlando has joined #openstack-sdks13:12
mordrededleafe: yah - cdent and I chatted briefly about a similar breakdown yesterday - which is "this is how a service should write an API" and "this is an action a deployer should take that affects the API"13:13
mordrededleafe: but I'm happy to be mixing things up and adding this new confusing "this is how to process things as a consumer" :)13:13
edleafeThe consumer angle is new to us. We've assumed that if you follow the guidelines, it'll be super-easy for the consumer. :)13:15
mordrededleafe: I look forward to the day when this is true!13:15
mordrededleafe: (I do agree quite strongly with that goal, btw - in some ways the "consuming-discovery" document is a bit of a cautionary tale explaining why the discovery guideline is important)13:16
edleafeAh, I like that POV. "While we strive to make things easy, we also need to handle some inherently complex stuff."13:17
*** d0ugal has joined #openstack-sdks13:20
openstackgerritBrian Curtin proposed openstack/python-openstacksdk master: Deprecated Message v1  https://review.openstack.org/46054413:23
dtroyerthat is an uphill battle on the client dev side too, not just the API developers…  it takes new contributors a bit to understand why OSC goes to extra lengths to make things simpler for the user.  "But I'd have to make another API call, can't the user just do that?"13:23
openstackgerritBrian Curtin proposed openstack/python-openstacksdk master: Deprecate Message v1  https://review.openstack.org/46054413:24
edleafedtroyer: heh, I feel your pain. Developing an intermediate layer, like an SDK or a client like OSC, requires both consuming and creating interfaces13:26
mordrededleafe: yup. super fun13:28
*** madorn has quit IRC13:39
*** annegentle has joined #openstack-sdks14:03
*** hongbin has joined #openstack-sdks14:13
openstackgerritMerged openstack/python-openstacksdk master: Port database v1 to resource2  https://review.openstack.org/46023614:16
openstackgerritMerged openstack/python-openstacksdk master: Port image v1 to resource2  https://review.openstack.org/46029114:17
openstackgerritMerged openstack/python-openstacksdk master: Port identity v2 to resource2  https://review.openstack.org/46028814:17
*** salv-orlando has quit IRC14:23
openstackgerritMonty Taylor proposed openstack/service-types-authority master: Add aliases for historical well-used block-storage types  https://review.openstack.org/46060914:27
*** d0ugal has quit IRC14:28
cdentedleafe: that's a good point about next_min_version. I think when I added it there I just kind of assumed next_min_version would resolve (to most likely merging) before the interop stuff did14:29
cdentit has not turned out that way14:29
*** gouthamr has joined #openstack-sdks14:31
amotokiankur-gupta-f4: ping14:31
edleafecdent: Let's bring that up at today's meeting14:32
*** d0ugal has joined #openstack-sdks14:32
dtroyer#RgentL3314:33
mordreddtroyer: welp, so much for that one14:34
dtroyernow if you only knoew what it actually was :)14:34
mordreddtroyer: it's like the key I have on my keychain that I have no idea what lock it's for14:35
dtroyerI have a box of those.  it's the one marked "Bugatti" that I'm most interested in finding14:36
mordredmmm veyron ...14:36
dtroyerthe alarm on the one across town goes off every time I try to test it14:37
*** gildub has quit IRC14:38
*** salv-orlando has joined #openstack-sdks14:38
*** d0ugal has quit IRC14:39
mordredthat's so rude!14:39
openstackgerritRodrigo Duarte proposed openstack/python-openstackclient master: Implied Roles  https://review.openstack.org/29025314:40
*** d0ugal has joined #openstack-sdks14:48
*** ralonsoh_ has joined #openstack-sdks14:55
*** d0ugal has quit IRC14:56
*** ralonsoh has quit IRC14:57
*** Serlex1 has quit IRC14:57
openstackgerritMerged openstack/python-openstacksdk master: Deprecate Message v1  https://review.openstack.org/46054414:58
openstackgerritMerged openstack/python-openstacksdk master: Revert "HAProxy uses milliseconds ..."  https://review.openstack.org/35703114:59
*** Kevin_Zheng has joined #openstack-sdks14:59
*** ralonsoh has joined #openstack-sdks15:02
*** ralonsoh_ has quit IRC15:02
*** john-dav_ is now known as john-davidge15:04
annegentleopenstackclient question in #openstack, y'all15:07
dtroyerhi annegentle!15:07
annegentlehi dtroyer !15:07
openstackgerritMonty Taylor proposed openstack/service-types-authority master: Add explicit mention of versions  https://review.openstack.org/46054015:11
openstackgerritMonty Taylor proposed openstack/service-types-authority master: Add an attribute for aliases  https://review.openstack.org/46053915:11
*** zigo has quit IRC15:17
*** d0ugal has joined #openstack-sdks15:21
*** markvoelker has quit IRC15:30
*** markvoelker has joined #openstack-sdks15:31
*** john-davidge has quit IRC15:33
*** aarefiev is now known as aarefiev_afk15:35
briancurtinQiming: https://pypi.python.org/pypi/openstacksdk/0.9.16 released. doesn’t have a whole lot, just the porting changes plus two neutron changes15:45
openstackgerritEric Fried proposed openstack/keystoneauth master: WIP: Introduce keystoneauth1.loading.adapter  https://review.openstack.org/46033715:47
*** salv-orl_ has joined #openstack-sdks15:50
openstackgerritMonty Taylor proposed openstack/api-wg master: Add support for historical service type aliases  https://review.openstack.org/46065415:50
mordreddtroyer: ^^ that should handle your feedback about volumev2 and friends15:50
*** salv-orlando has quit IRC15:52
openstackgerritMonty Taylor proposed openstack/api-wg master: Add support for historical service type aliases  https://review.openstack.org/46065415:52
openstackgerritMonty Taylor proposed openstack/api-wg master: Add document describing consuming discovery  https://review.openstack.org/45940515:52
*** e0ne has quit IRC15:57
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery  https://review.openstack.org/45971015:59
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guideline describing a cloud profile document  https://review.openstack.org/45986915:59
openstackgerritMonty Taylor proposed openstack/api-wg master: Add a guideline about use of the service catalog  https://review.openstack.org/46065615:59
*** chlong_ has joined #openstack-sdks16:08
*** chlong_ has quit IRC16:14
openstackgerritChris Dent proposed openstack/api-wg master: Create a set of api interoperability guidelines  https://review.openstack.org/42184616:20
*** jpich has quit IRC16:30
*** gouthamr has quit IRC16:31
*** salv-orl_ has quit IRC16:36
*** gouthamr has joined #openstack-sdks16:40
*** ralonsoh has quit IRC16:51
openstackgerritBrian Curtin proposed openstack/python-openstacksdk master: Port metric v1 to resource2  https://review.openstack.org/46067316:51
openstackgerritMerged openstack/api-wg master: Define pagination guidelines  https://review.openstack.org/44671616:52
openstackgerritMerged openstack/api-wg master: Recommend the correct HTTP method for tags  https://review.openstack.org/45153616:52
*** _alastor_ has joined #openstack-sdks16:54
briancurtin_alastor_: https://bugs.launchpad.net/python-openstacksdk/+bug/1680446 we don’t fully support what you’re looking for right now. yet16:54
openstackLaunchpad bug 1680446 in OpenStack SDK "Adding a User in a Project is not Possible" [Critical,Confirmed] - Assigned to Brian Curtin (brian.curtin)16:54
_alastor_briancurtin: ok, looks like I'll probably just have to call out to the CLI for that functionality right now16:55
_alastor_briancurtin: also, based on what Dave is saying in that bug, I could get a Project object and use those helper methods to accomplish what I need16:58
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Fix Nova-net netowrk commands  https://review.openstack.org/46067917:00
elmikocdent: you doin the newsletter now?17:00
cdentyeah17:00
elmikogonna leave for lunch soon, but i'll wait around17:00
elmikocool17:00
cdentelmiko: don't worry17:00
cdentI'll read it closely17:00
elmikohaha17:01
cdentif you're not about, it'll be fine17:01
*** salv-orlando has joined #openstack-sdks17:01
elmikonah, it's all good. it's raining here anyways, so i am putting off a soggy walk17:01
cdentmordred: what are the forces that would cause delay from when a new api-wg is +W (and announced as merged above) and when it shows up on specs.openstack.org?17:04
cdentor rather: is that delay predictable?17:05
cdent(it's caught up now)17:10
elmikocdent: please attend, maybe we need "GET YO ASS THERE!"17:18
elmikotoo strong?17:18
* cdent is thinking of the children17:18
elmikohaha17:19
briancurtin_alastor_: yeah, i think so. it’s not the ideal interface, but it might do what you need for now. I’m in the middle of getting some other changes out, but we should be able to add what’s needed in that bug and what you’re doing fairly soon17:19
cdentelmiko: any thing to add?17:19
* elmiko double checks17:20
elmikocdent: lgtm17:20
elmikosecond para is a little passive-aggressive, but i think it's worth it17:20
cdentyeah, I waffled on that and then decided "why not?"17:21
elmikoi'm with you, we need to shake the tree if we are going to get some fruit in this case17:21
cdentshipped17:22
mordredcdent: the delay is mostly just that publication jobs run just like any other one, so it's gotta get queued up, wait for a node to become available, then get built and published17:22
cdentelmiko: while you lunch, I will dine17:22
elmikocdent: enjoy =)17:22
cdentmordred: ah, I hadn't really considered that the publish jobs would get a new node. makes sense now17:23
* cdent locates dinner17:24
*** cdent has quit IRC17:24
*** amotoki has quit IRC17:32
gouthamredleafe: Re: '<edleafe> Heat will have to find a new liaison, <edleafe> Manilla too'17:48
gouthamredleafe: manila had two liaisons, now it's one. however, more the merrier, i can ask for more people to help out17:49
*** Kevin_Zheng has quit IRC17:57
*** erlon has joined #openstack-sdks17:57
*** Matias has joined #openstack-sdks18:05
edleafegouthamr: Ah, I misunderstood the email. Thanks for the clarification!18:25
edleafegouthamr: And we always welcome more participation18:26
gouthamredleafe: yep. i'll bring this up in the manila meeting soon :)18:27
*** salv-orlando has quit IRC18:27
*** e0ne has joined #openstack-sdks18:29
*** Daviey_ has joined #openstack-sdks18:38
*** Daviey has quit IRC18:38
*** annegentle has quit IRC18:38
*** dhellmann has quit IRC18:38
*** dhellmann has joined #openstack-sdks18:39
*** openstackgerrit has quit IRC18:48
*** annegentle has joined #openstack-sdks18:57
dtroyerShall we do an OSC meeting?  #openstack-meetin now-ish18:59
*** cdent has joined #openstack-sdks19:01
mordreddtroyer: so - on the skip validating front, I've got this: https://review.openstack.org/#/c/438196/ - which TOTALLY skips that stuff until it's requested19:18
mordreddtroyer: last we chatted about it you liked the idea but thought it was too soon / too much stuff still to unwind19:19
mordreddtroyer: do you think its time to start tugging at that thread again? or still hold off for a bit?19:19
dtroyerI was hoping the stuff jamielennox started that we talked about in ATL was going to handle that and I haven't gotten back to it yet19:20
* dtroyer is re-looking19:20
dtroyerso I forgot about that, and given what I've just unwound in OSC in prep for 1.27.0 I may be ready to go there now19:22
mordredoh cool19:22
mordredI'll get those two patches cleaned up and resubmitted for you19:22
mordredand we can see how far off they are19:22
dtroyercool.  I want to get the next OSC release out early next week, will dive in deeper after that19:23
* dtroyer is still cleaning up functional tests for nova-net19:23
mordredsomething just broke shade's legacy functional test jobs19:23
mordredwhich is the next thing _I_ get to look in to19:24
*** salv-orlando has joined #openstack-sdks19:30
*** john-davidge has joined #openstack-sdks19:36
*** prg3 has quit IRC19:37
*** prg3 has joined #openstack-sdks19:39
*** john-davidge has quit IRC19:40
*** e0ne has quit IRC20:03
*** e0ne has joined #openstack-sdks20:10
*** cdent has quit IRC20:19
*** openstackgerrit has joined #openstack-sdks20:24
openstackgerritBrian Curtin proposed openstack/python-openstacksdk master: Port unversioned Version resources to resource2  https://review.openstack.org/46073520:24
*** jkilpatr has quit IRC20:29
*** e0ne has quit IRC20:34
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Fix Nova-net netowrk commands  https://review.openstack.org/46067920:36
*** salv-orlando has quit IRC20:39
*** salv-orlando has joined #openstack-sdks20:39
*** jkilpatr has joined #openstack-sdks20:43
*** efried has joined #openstack-sdks20:54
efriedo/20:55
efriedmordred Gonna be multitasking the nova mtg in 5, FYI.20:55
mordredefried: me too - I'm still in this glorious phone call20:57
*** jkilpatr has quit IRC20:57
efriedmordred Cool, I'll just dump my questions and we can discuss as thread manager allows.20:57
mordredsweet20:58
efriedSo I noodled around with os-client-config a little bit.  I found that I could get all the known ClientConfig instances - so first question is: how do I figure out which one to use?  Or to know which one to get_one_cloud() for?20:59
efried(or is that the wrong starting point entirely?)20:59
efriedSecond, I found I couldn't do anything service_catalog-y with a CloudConfig unless I also supplied an auth somehow.  I can set up the auth (by loading from conf options), but it's not clear me where/how I'm supposed to hand it off to the CloudConfig.21:00
mordredefried: gotcha. good questions - SO - part of the issue is that the interfaces you're talking about in occ are oriented towards end-users21:12
mordredefried: I put this together: https://review.openstack.org/#/c/439120/21:13
mordredefried: which needs to be finished21:13
mordredefried: but the idea there woudl be to give you some nice factory functions that you could use to pass in your CONF object and the other context you have at the calling points in nova21:14
mordredand have it return you either a ksa.adapter or a glanceclient.Client or a neutronclient.Client or whatever is appropriate21:14
* dtroyer looks warily at those talking of using o-c-c in services21:22
efrieddtroyer If interested, here's where it's coming from: https://review.openstack.org/#/c/458257/21:23
dtroyerefried: thanks.  my concerns are much more about policy and expectations than code itself, learned from experience.  there's a reason I don't want most oslo bits in clients, they make server-side assumptions21:25
mordreddtroyer: oh yah - I definitely agree about oslo and clients21:26
efrieddtroyer Mainly the issue that prompted this discussion was that we want to be able to divine an appropriate list of service_type values to try successively to discover the endpoint for a given conf group, if they don't set a service_type in their conf.21:27
mordreddtroyer: in this case my suggestion was coming basically from seeing the amount of boilerplate (that's getting it wrong in several places) inside of nova when they're creating their clients to talk to the other services21:27
efriedExactly, the point of this blueprint.21:27
mordreddtroyer: the _basic_ thought was to make few functions that skip any processing of envvars or clouds.yaml files and only work via python-level parameters21:28
dtroyersure, and maybe some of that needs to be extracted.  my worry is the constant tension between client and service versions and dependencies and expectations.  I once thought the SDK should be so damn good that all of the services should use it too.  bad idea.  really bad idea in the real world21:28
mordreddtroyer: yup. totally agree with the concern21:29
dtroyermordred: that requires a discipline we have yet to be able to maintain within a lib.21:30
efriedBaby steps, we can start by putting the boilerplate in one place (and hopefully getting it right); and then later offload more of that functionality to o-c-c (and/or service-types-authority) as appropriate.21:30
dtroyerI'm onboard with the idea, just don't think o-c-c is the place fro shared client/service code21:30
mordredto be fair - I think with the ksa patch efried wrote, we might be able to just deal with it all with some repeated patterns21:30
mordredefried: ++21:30
mordredyah. I think the register_adapter_options in ksa is key21:30
dtroyerksa is a good example of us actually pulling this off.  it'll take that level of discipline for dependencies and interfaces21:31
mordredand then using session.get_endpoint21:31
efriedmordred With the exception of the "guess service_types for your group if not specified" thing, I have everything I need with the ksa patch.21:31
efriedThat part is still the hole.21:31
mordredbecause there's a few of the logic things nova needs that are in occ now but I think we may be close to being able to say that ksa should understand21:31
efriedI guess, back to baby steps, I can make service_type required for now...?21:32
*** zigo has joined #openstack-sdks21:32
efriedI don't think that'll make jaypipes happy, tho21:32
mordredoh - yes - service_type is/should be required21:32
mordredoh - wait21:32
mordredhang on - context switching21:32
efried...in the conf21:32
mordredI think we can put the thing you're talking about in like you have it now in the nova patches21:32
efriedokay, just a hardcoded dict of group: service_types in nova.utils?21:33
mordredI _eventually_ think we should move that little bit to ksa - but we need a broader buy-in on that21:33
mordredyah21:33
efriedOkay, cool.21:33
efriedSo then the remaining gap is fleshing that guy out.21:33
mordredit's basically a duplicate of code we have in occ too - but it's very small code  - so we can improve the duplication over time21:33
efriedI don't begin to know what the comprehensive mapping should be there.  Also didn't see it (except for some funky cinder stuff) in occ.  Can you point me to it?21:34
mordredthat's pretty much it21:34
mordredit's the funky cinder stuff21:34
mordredthere's also a mapping for mistral - but you don't need to worry about that one21:34
efried(I gather than "funky cinder stuff" is often redundant)21:34
mordredall the rest of them have one and only one service-type21:34
mordreddtroyer: you saw they're talking about a new service-typefor cinder right?21:35
efriedwould that be block-storage?21:35
mordredyah21:35
dtroyermordred: I recall that from long ago, didn't know it was active again.21:35
dtroyerI'm hoping it'll never gain a version?21:36
efrieddtroyer https://review.openstack.org/#/c/436178/21:36
efriedActive today21:36
*** bobh has quit IRC21:42
mordreddtroyer: I did make a new patch to the consume-discovery patch covering the sad pile of multiple types21:44
mordreddtroyer: https://review.openstack.org/#/c/460654/21:44
*** salv-orl_ has joined #openstack-sdks21:50
*** salv-orlando has quit IRC21:52
*** annegentle has quit IRC22:00
*** annegentle has joined #openstack-sdks22:01
*** salv-orl_ has quit IRC22:01
*** annegentle has quit IRC22:06
jamielennoxmordred, efried: sorry was in a meeting, yea, it's probably too much to make every client subclass something from keystoneauth22:06
jamielennoxthough the idea of a base useful client has come up a few times before22:06
jamielennoxmostly i was just trying to say i'm fine to have the pure adapter loading stuff because i know mordred wants that for os-c-c and shade22:07
jamielennoxbut for most uses we just need a way for novaclient to say that it passes all these options through from Client.__init__ to keystoneauth22:07
*** erlon has quit IRC22:13
*** peterstac has quit IRC22:14
*** nikhil has quit IRC22:17
*** purplerbot has quit IRC22:19
mordredjamielennox: yup - totally agree22:19
mordredjamielennox: an I _think_ that's the intent here22:20
*** heyongli has quit IRC22:20
*** heyongli has joined #openstack-sdks22:21
*** prg3 has quit IRC22:25
*** nikhil has joined #openstack-sdks22:26
mordredjamielennox: btw - you are probably one of the only other people on planet who can actually review this: https://review.openstack.org/#/c/459405/ for accuracy or intent22:28
*** prg3 has joined #openstack-sdks22:28
mordredjamielennox: so I'd love your feedback - although for the love of all that is holy don't start reading it until you're in a frame of mind that wants to bepunched in the face repeatedly for quite a while22:28
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 1  https://review.openstack.org/46035622:29
jamielennoxmordred: ok, that's not in my current pre-coffee state22:29
mordrednope22:29
mordrednot even close :)22:29
*** madorn has joined #openstack-sdks22:31
mordredjamielennox: specific things of concern are that it doesn't describe user-visible behavior changes from that ksa is doing now, and that if we added support to ksa for the additional things described we could do so without introducing behavior changes (although might possibly introduce an additional api call somewhere that the user wouldn't notice)22:31
*** salv-orlando has joined #openstack-sdks22:37
*** peterstac has joined #openstack-sdks22:37
*** jkilpatr has joined #openstack-sdks22:53
*** jkilpatr has quit IRC22:53
*** jkilpatr has joined #openstack-sdks22:54
*** adriant_ has joined #openstack-sdks23:07
*** salv-orlando has quit IRC23:08
*** sdague has quit IRC23:08
*** Kevin_Zheng has joined #openstack-sdks23:13
*** hongbin has quit IRC23:29
*** markvoelker has quit IRC23:40
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 1  https://review.openstack.org/46035623:41
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 2  https://review.openstack.org/46077723:41

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