Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-04-13
00:10:10 mordred ++
00:10:12 jamielennox realistically the best thing to do would be to have the python clients actually insert their name into the sequence
00:10:43 mordred yah. although hopefully 'soon' I won't be using them for anything anymore and that can be someone else's problem
00:12:17 mordred jamielennox: for the other patch - the main thing was that without adding client_name to session and only on adater, we lose it when ksa makes discovery calls on behalf of the user
00:12:38 mordred so I figured just setting it on session would be eaiser than plumbing client_name to wherever it needed to be for discovery calls
00:12:46 mordred but again, it's possible I'm making it too difficult :)
00:12:52 jamielennox oh, because discovery is internal to ksa?
00:13:30 jamielennox i mean we can plumb it through - but maybe it doesn't matter
00:13:36 mordred yah - and ksa doens't use an adapter for it
00:13:42 jamielennox things done internal to ksa don't care what invoked the request
00:13:57 jamielennox i mean it would be useful to know what client started the call
00:14:42 jamielennox mordred: i don't mean to argue this too much, but it's two new params in an already complicated list for fairly small benefit
00:14:42 mordred right - but when I think of user-agents I think of ops looking at logs to figure out why a particuar traffic pattern came in - and if it was shade that asked ksa for something, I feel like I'd be missing part of the story?
00:14:51 mordred _but_.... I don't have an actual use case
00:15:16 mordred jamielennox: totally understand - I appreciate you curating the library well :)
00:15:40 jamielennox mordred: is shade creating the session?
00:16:09 jamielennox oh right, this is why shade is out of order with the python-*client in additional
00:16:21 mordred yah
00:16:50 mordred actually - shade does not create the session - occ creates the session for it
00:18:51 mordred in general, it 'should' be occ creating session, adapater and/or python-client - but shade directly instantiates Adapter because it wraps it in a wrapper class
00:19:44 mordred and I gave up on giving occ a "set_adapter_factory" feature because that was just too much :)
00:23:40 jamielennox so i mean shade could add itself to additional at that point
00:25:55 mordred jamielennox: hrm. maybe that's not a bad idea
04:08:25 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: Make block-device-mapping more stable and clear https://review.openstack.org/438141
06:19:00 openstackgerrit Rui Chen proposed openstack/cliff master: Add smart help formatter for command parser https://review.openstack.org/454613
07:39:34 openstackgerrit Reedip proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
07:39:46 reedip RuiChen : Done ^^
11:21:21 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
12:07:19 openstackgerrit Merged openstack/python-openstacksdk master: Add support for volume attachments in compute v2 https://review.openstack.org/455335
12:28:05 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk master: Add is_profile_only to Cluster resource https://review.openstack.org/455211
13:15:21 elmiko edleafe: heads up, good chance i will be a few minutes late to the meeting today
13:15:50 edleafe elmiko: ok, we'll gossip about you until then
13:16:53 elmiko haha, i would expect nothing less ;)
13:43:16 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk master: Add docs for volume_attachment compute methods https://review.openstack.org/456620
14:27:58 openstackgerrit Merged openstack/python-openstacksdk master: Add docs for volume_attachment compute methods https://review.openstack.org/456620
14:49:43 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk master: Add is_profile_only to Cluster resource https://review.openstack.org/455211
14:50:27 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk master: Enable warnings_as_errors in doc enforcer https://review.openstack.org/453676
15:19:32 openstackgerrit Merged openstack/python-openstacksdk master: Enable warnings_as_errors in doc enforcer https://review.openstack.org/453676
15:39:44 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Add ability to pass in user_agent https://review.openstack.org/452550
15:40:02 mordred1 jamielennox|away: ^^ I think that is a better version of that patch
18:58:26 rabel osc-meeting today? dtroyer_zz?
19:06:45 rabel dtroyer_zz: so probably no meeting today. ;) i will go then. could you please have a look at https://review.openstack.org/#/c/444924/ ? it's ready for merging, i think. current patch set is from last friday. and also if you could have a look at https://review.openstack.org/#/c/449707/ , it would be nice. just to say if it is ok to deprecate the command like that. i think the gnocchi developers are fine with it.
19:06:58 rabel dtroyer_zz: thanks and see you around
23:28:26 openstackgerrit ayoung proposed openstack/python-openstackclient master: Implied Roles https://review.openstack.org/290253
#openstack-sdks - 2017-04-14
03:17:08 openstackgerrit Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
06:10:50 openstackgerrit Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585
06:37:13 openstackgerrit Reedip proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
06:50:17 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263
06:50:19 openstackgerrit Reedip proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
08:28:36 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet-pools https://review.openstack.org/437367
08:44:39 openstackgerrit Reedip proposed openstack/osc-lib master: Add MultiKeyValueCommaAction to osc-lib https://review.openstack.org/425515
09:15:54 openstackgerrit Rui Chen proposed openstack/cliff master: Add smart help formatter for command parser https://review.openstack.org/454613
09:53:59 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: Make block-device-mapping more stable and clear https://review.openstack.org/438141
#openstack-sdks - 2017-04-15
05:46:17 reedip_ o/
05:46:51 openstackgerrit Reedip proposed openstack/python-openstackclient master: Add default-quota to subnet-pools https://review.openstack.org/437367
06:26:04 openstackgerrit Reedip proposed openstack/python-openstackclient master: Remove workarounds for 0.9.12 SDK version https://review.openstack.org/439299
#openstack-sdks - 2017-04-17
00:35:11 chatter29 hey guys
00:35:14 chatter29 allah is doing
00:35:20 chatter29 sun is not doing allah is doing
00:35:22 chatter29 to accept Islam say that i bear witness that there is no deity worthy of worship except Allah and Muhammad peace be upon him is his slave and messenger
02:07:04 yuanbin briancurtin, this patch has finish, please help review https://review.openstack.org/#/c/455211/
05:33:20 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
05:37:58 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
06:31:35 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Fix nova create_server availability_zone body parameter error https://review.openstack.org/457154
08:06:30 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
12:25:35 openstackgerrit Merged openstack/python-openstacksdk master: Add is_profile_only to Cluster resource https://review.openstack.org/455211
14:20:06 briancurtin Qiming, stevemar, people of OSC, people of the world, etc: openstacksdk 0.9.15 was released: https://pypi.python.org/pypi/openstacksdk/0.9.15
14:20:40 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
14:57:04 dtroyer_zz briancurtin: thanks for the heads-up
15:18:20 openstackgerrit Merged openstack/python-openstackclient master: Allow override of distributed router flag https://review.openstack.org/433442
16:16:01 openstackgerrit Merged openstack/cliff master: Add 'autoprogram-cliff' Sphinx directive https://review.openstack.org/450322
16:17:37 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/455972
17:11:45 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/455906
17:35:34 openstackgerrit Merged openstack/python-openstackclient master: Low-level Compute v2 API: security group https://review.openstack.org/454310
17:44:26 openstackgerrit Merged openstack/python-openstackclient master: Update to tox.ini https://review.openstack.org/454455
19:40:34 openstackgerrit Merged openstack/python-openstackclient master: Low-level Compute v2 API: security group rules https://review.openstack.org/454939
20:13:43 elmiko edleafe: ping
20:45:00 edleafe elmiko: pong
20:45:49 elmiko edleafe: a meeting got bumped on my schedule and it's unclear whether i'll make the start of the meeting this thursday, any chance you could run it?
20:46:09 edleafe elmiko: sure, I can do it
20:46:34 elmiko edleafe: sweet, thanks! i will hopefully be there, but i can't get a firm answer on how long this meeting will last, you know the drill...
21:14:06 openstackgerrit Merged openstack/python-openstackclient master: Low-level Compute v2 API: floating ip https://review.openstack.org/455555
21:20:07 mordred jamielennox: if you get a few, eyes on https://review.openstack.org/#/c/452550/ when you have a sec
21:20:24 mordred jamielennox: I think that fits in better with how we talked about it
21:31:08 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Low-level Compute v2 API: floating ip pool https://review.openstack.org/457368
21:44:14 jamielennox mordred: commented
21:44:21 mordred jamielennox: thanks!
21:44:34 jamielennox mordred: i would drop the client_name bit, i think it's confusing when you have to pass it to the config object
21:44:44 jamielennox the additional part will take care of that anyway
21:44:59 jamielennox just stick with app_name and app_version
21:46:13 mordred jamielennox: replied - it was actually to expose it to users if they wanted to set it to something else themselves
21:46:24 mordred but I'm happy to remove it and just leave it as app_name
21:47:11 jamielennox mordred: well once you remove all the *clients os-c-c is the client, additional just gives you somewhere to put it until then
21:47:29 jamielennox in the adapter i don't consider client_X something that a user would set
21:47:38 mordred cool
22:13:34 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Split network tests https://review.openstack.org/457385
#openstack-sdks - 2017-04-18
00:11:07 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Low-level Compute v2 API: network https://review.openstack.org/457402
00:49:01 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Low-level Compute v2 API: network https://review.openstack.org/457402
01:03:16 openstackgerrit yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429
01:06:29 openstackgerrit Merged openstack/python-openstackclient master: Split network tests https://review.openstack.org/457385

Earlier   Later