| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-08 | |||
| 18:28:59 | mordred | tobiash: ok - the counter docs said something about resetting on restarts - was I just understanding that wrong? | |
| 18:29:32 | mordred | tobiash: ah - I see! (found better docs) | |
| 18:29:34 | mordred | and I agre | |
| 18:29:52 | mordred | and good - because I find spelling gauge very hard | |
| 18:29:52 | tobiash | mordred: https://prometheus.io/docs/concepts/metric_types/ | |
| 18:30:09 | tobiash | that's the reference of the prometheus metric types | |
| 18:30:21 | tobiash | mordred: in statsd this is probably a counter too? | |
| 18:31:37 | mordred | tobiash: yah | |
| 18:31:44 | tobiash | gauge is not very useful for aggregating stuff, it should only be used if you are interested in the current value of something like current quota usage | |
| 18:32:07 | mordred | yah - makes sense | |
| 18:32:16 | tobiash | with counters instead you can do things like calculating rates per any time interval | |
| 18:34:39 | tobiash | mordred: I think using a histogram however is a good choice | |
| 18:34:48 | tobiash | for the timers | |
| 18:35:16 | mordred | woot! | |
| 18:36:26 | tobiash | especially if you want to aggregate and calculate quantiles of more than one series you *need* historgrams instead of summaries | |
| 18:36:50 | tobiash | some background info on that: https://prometheus.io/docs/practices/histograms/ | |
| 18:37:00 | tobiash | it's really interesting | |
| 18:38:01 | Shrews | ya know, i was thinking about stats stuff (since it seems to be all the rage this week)... we could just let folks configure the statsd keys themselves via config file (at least in nodepool). | |
| 18:39:08 | Shrews | that probably makes less sense in sdk | |
| 18:39:33 | Shrews | where there can be lots of them | |
| 18:39:46 | Shrews | bah, silly thinking | |
| 18:41:25 | Shrews | mordred: can we please not add both statsd and the prometheus stuff in a single review? | |
| 18:44:49 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | |
| 18:45:19 | mordred | tobiash: updated to counter | |
| 18:45:37 | mordred | Shrews: sure - once I get the docs written, I'll split it up | |
| 18:45:53 | Shrews | mordred: actually, it's not that overly burdonsome here | |
| 18:45:58 | mordred | *phew | |
| 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 | :) | |
| 19:06:32 | kmalloc | mordred: erm dogpile.cache | |
| 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: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | |
| 14:32:50 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | |
| 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! | |