| *** dhellmann has joined #openstack-sdks | 00:07 | |
| *** bobh has joined #openstack-sdks | 01:12 | |
| *** bobh has quit IRC | 01:28 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | 01:45 |
|---|---|---|
| *** mriedem has quit IRC | 01:50 | |
| openstackgerrit | wangqi proposed openstack/python-openstackclient master: Trivial: Fix the pep8 warning https://review.openstack.org/616395 | 02:44 |
| *** markvoelker has joined #openstack-sdks | 02:45 | |
| *** bobh has joined #openstack-sdks | 02:49 | |
| *** markvoelker has quit IRC | 02:50 | |
| *** markvoelker has joined #openstack-sdks | 02:52 | |
| *** markvoelker has quit IRC | 02:52 | |
| *** markvoelker has joined #openstack-sdks | 02:55 | |
| *** fungi has quit IRC | 03:06 | |
| *** mrhillsman has joined #openstack-sdks | 03:19 | |
| *** bobh has quit IRC | 03:55 | |
| *** bobh has joined #openstack-sdks | 04:08 | |
| *** bobh has quit IRC | 04:10 | |
| openstackgerrit | Merged openstack/python-openstackclient master: image/v2: support multiple property filters https://review.openstack.org/615850 | 05:40 |
| openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Modify the help message for 'registered limit set' https://review.openstack.org/609985 | 06:21 |
| *** slaweq has joined #openstack-sdks | 07:39 | |
| *** gtema has joined #openstack-sdks | 07:49 | |
| *** Luzi has joined #openstack-sdks | 08:30 | |
| *** jpena|off is now known as jpena | 08:36 | |
| *** noama has joined #openstack-sdks | 08:38 | |
| *** jpich has joined #openstack-sdks | 09:14 | |
| *** e0ne has joined #openstack-sdks | 09:50 | |
| *** ttsiouts has joined #openstack-sdks | 09:54 | |
| *** dtantsur|afk is now known as dtantsur | 09:55 | |
| *** ttsiouts has quit IRC | 09:56 | |
| *** ttsiouts has joined #openstack-sdks | 09:57 | |
| *** ttsiouts_ has joined #openstack-sdks | 09:59 | |
| *** ttsiouts has quit IRC | 09:59 | |
| *** ttsiouts_ has quit IRC | 10:19 | |
| *** ttsiouts has joined #openstack-sdks | 10:20 | |
| *** ttsiouts has quit IRC | 10:24 | |
| *** ttsiouts has joined #openstack-sdks | 10:34 | |
| openstackgerrit | melissaml proposed openstack-infra/shade master: fix the word 'bandwith' https://review.openstack.org/616513 | 11:12 |
| *** ttsiouts has quit IRC | 11:35 | |
| *** ttsiouts has joined #openstack-sdks | 11:35 | |
| *** ttsiouts has quit IRC | 11:38 | |
| *** ttsiouts_ has joined #openstack-sdks | 11:38 | |
| *** jpena is now known as jpena|lunch | 12:03 | |
| *** Luzi has quit IRC | 12:05 | |
| *** ttsiouts_ has quit IRC | 12:14 | |
| *** ttsiouts has joined #openstack-sdks | 12:14 | |
| *** ttsiouts has quit IRC | 12:19 | |
| *** Luzi has joined #openstack-sdks | 12:20 | |
| *** dtantsur is now known as dtantsur|brb | 12:32 | |
| *** gtema has quit IRC | 12:49 | |
| *** ttsiouts has joined #openstack-sdks | 12:52 | |
| *** ttsiouts has quit IRC | 12:56 | |
| *** ttsiouts has joined #openstack-sdks | 12:57 | |
| mordred | mnaser: you are amazing | 12:57 |
| *** e0ne has quit IRC | 13:03 | |
| *** jpena|lunch is now known as jpena | 13:24 | |
| *** e0ne has joined #openstack-sdks | 13:37 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Support remote vendor profiles https://review.openstack.org/616228 | 13:55 |
| *** jpich has quit IRC | 14:02 | |
| *** irclogbot_2 has quit IRC | 14:02 | |
| *** tonyb has quit IRC | 14:02 | |
| *** dmellado has quit IRC | 14:02 | |
| *** eandersson has quit IRC | 14:02 | |
| *** elmiko has quit IRC | 14:02 | |
| *** tobiash has quit IRC | 14:02 | |
| *** dmellado has joined #openstack-sdks | 14:04 | |
| *** jpich has joined #openstack-sdks | 14:04 | |
| *** christi3k has quit IRC | 14:04 | |
| *** tobiash has joined #openstack-sdks | 14:05 | |
| *** tonyb has joined #openstack-sdks | 14:07 | |
| *** christi3k has joined #openstack-sdks | 14:07 | |
| *** mriedem has joined #openstack-sdks | 14:13 | |
| *** bobh has joined #openstack-sdks | 14:16 | |
| *** ttsiouts has quit IRC | 14:16 | |
| *** ttsiouts has joined #openstack-sdks | 14:17 | |
| *** ttsiouts has quit IRC | 14:19 | |
| *** ttsiouts has joined #openstack-sdks | 14:20 | |
| samueldmq | morning | 14:26 |
| samueldmq | is it correcto to say "At the service level, OpenStack has native support for API level interoperability, certifying clouds via the RefStack program." | 14:27 |
| samueldmq | this allows heterogeneous clouds running the same service API versions to fully interoperate | 14:29 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | 14:29 |
| samueldmq | and then it comes to sdk providing an upper abstraction layer for supporting multiple service (nova-net vs neutron) and versions of the smae service (keystone v2 and v3) | 14:29 |
| mordred | samueldmq: it's ... the optimistic way of saying it, yes | 14:35 |
| samueldmq | mordred: but that's what refstack does, correct? | 14:36 |
| samueldmq | you mean optimistic because I am putting so much of the glory in sdk I guess | 14:37 |
| mordred | samueldmq: no - I mean optimistic about RefStack :) | 14:50 |
| samueldmq | mordred: oh | 14:50 |
| mordred | samueldmq: but I think what you're saying is the intent - and my pessimistic side just sees the bugs | 14:51 |
| samueldmq | what does it do in practice? just run tempest tests against clouds and reports results? | 14:51 |
| samueldmq | mordred: bugs.. for example? I need to know more downsides of it | 14:52 |
| mordred | samueldmq: so you could say all of the above, but then also that sdk's abstraciton layer does what you say it does - and also works around bugs/holes in the interoperability that refstack aims to provide | 14:52 |
| mordred | samueldmq: well - places where refstack doesn't have a story for something yet - or where people can't agree that a given feature should be covered by refstack certification | 14:52 |
| mordred | samueldmq: at the sdk level we can provide a consistency quickly - while aligning different operating clouds might take a bit longer | 14:53 |
| samueldmq | hmmm | 14:54 |
| samueldmq | so refstack's intent was to standardize? | 14:54 |
| samueldmq | is* | 14:54 |
| samueldmq | mordred: that's very interesting... standardizing things that are already alive don't work well. literature and experience shows that | 14:55 |
| samueldmq | also - standards translates into cutting some use cases, and flexibility :-) | 14:56 |
| mordred | samueldmq: yes - that's right - and you're exactly right | 14:58 |
| mordred | samueldmq: refstack started well after there were already many production deployments | 14:59 |
| mordred | so it has to take a conservative approach | 14:59 |
| samueldmq | oh well, that's a good story to be written... thanks!! | 15:00 |
| mordred | sdk also started after many production deployments - so it was able to look at what reality was, and then provide an abstraction on top of that to hide differences that exist due to refstack not existing in the beginning | 15:00 |
| mordred | samueldmq: \o/ | 15:00 |
| samueldmq | \o/ | 15:00 |
| *** dtantsur|brb is now known as dtantsur | 15:02 | |
| *** ttsiouts has quit IRC | 15:15 | |
| *** ttsiouts has joined #openstack-sdks | 15:15 | |
| *** ttsiouts has quit IRC | 15:19 | |
| *** ttsiouts has joined #openstack-sdks | 15:22 | |
| *** ttsiouts has quit IRC | 15:46 | |
| *** ttsiouts has joined #openstack-sdks | 15:47 | |
| *** ttsiouts has quit IRC | 15:49 | |
| *** ttsiouts has joined #openstack-sdks | 15:49 | |
| *** elmiko has joined #openstack-sdks | 15:51 | |
| elmiko | edleafe: i will be around here, but i have a call to listen in on at the same time. just a heads up | 15:51 |
| * dtantsur ditto | 15:54 | |
| edleafe | elmiko: dtantsur: no problem | 15:55 |
| edleafe | I'll try to handle the crowds all by myself :) | 15:55 |
| * mordred crowds edleafe | 15:55 | |
| * edleafe reaches for his mace | 15:55 | |
| * mordred equips face mask | 15:56 | |
| edleafe | The API-SIG Office Hour has officially begun. | 16:00 |
| elmiko | lol | 16:01 |
| edleafe | Well, we don't use #startmeeting, so I gotta write something :) | 16:02 |
| elmiko | #putsfeetondesk | 16:02 |
| elmiko | and i wasn't lol'ing at your announcement (i like it), i was laughing at the exchange between you and mordred =) | 16:03 |
| edleafe | That mordred thinks he's so tough | 16:04 |
| elmiko | indeed | 16:05 |
| *** noama has quit IRC | 16:06 | |
| *** ttsiouts has quit IRC | 16:07 | |
| *** ttsiouts has joined #openstack-sdks | 16:08 | |
| *** Luzi has quit IRC | 16:22 | |
| openstackgerrit | Ed Leafe proposed openstack/api-sig master: Add idempotent DELETE guidance https://review.openstack.org/616610 | 16:36 |
| edleafe | elmiko: dtantsur: finally got around to writing this ^^ | 16:36 |
| elmiko | \o/ | 16:37 |
| dtantsur | w00t | 16:38 |
| *** slaweq has quit IRC | 16:40 | |
| *** e0ne has quit IRC | 16:48 | |
| *** slaweq has joined #openstack-sdks | 16:58 | |
| edleafe | wow, another busy office hour | 17:01 |
| elmiko | lol | 17:02 |
| elmiko | hey, you posted a PR! | 17:02 |
| elmiko | edleafe: just a heads up, i won't be around next week. gonna be at a conference | 17:03 |
| elmiko | although, isn't next week the forum too? | 17:04 |
| edleafe | elmiko: Unfortunately, next week I *won't* be at a conference | 17:04 |
| elmiko | =( | 17:04 |
| * edleafe grumbles about IBM bean counters | 17:04 | |
| elmiko | ++ | 17:04 |
| elmiko | just running around all willy-nilly dropping 34B here and there XD | 17:05 |
| edleafe | They just wanted to get you working for them | 17:05 |
| elmiko | lol | 17:05 |
| *** ttsiouts has quit IRC | 17:16 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add project param in LimitList parser https://review.openstack.org/611789 | 17:20 |
| *** Leo_m has joined #openstack-sdks | 17:27 | |
| *** jpich has quit IRC | 17:34 | |
| *** gtmanfred is now known as yaml | 17:36 | |
| *** yaml is now known as gtmanfred | 17:36 | |
| *** jpena is now known as jpena|off | 17:51 | |
| *** irclogbot_2 has joined #openstack-sdks | 18:01 | |
| mordred | Shrews: incidentally, in writing tests for the statsd patch, I actually discovered a bug that was lurking in the existing code | 18:07 |
| mordred | Shrews: also - I have tests now | 18:07 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | 18:10 |
| mordred | Shrews, tobiash: ^^ it still needs docs added ... and tobiash, I'd love feedback on the metric names and labels (obviously it's not immediately useful if we don't decided to expose that directly in nodepool - but since you're using things already, I figureyou've got good context) | 18:12 |
| tobiash | mordred: what is the openstack_http_requests supposed to be? | 18:27 |
| mordred | tobiash: the equiv of the task counter | 18:27 |
| mordred | tobiash: so what you're translating to nodepool_task now | 18:28 |
| tobiash | mordred: then I think it should be a counter, not a gauge | 18:28 |
| mordred | tobiash: ok - the counter docs said something about resetting on restarts - was I just understanding that wrong? | 18:28 |
| mordred | tobiash: ah - I see! (found better docs) | 18:29 |
| mordred | and I agre | 18:29 |
| tobiash | mordred: https://prometheus.io/docs/concepts/metric_types/ | 18:29 |
| mordred | and good - because I find spelling gauge very hard | 18:29 |
| *** bobh has quit IRC | 18:29 | |
| tobiash | that's the reference of the prometheus metric types | 18:30 |
| tobiash | mordred: in statsd this is probably a counter too? | 18:30 |
| mordred | tobiash: yah | 18:31 |
| 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:31 |
| mordred | yah - makes sense | 18:32 |
| tobiash | with counters instead you can do things like calculating rates per any time interval | 18:32 |
| tobiash | mordred: I think using a histogram however is a good choice | 18:34 |
| tobiash | for the timers | 18:34 |
| mordred | woot! | 18:35 |
| tobiash | especially if you want to aggregate and calculate quantiles of more than one series you *need* historgrams instead of summaries | 18:36 |
| tobiash | some background info on that: https://prometheus.io/docs/practices/histograms/ | 18:36 |
| tobiash | it's really interesting | 18:37 |
| 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:38 |
| Shrews | that probably makes less sense in sdk | 18:39 |
| Shrews | where there can be lots of them | 18:39 |
| Shrews | bah, silly thinking | 18:39 |
| Shrews | mordred: can we please not add both statsd and the prometheus stuff in a single review? | 18:41 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | 18:44 |
| mordred | tobiash: updated to counter | 18:45 |
| mordred | Shrews: sure - once I get the docs written, I'll split it up | 18:45 |
| Shrews | mordred: actually, it's not that overly burdonsome here | 18:45 |
| mordred | *phew | 18:45 |
| 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:46 |
| 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 |
| 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:47 |
| Shrews | mordred: weird | 18:50 |
| tobiash | mordred: regarding openstack_http_requests I'd even add the response code as a label | 19:04 |
| kmalloc | mordred: so.. about to land some pymemcache awesomeness to oslo.cache | 19:06 |
| kmalloc | mordred: erm dogpile.cache | 19:06 |
| kmalloc | :) | 19:06 |
| openstackgerrit | Merged openstack/python-openstackclient master: Add --name-lookup-one-by-one option to server list https://review.openstack.org/568345 | 19:07 |
| openstackgerrit | Merged openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | 19:11 |
| *** mriedem has quit IRC | 19:32 | |
| *** dave-mccowan has joined #openstack-sdks | 19:56 | |
| 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? | 19:58 |
| *** mriedem has joined #openstack-sdks | 19:59 | |
| 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 |
| kmalloc | mgagne: it's to handle the case of passing a name instead of an id and having it magically work | 20:07 |
| mgagne | kmalloc: yes, if a UUID like name is found, would it be reasonable to not perform a full list? | 20:07 |
| kmalloc | so something like --id <uuid> might need to be implemented so it doesn't do an id then name | 20:07 |
| mgagne | or are people crazy to name their ports with UUID? | 20:07 |
| kmalloc | yes | 20:07 |
| kmalloc | people name everything with uuids because operators often dynamically generate that stuff | 20:08 |
| kmalloc | i see it all the time with projects/users | 20:08 |
| kmalloc | ID is UUID X, name is UUID Y | 20:08 |
| mgagne | so you have port.id != port.name but both are UUIDs? crazy stuff | 20:08 |
| kmalloc | i wouldn't be surprised in ports or servers or anything | 20:08 |
| kmalloc | welcome to insanity | 20:09 |
| kmalloc | it's unfortuantely the norm | 20:09 |
| mgagne | can we have OS_INSANE=0 then? :D | 20:09 |
| * kmalloc would rather just do a --id and --name explicit flags | 20:09 | |
| kmalloc | and throw a warning "hey, the get_name_or_id functionality is going away" | 20:10 |
| kmalloc | but i don't have a lot of say in that part | 20:10 |
| mgagne | ok, at least I got an explanation | 20:10 |
| kmalloc | that is a guess fwiw | 20:11 |
| kmalloc | i didn't look at the code | 20:11 |
| kmalloc | but it sure sounds like what we do elsewhere | 20:11 |
| 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:12 |
| kmalloc | yep, so basically the same concept though | 20:13 |
| mgagne | something* | 20:16 |
| *** e0ne has joined #openstack-sdks | 20:19 | |
| *** e0ne has quit IRC | 20:22 | |
| *** dave-mccowan has quit IRC | 20:33 | |
| dtroyer | melwitt: do you want to backport the add floating ip fix past rocky? | 20:40 |
| dtroyer | I was preparing a set of stable releases, if so I'll wait for that… | 20:41 |
| melwitt | dtroyer: yes, I was hoping for queens (where my downstream bug was reported). ah, ok | 20:41 |
| melwitt | sure | 20:41 |
| *** mriedem has quit IRC | 20:41 | |
| dtroyer | go ahead and propose that then | 20:42 |
| melwitt | k | 20:42 |
| melwitt | available at https://review.openstack.org/616690 | 20:45 |
| *** mriedem has joined #openstack-sdks | 20:45 | |
| melwitt | dtroyer ^ | 20:55 |
| *** mriedem has left #openstack-sdks | 21:18 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | 21:45 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Collect request stats https://review.openstack.org/614834 | 22:03 |
| mordred | mgagne: fwiw - sdk by default ALWAYS does a list and filters for name or id client-side :) | 22:16 |
| *** mriedem has joined #openstack-sdks | 22:36 | |
| *** mriedem has quit IRC | 22:46 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710 | 22:52 |
| *** Leo_m has quit IRC | 23:14 | |
| *** slaweq has quit IRC | 23:15 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Add project param in LimitList parser https://review.openstack.org/611789 | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!