Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-11-08
18:46:54 mordred Shrews: I tink that test file should have all the things we need to do further tests - I put in things to add the statsd and prometheus content to details on failure - cause otherwise it's like "1 doesn't match None" and you're like WTF???
18:47:24 mordred Shrews: that said - the stastd fixture starts a thread - and when I added it just to the base test case I wound up hitting too many open files :)
18:47:58 mordred which - might be a thing we want to look in to in general, as that tells me something isn't properly getting shut down - but it only shows up when you've got a bazillion tests like we do in sdk
18:50:28 Shrews mordred: weird
19:04:58 tobiash mordred: regarding openstack_http_requests I'd even add the response code as a label
19:06:27 kmalloc mordred: so.. about to land some pymemcache awesomeness to oslo.cache
19:06:32 kmalloc mordred: erm dogpile.cache
19:06:32 kmalloc :)
19:07:03 openstackgerrit Merged openstack/python-openstackclient master: Add --name-lookup-one-by-one option to server list https://review.openstack.org/568345
19:11:07 openstackgerrit Merged openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910
19:58:18 mgagne I'm currently using python-openstackclient and performing "openstack port show <UUID>" on a non-existing port. I found with --debug that if GET /v2.0/ports/<UUID> returns a 404, GET /v2.0/ports is performed instead which can be *very* costly if you happen to have a LOT of ports or are an admin. Is there anyway to improve that?
20:07:01 kmalloc mgagne: it would require a flag or something to not perform a list to "find" a port. this is a behavior for "get name or id" type thing that we typically do
20:07:17 kmalloc mgagne: it's to handle the case of passing a name instead of an id and having it magically work
20:07:42 mgagne kmalloc: yes, if a UUID like name is found, would it be reasonable to not perform a full list?
20:07:54 kmalloc so something like --id <uuid> might need to be implemented so it doesn't do an id then name
20:07:55 mgagne or are people crazy to name their ports with UUID?
20:07:59 kmalloc yes
20:08:20 kmalloc people name everything with uuids because operators often dynamically generate that stuff
20:08:30 kmalloc i see it all the time with projects/users
20:08:41 kmalloc ID is UUID X, name is UUID Y
20:08:55 mgagne so you have port.id != port.name but both are UUIDs? crazy stuff
20:08:57 kmalloc i wouldn't be surprised in ports or servers or anything
20:09:06 kmalloc welcome to insanity
20:09:16 kmalloc it's unfortuantely the norm
20:09:21 mgagne can we have OS_INSANE=0 then? :D
20:10:05 kmalloc and throw a warning "hey, the get_name_or_id functionality is going away"
20:10:16 kmalloc but i don't have a lot of say in that part
20:10:51 mgagne ok, at least I got an explanation
20:11:40 kmalloc that is a guess fwiw
20:11:43 kmalloc i didn't look at the code
20:11:49 kmalloc but it sure sounds like what we do elsewhere
20:12:56 mgagne yea... I'm sure it's nothing like that because it uses find_port which I guess support both id and name at the same time.
20:13:15 kmalloc yep, so basically the same concept though
20:16:33 mgagne something*
20:40:38 dtroyer melwitt: do you want to backport the add floating ip fix past rocky?
20:41:08 dtroyer I was preparing a set of stable releases, if so I'll wait for that…
20:41:32 melwitt dtroyer: yes, I was hoping for queens (where my downstream bug was reported). ah, ok
20:41:38 melwitt sure
20:42:43 dtroyer go ahead and propose that then
20:42:58 melwitt k
20:45:40 melwitt available at https://review.openstack.org/616690
20:55:07 melwitt dtroyer ^
21:45:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834
22:03:12 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834
22:16:37 mordred mgagne: fwiw - sdk by default ALWAYS does a list and filters for name or id client-side :)
22:52:52 openstackgerrit Slawek Kaplonski proposed openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710
23:27:05 openstackgerrit Merged openstack/python-openstackclient master: Add project param in LimitList parser https://review.openstack.org/611789
#openstack-sdks - 2018-11-09
00:16:26 openstackgerrit Ian Wienand proposed openstack/keystoneauth master: Fair semaphore fixes https://review.openstack.org/616717
00:27:38 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043
00:46:40 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926
01:53:20 openstackgerrit Merged openstack/openstacksdk master: Add a __main__ handler, version command https://review.openstack.org/616070
02:24:10 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Updated the take_actions for unified limits https://review.openstack.org/612283
02:46:00 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Modify the help message for 'registered limit set' https://review.openstack.org/609985
02:47:16 openstackgerrit liuyamin proposed openstack/python-openstackclient master: Remove .testr.conf configuration https://review.openstack.org/616733
09:23:18 openstackgerrit liuyamin proposed openstack/python-openstackclient master: Remove .testr.conf configuration https://review.openstack.org/616733
13:34:10 martin_85 hi, is there any way how to get nova cpu_allocation_ratio python-openstackclient or python-novaclient ?
13:34:31 martin_85 hi, is there any way how to get nova cpu_allocation_ratio through python-openstackclient or python-novaclient ?
13:53:56 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add wait functions to orchestration proxy https://review.openstack.org/615702
14:32:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834
14:32:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926
14:46:40 openstackgerrit Ed Leafe proposed openstack/api-sig master: Add idempotent DELETE guidance https://review.openstack.org/616610
14:46:59 edleafe elmiko: ^^ improved the wording a bit
15:05:36 openstackgerrit Merged openstack-infra/shade master: Add python 3.6 unit test job https://review.openstack.org/615783
15:21:19 mordred edleafe, elmiko: I agree with that guidance. that is, fwiw, already the behavior shade exposes to users
15:23:14 edleafe mordred: I was reading _A Philosophy of Software Design_ by John Ousterhout, and he mentioned something along those lines, and I felt that we should state it explicitly.
15:24:18 mordred ++
16:26:10 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix neutron endpoint mangling https://review.openstack.org/616989
16:27:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix neutron endpoint mangling https://review.openstack.org/616989
16:27:35 mordred slaweq: ^^
16:37:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834
16:37:32 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926
16:41:52 slaweq mordred: I will check that today evening
16:42:23 mordred slaweq: thanks!
19:43:02 tobiash mordred: just curious, is there any official api in openstack to get lbaas io counters as non-admin?
20:54:15 kmalloc ianw: i want to say your semaphore fix might have a minor race condition in it where it could force more-than-allowed concurrency (by ~1)
20:55:07 kmalloc ianw: it's not bad. just working through the logic. it's because of the count increment/decrement/trigger of execution. but... i don't think it's a huge issue.
20:56:12 kmalloc ianw: and really the race might be mordred's fault
20:56:13 kmalloc :P
21:14:07 kmalloc ianw, mordred: the fair semaphore is looking pretty good. I'll need to run through the tests some more and i think we cna improve the testing.
21:14:34 kmalloc I'd like to get a functional test of the rate limit happening as well through KSA not just synthetic [functional testing? etc]
21:15:11 kmalloc to be sure we don't need to re-work the mechanics. Remember KSA's contract is extremely strict, and once code lands, i am hesitant to change it.
22:35:46 mordred kmalloc: ++ to functional test. fwiw, we've got a nodepool devstack functional test - but I think that's lkely TOO functional for us to be able to properly exercise the rate limiting
22:48:02 openstackgerrit Merged openstack/openstacksdk master: Add wait functions to orchestration proxy https://review.openstack.org/615702
22:56:23 openstackgerrit Slawek Kaplonski proposed openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710
22:56:42 slaweq mordred: if You will have a minute, please take a look at ^^
23:20:27 openstackgerrit Daniel Speichert proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544544
23:21:15 kmalloc mordred: yah
23:36:15 ianw kmalloc: i'll take a look on my monday with fresh eyes with that in consideration ... sounds very plausible :)
23:37:01 kmalloc ianw: possible, but it's not a major concern, a functional test would be more important :)
23:50:25 openstackgerrit Merged openstack/openstacksdk master: Fix neutron endpoint mangling https://review.openstack.org/616989
23:51:16 openstackgerrit Slawek Kaplonski proposed openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710
#openstack-sdks - 2018-11-10
08:11:14 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Add volume backup import/export commands https://review.openstack.org/612735
08:22:31 openstackgerrit Slawek Kaplonski proposed openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710
20:27:36 openstackgerrit Merged openstack/python-openstackclient master: Add volume backup import/export commands https://review.openstack.org/612735
#openstack-sdks - 2018-11-11
04:38:00 openstackgerrit Merged openstack/openstackclient master: fix tox python3 overrides https://review.openstack.org/606685
13:46:42 openstackgerrit Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139
16:47:36 openstackgerrit melissaml proposed openstack-infra/shade master: change default python 3 env in tox to 3.5 https://review.openstack.org/617151
#openstack-sdks - 2018-11-12
01:24:48 openstackgerrit liuyamin proposed openstack/python-openstackclient master: Remove .testr.conf configuration https://review.openstack.org/616733
08:42:54 openstackgerrit Vieri proposed openstack/openstackclient master: removing older python version 3.4 from setup.cfg https://review.openstack.org/606361
10:47:15 openstackgerrit Andreas Jaeger proposed openstack-infra/shade master: Use openSUSE 15.0 for testing https://review.openstack.org/617276

Earlier   Later