Thursday, 2017-05-25

*** hongbin has quit IRC00:10
*** bobh has joined #openstack-sdks00:12
*** reedip_ has joined #openstack-sdks00:38
*** bobh has quit IRC00:39
*** hoangcx has joined #openstack-sdks00:53
*** bobh has joined #openstack-sdks01:00
*** bobh has quit IRC01:37
*** annp has joined #openstack-sdks01:44
*** openstackgerrit has joined #openstack-sdks02:09
openstackgerritHong Hui Xiao proposed openstack/python-openstackclient master: Don't show hint about vlan transparent in network set  https://review.openstack.org/46600202:09
*** reedip_ has quit IRC02:30
*** prg3 has quit IRC02:51
*** prg3 has joined #openstack-sdks02:55
*** bobh has joined #openstack-sdks03:10
*** yuanying has joined #openstack-sdks03:24
*** bobh has quit IRC03:33
*** gouthamr has quit IRC03:57
*** markvoelker has joined #openstack-sdks04:13
*** prg3 has quit IRC04:45
openstackgerritReedip proposed openstack/python-openstackclient master: Add default-quota to subnet pool commands  https://review.openstack.org/43736704:52
*** prg3 has joined #openstack-sdks05:00
*** markvoelker_ has joined #openstack-sdks05:11
*** markvoelker has quit IRC05:14
*** markvoelker has joined #openstack-sdks05:17
*** markvoel_ has joined #openstack-sdks05:19
*** markvoelker has quit IRC05:19
*** markvoelker_ has quit IRC05:21
*** prg3 has quit IRC05:38
*** prg3 has joined #openstack-sdks05:42
*** markvoel_ has quit IRC05:53
*** markvoelker has joined #openstack-sdks05:53
*** markvoelker has quit IRC05:54
*** Dinesh_Bhor has quit IRC06:06
*** Dinesh_Bhor has joined #openstack-sdks06:11
*** prg3 has quit IRC06:14
*** markvoelker has joined #openstack-sdks06:27
*** markvoelker has quit IRC06:32
*** ralonsoh has joined #openstack-sdks06:55
*** hoangcx_ has joined #openstack-sdks06:55
*** hoangcx has quit IRC06:57
*** hoangcx_ is now known as hoangcx06:58
*** markvoelker has joined #openstack-sdks06:59
*** markvoelker has quit IRC07:03
*** markvoelker has joined #openstack-sdks07:09
*** markvoelker has quit IRC07:14
*** Serlex has joined #openstack-sdks07:25
*** markvoelker has joined #openstack-sdks07:29
*** jpich has joined #openstack-sdks07:35
*** markvoelker has quit IRC07:37
*** dtantsur|afk is now known as dtantsur07:38
*** markvoelker has joined #openstack-sdks07:42
*** markvoelker has quit IRC07:49
*** hoangcx_ has joined #openstack-sdks07:59
*** hoangcx has quit IRC08:02
*** hoangcx_ is now known as hoangcx08:08
*** sheel has joined #openstack-sdks08:13
*** gildub has quit IRC08:14
*** markvoelker has joined #openstack-sdks08:24
*** markvoelker has quit IRC08:30
*** markvoelker has joined #openstack-sdks08:32
*** hoangcx_ has joined #openstack-sdks08:32
*** hoangcx has quit IRC08:34
*** markvoelker has quit IRC08:40
openstackgerritRui Chen proposed openstack/python-openstackclient master: Rework server migrate to support cold and live  https://review.openstack.org/46005908:46
*** e0ne has joined #openstack-sdks08:56
*** markvoelker has joined #openstack-sdks09:16
*** markvoelker has quit IRC09:21
*** hoangcx has joined #openstack-sdks09:25
*** hoangcx_ has quit IRC09:26
*** cdent has joined #openstack-sdks09:46
*** sdague has joined #openstack-sdks09:46
*** markvoelker has joined #openstack-sdks09:54
*** adriant has quit IRC09:58
*** markvoelker has quit IRC09:58
*** sfinucan has quit IRC10:19
*** sfinucan has joined #openstack-sdks10:19
*** hoangcx has quit IRC10:21
*** sheel has quit IRC10:22
*** annp has quit IRC10:27
openstackgerritSean Dague proposed openstack/keystoneauth master: Add global_request_id param to Adapter constructor  https://review.openstack.org/46799110:43
sdaguejamielennox: if you happen to still be awake ^^^^ would be great to have eyes on10:44
*** markvoelker has joined #openstack-sdks10:47
*** markvoelker has quit IRC10:52
*** markvoelker has joined #openstack-sdks10:55
*** jkilpatr has quit IRC10:57
*** markvoelker has quit IRC10:59
*** jkilpatr has joined #openstack-sdks11:14
*** markvoelker has joined #openstack-sdks11:14
*** markvoelker has quit IRC11:19
*** markvoelker has joined #openstack-sdks11:29
*** markvoelker has quit IRC11:34
jamielennoxsdague: that spec looks remarkably like what request id was supposed to be before it got all committeed11:35
sdaguejamielennox: yeh, well what is old is new again?11:40
jamielennoxit's nice to see some sanity being restored11:41
sdagueso, all that previous work predates me, but we seemed to get agreement at the summit, so I'm running at it as fast as I can while we have that11:41
sdaguehaving the 2 ids does end up being useful, so you can distinguish the fact that there are a lot of cross calls some times11:42
sdaguethe oslo side changes are all landed and released, so now it's just plumbing clients and callers11:42
jamielennoxsdague: commented, code feels odd, i don't mind which way it's fixed11:44
jamielennoxsdague: it does feel weird to be setting this on the Adapter, adapters are created per client so you're not going to share that request_id across a lot of connections that way11:44
jamielennoxthough with sessions lasting a long time there's not really a good spot to put a request level middleware in currently11:45
sdaguejamielennox: ok, I'm not sure I quite understand the concerns there11:47
sdaguebecause we don't want to be sharing request_id more broadly, the important thing is that it's relevant to the flow right now11:48
jamielennoxnote the way i always hoped this would work is that auth_token middleware currently passes down an auth plugin. that auth plugin when used to contact another service does all the correct things like uses a service token and presents the right catalog and stuff11:49
jamielennoxand would be able to pass through a request-id that it received to the auth plugin and auto end up in other requests11:49
jamielennoxbut it was super slow work getting people to adopt that and it didn't get very far11:49
jamielennoxsdague: yea, if it's useful there then that's fine. Re code, i just mean to store the variable in one place. I don't mind if it's a property around additional_headers or if you manually add it to headers later like we do for user_agent, just not stored twice11:51
sdagueok11:52
*** markvoelker has joined #openstack-sdks12:00
*** jkilpatr has quit IRC12:00
*** jkilpatr has joined #openstack-sdks12:01
*** chenyb4_ has joined #openstack-sdks12:02
*** markvoelker has quit IRC12:04
sdaguejamielennox: for reference - https://review.openstack.org/#/c/467242/ is what the nova change looks like with some passing on to clients12:09
sdaguehttps://review.openstack.org/#/c/467417/ is it all glued together, you'll see some global_ids logged in cinder / neutron with that12:10
jamielennoxsdague: yea, you can get a fair way with additional_headers now, but i'm happy to make it a real thing12:10
*** markvoelker has joined #openstack-sdks12:11
sdaguejamielennox: yeh, I've been just direct plumbing into neutron/cinder/glance clients, but the neutron folks wanted it in keystonauth. I'm kind of fine either way, because it's pretty much just a single field to pop out on the constructor if it's totally at the top level12:12
sdagueeven though we're setting global_request_id in constructors quite often in time, in code, it's typically consolidated to a single location in the caller12:12
*** markvoelker has quit IRC12:15
*** jkilpatr has quit IRC12:16
*** markvoelker has joined #openstack-sdks12:17
*** markvoelker has quit IRC12:22
openstackgerritMatthew Booth proposed openstack/os-client-config master: Don't pop from os.environ  https://review.openstack.org/46802012:27
*** jkilpatr has joined #openstack-sdks12:43
openstackgerritSean Dague proposed openstack/keystoneauth master: Add global_request_id param to Adapter constructor  https://review.openstack.org/46799112:47
*** bobh has joined #openstack-sdks12:48
openstackgerritRui Chen proposed openstack/python-openstackclient master: Rework server migrate to support cold and live  https://review.openstack.org/46005912:53
*** fguillot has joined #openstack-sdks12:55
*** markvoelker has joined #openstack-sdks12:56
*** prg3 has joined #openstack-sdks13:00
*** gouthamr has joined #openstack-sdks13:02
*** chlong has quit IRC13:06
*** chenyb4_ has quit IRC13:19
openstackgerritMonty Taylor proposed openstack/os-client-config master: Don't pop from os.environ  https://review.openstack.org/46802013:31
mordredShrews, dtroyer: if you get time, I'd really like to get the top five patches from https://review.openstack.org/#/q/project:openstack/os-client-config+status:open+label:Workflow=0 landed and a new occ out - they're all small/easy13:32
mordredShrews, dtroyer: but there are a few behavior bugs that are biting people13:32
*** markvoelker has quit IRC13:33
Shrewsmordred: k13:38
*** fzdarsky has joined #openstack-sdks13:39
*** chenyb4_ has joined #openstack-sdks13:41
Shrewsmordred: is https://review.openstack.org/465195 still relevant with https://review.openstack.org/468020 ?13:42
*** chenyb4_ has quit IRC13:42
*** chenyb4_ has joined #openstack-sdks13:42
mordredShrews: the reason for writing it isn't - but it's still a good idea I think - the OpenStackConfig object is really intended to be a singleton13:42
*** reedip_ has joined #openstack-sdks13:57
*** fzdarsky has quit IRC14:09
*** chenyb4_ has quit IRC14:12
*** chenyb4_ has joined #openstack-sdks14:13
*** markvoelker has joined #openstack-sdks14:15
*** chenyb4_ has quit IRC14:17
*** fzdarsky has joined #openstack-sdks14:19
*** markvoelker has quit IRC14:23
*** chlong has joined #openstack-sdks14:27
*** dtantsur is now known as dtantsur|brb14:36
*** hongbin has joined #openstack-sdks14:45
*** dave-mccowan has joined #openstack-sdks15:00
*** markvoelker has joined #openstack-sdks15:01
*** marst_ has joined #openstack-sdks15:05
*** Serlex has quit IRC15:07
openstackgerritReedip proposed openstack/python-openstackclient master: Introduce quota unset command  https://review.openstack.org/37631115:19
openstackgerritReedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset'  https://review.openstack.org/35626315:34
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery  https://review.openstack.org/45971015:36
openstackgerritMonty Taylor proposed openstack/api-wg master: Add document describing consuming version discovery  https://review.openstack.org/45940515:36
openstackgerritMonty Taylor proposed openstack/api-wg master: Describe the publication of service-types-authority data  https://review.openstack.org/46281515:36
mordredcdent, edleafe, elmiko: I believe I've gotten all current reviews taken care of15:37
cdentyay!15:37
edleafemordred: on my list15:38
elmikomordred: sweet, i'm rounding the corner on my last lap for the reviews you posted15:46
mordredelmiko: woot!15:47
elmikoalso kudos on consuming-version-discovery.rst, this thing is a beast!15:48
*** dtantsur|brb is now known as dtantsur15:49
*** e0ne has quit IRC15:51
mordredelmiko: it really is, isn't it?15:52
mordredelmiko: the scary part is that somehow I actually have all of that stored in my head15:52
elmikomordred: that is scary...15:53
elmikomordred: i really like your enumeration of the algorithms though, i think it will help readers understand this better15:53
mordredelmiko: thanks! ... I certainly hope so15:54
mordredthe fun part is that next I need to write a spec for keystone about implementing this in keystoneauth :)15:55
elmikogood luck sir15:56
openstackgerritYuval Shalev proposed openstack/python-openstacksdk master: Added server console output method  https://review.openstack.org/46807815:58
*** jpich has quit IRC16:17
*** reedip_ has quit IRC16:19
*** cdent has quit IRC16:22
*** ralonsoh has quit IRC17:01
*** dtantsur is now known as dtantsur|afk17:03
*** prg3 has quit IRC17:16
elmikoedleafe: draft up, https://etherpad.openstack.org/p/api-wg-newsletter17:32
elmikoi realize you may be out, i'll wait 15-20 then post it17:32
*** fzdarsky has quit IRC17:51
*** prg3 has joined #openstack-sdks18:02
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Add flavor value options  https://review.openstack.org/30279518:08
*** prg3 has quit IRC18:13
*** dgonzalez has quit IRC18:22
*** dgonzalez has joined #openstack-sdks18:22
*** fzdarsky has joined #openstack-sdks18:24
*** fzdarsky has quit IRC18:29
*** e0ne has joined #openstack-sdks19:01
*** e0ne has quit IRC19:17
*** e0ne has joined #openstack-sdks19:27
openstackgerritMerged openstack/python-openstackclient master: Convert volume functional tests into JSON format  https://review.openstack.org/46520819:30
openstackgerritMonty Taylor proposed openstack/api-wg master: Add support for historical service type aliases  https://review.openstack.org/46065419:31
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guidelines on Version Discovery  https://review.openstack.org/45971019:31
openstackgerritMonty Taylor proposed openstack/api-wg master: Add document describing consuming version discovery  https://review.openstack.org/45940519:31
openstackgerritMonty Taylor proposed openstack/api-wg master: Describe the publication of service-types-authority data  https://review.openstack.org/46281519:31
openstackgerritMonty Taylor proposed openstack/api-wg master: Add guideline about consuming endpoints from catalog  https://review.openstack.org/46281419:31
*** cdent has joined #openstack-sdks19:32
*** bobh has quit IRC19:50
*** annegentle has joined #openstack-sdks19:53
openstackgerritMerged openstack/python-openstackclient master: Add functional test for volume service  https://review.openstack.org/43228219:53
*** jamielennox is now known as jamielennox|away19:56
*** amotoki is now known as amotoki_away19:58
*** amotoki_away is now known as amotoki19:59
*** amotoki is now known as amotoki_away19:59
*** fguillot has quit IRC20:00
*** cdent has quit IRC20:02
*** prg3 has joined #openstack-sdks20:08
*** chlong has quit IRC20:15
*** prg3 has quit IRC20:19
dhellmanndtroyer : in ATL we talked about a daemon mode for cliff/OSC. My notes say "add an option to emit a null value after the formatter output". Is that really all we need there?20:20
*** prg3 has joined #openstack-sdks20:28
*** jkilpatr has quit IRC20:29
*** chlong has joined #openstack-sdks20:33
*** chlong has quit IRC20:37
*** chlong has joined #openstack-sdks20:38
*** prg3 has quit IRC20:40
*** markvoelker has quit IRC20:42
*** prg3 has joined #openstack-sdks20:45
*** jkilpatr has joined #openstack-sdks20:49
dtroyerdhellmann: that is the minimum needed to delimit return data for each command.  I didn't want to invent a protocol for this so hopefully that is all we would need20:50
dhellmanndtroyer : ok20:50
dhellmanndo we want a --daemon-mode that implies --interactive and also adds the null? or just a --null-delimited-output option?20:50
dtroyerI was only thinking about the —null option as interactive is already implied when no command is given20:52
dhellmannmakes sense20:52
dhellmannjust --null then?20:52
dhellmannnot --null-delimited-output?20:52
dtroyerhmmm, it may need to be more specific, to be honest I have been thinking about cribbing the xargs -0 option, not sure if we need to be able to specify the delimiter char20:53
dhellmannoh, -0 works20:53
dhellmannok, the other thing we talked about was "command extensions"20:53
dhellmannI'm looking at that right now20:54
dtroyerthe semantics are not identical to xargs though so maybe that connection is not quite as useful20:54
dhellmannI can either load them from a separate namespace for each use (args, pre, & post hooks)20:54
dhellmannor I can load them from one namespace and expect a class with a specific API for each of those cases20:54
dhellmannany preference?20:54
dhellmannI'll make --null-delimited-output and -0 aliases20:55
dtroyerI've imagined it as a single class with essentially hook methods to override, but I don't have a huge preference yet20:55
dtroyerthe alias sounds good20:55
dhellmannok, I was leaning toward a class, too20:55
dtroyerhaving a single namespace does feel cleaner20:56
dhellmannI was going to make the namespace the application namespace + cmd_name.replace(' ', '_')20:56
dhellmannthen plugins can have their own names20:56
*** markvoelker has joined #openstack-sdks20:56
dtroyersounds good20:56
dhellmannk20:56
*** markvoelker has quit IRC21:00
*** e0ne has quit IRC21:05
*** fzdarsky has joined #openstack-sdks21:05
*** chlong has quit IRC21:11
*** sdague has quit IRC21:24
*** marst_ has quit IRC21:30
*** dave-mccowan has quit IRC21:33
*** fzdarsky has quit IRC21:39
dtroyerdhellmann: if you're still around… with the major rev of cliff on the near horizon, do you want to wait for that release for this stuff to land ^^^^21:42
*** bobh has joined #openstack-sdks21:50
*** bobh has quit IRC21:55
*** markvoelker has joined #openstack-sdks22:05
*** annegentle has quit IRC22:06
*** markvoelker has quit IRC22:15
*** markvoelker has joined #openstack-sdks22:16
*** prg3 has quit IRC22:20
*** gouthamr has quit IRC22:29
*** jamielennox|away is now known as jamielennox22:36
openstackgerritMerged openstack/os-client-config master: Add helper method to fetch service catalog  https://review.openstack.org/46423822:55
openstackgerritMerged openstack/os-client-config master: Add ironicclient to constructors list  https://review.openstack.org/46423922:55
openstackgerritMerged openstack/os-client-config master: Keep a singleton to support multiple get_config calls  https://review.openstack.org/46519522:55
openstackgerritMerged openstack/os-client-config master: Don't pop from os.environ  https://review.openstack.org/46802022:55
openstackgerritDoug Hellmann proposed openstack/cliff master: add cmd_name argument to CompleteCommand  https://review.openstack.org/46819523:05
openstackgerritDoug Hellmann proposed openstack/cliff master: pass the command name from HelpCommand  https://review.openstack.org/46819623:05
openstackgerritDoug Hellmann proposed openstack/cliff master: add hook for manipulating the argument parser  https://review.openstack.org/46819723:05
openstackgerritDoug Hellmann proposed openstack/cliff master: add hook for get_epilog  https://review.openstack.org/46819823:05
openstackgerritDoug Hellmann proposed openstack/cliff master: add before and after hooks  https://review.openstack.org/46819923:05
*** gouthamr has joined #openstack-sdks23:05
dhellmanndtroyer : let's see how closely this matches up with what you were thinking ^^23:05
openstackgerritDoug Hellmann proposed openstack/cliff master: add hook for get_epilog  https://review.openstack.org/46819823:10
openstackgerritDoug Hellmann proposed openstack/cliff master: add before and after hooks  https://review.openstack.org/46819923:10
openstackgerritMerged openstack/os-client-config master: Revert "Revert "Use interface not endpoint_type for keystoneclient""  https://review.openstack.org/46429923:10
*** markvoelker has quit IRC23:11
*** gildub has joined #openstack-sdks23:30
*** markvoelker has joined #openstack-sdks23:44
*** markvoelker has quit IRC23:45
*** hongbin has quit IRC23:58

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