Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-04-12
21:41:10 mordred edleafe: that'sa great idea
23:24:21 jamielennox mordred: with the additional shift why do you want it there?
23:24:37 jamielennox mordred: not that anyone else uses this, but my thought would be you would want to see like
23:24:52 jamielennox nova/A os-c-c/B glanceclient/C
23:25:03 jamielennox because os-c-c is like creating the glanceclient
23:28:48 jamielennox mordred: yea, i'd like to have a bit more discussion on what you're trying to do there
#openstack-sdks - 2017-04-13
00:00:36 mordred jamielennox: ah- I see what you mean - so I'll try to make this extra complex ...
00:01:40 mordred but first - I mostly have stopped caring about python-*client - but let's keep them for now because they make the discussion more interesting (harder)
00:02:57 mordred so - for the shade case, we have nodepool uses shade which uses os-client-config which uses keystoneauth which uses requests (at least that's how I think of it in my head)
00:04:11 mordred then for novaclient, it would be nodepool -> shade -> os-client-config -> novaclient -> keystoneauth ... or is it?
00:04:34 mordred maybe for the novaclient/glanceclient path the occ part is not meaningful
00:04:46 jamielennox sounds right
00:04:52 mordred in fact, I wonder if occ is important in the user agent at all?
00:05:14 jamielennox i think this is what dtroyer came to as well, telling people osc-lib version wasn't that important - but it's there if you want it
00:05:15 mordred it's setting up objects, but it's not really involved (atm) in making actual calls
00:05:20 mordred nod
00:09:05 mordred jamielennox: so maybe focusing on interior ordering isn't important - nodepool "the app" being first in the list seems important, and ksa/requests being at the end seems right
00:09:21 mordred other than that, I can make all sorts of arguments about why one sequence makes more sense than the other
00:09:28 mordred but I'm not sure any of them actually make sense :)
00:09:56 jamielennox mordred: yea, it took us a while to get to the layout we have because of these sorts of things - you can get really picky about it
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

Earlier   Later