| *** dayou_ has joined #openstack-sdks | 00:01 | |
| openstackgerrit | Ian Wienand proposed openstack/osc-lib master: Mask password when specified as an argument with a space https://review.openstack.org/603528 | 00:34 |
|---|---|---|
| *** slaweq has joined #openstack-sdks | 01:11 | |
| *** slaweq has quit IRC | 01:15 | |
| *** chenyb4 has joined #openstack-sdks | 01:35 | |
| openstackgerrit | Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | 02:08 |
| openstackgerrit | Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | 02:12 |
| *** slaweq has joined #openstack-sdks | 03:11 | |
| *** slaweq has quit IRC | 03:16 | |
| openstackgerrit | Ian Wienand proposed openstack/osc-lib master: Mask password when specified as an argument with a space https://review.openstack.org/603528 | 03:34 |
| *** bobh has quit IRC | 04:31 | |
| *** bobh has joined #openstack-sdks | 04:37 | |
| *** bobh has quit IRC | 04:42 | |
| *** bobh has joined #openstack-sdks | 04:51 | |
| *** bobh has quit IRC | 04:55 | |
| *** slaweq has joined #openstack-sdks | 05:11 | |
| *** slaweq has quit IRC | 05:15 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | 05:34 |
| *** chenyb4 has quit IRC | 05:54 | |
| AJaeger | dtantsur|afk, mordred, https://review.openstack.org/#/c/594393/ fails on queens - any ideas how to fix the test suite? That's the final python3-first change to import for SDK... | 06:03 |
| *** Luzi has joined #openstack-sdks | 06:16 | |
| *** slaweq has joined #openstack-sdks | 06:18 | |
| openstackgerrit | Andreas Jaeger proposed openstack/openstacksdk master: Update .zuul.yaml https://review.openstack.org/603563 | 06:20 |
| *** slaweq has quit IRC | 06:23 | |
| *** deepak_mourya_ has quit IRC | 06:40 | |
| *** slaweq has joined #openstack-sdks | 06:55 | |
| *** slaweq has quit IRC | 07:00 | |
| *** olivierb has joined #openstack-sdks | 07:09 | |
| *** slaweq has joined #openstack-sdks | 07:15 | |
| *** gkadam has joined #openstack-sdks | 07:36 | |
| *** e0ne has joined #openstack-sdks | 07:36 | |
| *** tosky has joined #openstack-sdks | 07:37 | |
| *** e0ne has quit IRC | 07:48 | |
| *** gkadam has quit IRC | 07:57 | |
| *** jpich has joined #openstack-sdks | 08:08 | |
| openstackgerrit | Ian Wienand proposed openstack/openstacksdk master: Handle empty values in regions https://review.openstack.org/601485 | 08:22 |
| *** e0ne has joined #openstack-sdks | 08:25 | |
| *** e0ne has quit IRC | 08:26 | |
| *** e0ne has joined #openstack-sdks | 08:33 | |
| *** skatsaounis has quit IRC | 08:34 | |
| *** gtema has joined #openstack-sdks | 08:53 | |
| *** dtantsur|afk is now known as dtantsur | 08:57 | |
| dtantsur | AJaeger: ouch. mordred, shade CI seems to use new ansible with old shade/openstacksdk | 08:59 |
| dtantsur | so it actually does not exercise shade. need to cap ansible? | 08:59 |
| dtantsur | AJaeger: I'd wait for mordred, I'm not overly familiar with shade. But I suspect we may need to upper-cap ansible on all its ansible jobs. | 09:01 |
| *** e0ne has quit IRC | 09:02 | |
| AJaeger | dtantsur: thanks for digging into it, hope mordred has an idea. | 09:18 |
| *** markvoelker has quit IRC | 09:25 | |
| mordred | AJaeger, dtantsur: I just pushed up another rev on 594393 | 09:57 |
| dtantsur | wow, an early mordred | 09:57 |
| dtantsur | (or is it late?) | 09:57 |
| mordred | dtantsur: it's early - and for no reason I can determine :( | 09:57 |
| dtantsur | happens :( | 09:58 |
| mordred | but - maybe that means I'm going to be productive today | 09:58 |
| dtantsur | early wake up - and productive? gotta need a few litres of coffee :) | 09:58 |
| mordred | oh. yes. very much so | 10:03 |
| ml | I got some issues with the CI too, maybe you can help | 10:03 |
| AJaeger | mordred: if that job passes, we might need to merge the project-config change first to remove the "broken" ansible job to merge 594393. But since 594393 is the last one, this is fine... | 10:03 |
| ml | want to enable neutron-fwaas for my implementation over here: https://review.openstack.org/#/c/592303/ but didn't have any success yet | 10:03 |
| openstackgerrit | Merged openstack/openstacksdk master: Add support for configured NAT source variable https://review.openstack.org/602618 | 10:04 |
| *** gtema has quit IRC | 10:04 | |
| ml | trying to enable neutron-fwaas for the devstack checks | 10:05 |
| mordred | AJaeger: ++ | 10:06 |
| mordred | ml: looking | 10:07 |
| ml | thank you | 10:07 |
| mordred | ml: I added a couple of comments - I think we need to move those devstack/settings into the .zuul.yaml | 10:13 |
| ml | thanks. I will apply those changes right now | 10:16 |
| mordred | dtantsur: my god. my brain is bleeding reading the microversion doc change - how did I ever write the initial versions of that | 10:22 |
| dtantsur | LOL | 10:23 |
| dtantsur | mordred: I guess you were really, REALLY inspired at that moment | 10:24 |
| dtantsur | it took me quite some time to wrap my head around the flow | 10:24 |
| mordred | we shoudl not let me write docs, I think :) | 10:24 |
| dtantsur | nobody likes writing docs, so we probably will let you anyway :) | 10:25 |
| openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | 10:28 |
| ml | can you quickly confirm, that it's valid: https://review.openstack.org/#/c/592303/12..13/.zuul.yaml | 10:29 |
| *** markvoelker has joined #openstack-sdks | 10:30 | |
| mordred | ml: it's possible it just wants to be service_plugins: router,metering,qos,trunk,firewall_v2 - but I'm not 100% sure on that, so let's see how it goes | 10:35 |
| ml | noted. | 10:36 |
| *** pooja_jadhav has quit IRC | 10:40 | |
| *** e0ne has joined #openstack-sdks | 10:47 | |
| *** pooja_jadhav has joined #openstack-sdks | 10:56 | |
| *** pooja-jadhav has joined #openstack-sdks | 10:57 | |
| *** markvoelker has quit IRC | 11:00 | |
| *** imacdonn has quit IRC | 11:19 | |
| *** imacdonn has joined #openstack-sdks | 11:20 | |
| openstackgerrit | Ian Wienand proposed openstack/openstacksdk master: Handle empty values in regions https://review.openstack.org/601485 | 11:20 |
| *** cdent has joined #openstack-sdks | 11:25 | |
| *** markvoelker has joined #openstack-sdks | 11:57 | |
| openstackgerrit | Merged openstack/os-client-config master: fix bug link in readme https://review.openstack.org/597558 | 12:02 |
| *** e0ne has quit IRC | 12:20 | |
| *** gtema has joined #openstack-sdks | 12:27 | |
| *** markvoelker has quit IRC | 12:31 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Run all tasks through the threadpool https://review.openstack.org/603739 | 12:43 |
| AJaeger | mordred: could you see scrollback on #openstack-infra and help promoting a job, please? | 13:04 |
| openstackgerrit | Merged openstack/microversion-parse master: Copy zuul job settings from project-config https://review.openstack.org/599448 | 13:12 |
| openstackgerrit | Merged openstack/microversion-parse master: Add python36 jobs to microversion-parse https://review.openstack.org/599539 | 13:12 |
| *** gtema has quit IRC | 13:16 | |
| *** mriedem has joined #openstack-sdks | 13:21 | |
| *** bobh has joined #openstack-sdks | 13:46 | |
| *** bobh has quit IRC | 13:51 | |
| *** bobh has joined #openstack-sdks | 13:58 | |
| *** bobh has quit IRC | 14:02 | |
| *** bobh has joined #openstack-sdks | 14:08 | |
| *** bobh has quit IRC | 14:12 | |
| *** bobh has joined #openstack-sdks | 14:19 | |
| *** Luzi has quit IRC | 14:19 | |
| *** Leo_m has joined #openstack-sdks | 14:24 | |
| *** bobh has quit IRC | 14:27 | |
| *** bobh has joined #openstack-sdks | 14:32 | |
| *** gouthamr_ is now known as gouthamr | 14:35 | |
| *** bobh has quit IRC | 14:36 | |
| *** dayou_ has quit IRC | 14:40 | |
| *** bobh has joined #openstack-sdks | 14:40 | |
| openstackgerrit | Merged openstack/openstacksdk master: Listing objects to return Munch objects https://review.openstack.org/596780 | 14:42 |
| *** bobh has quit IRC | 14:45 | |
| *** bobh has joined #openstack-sdks | 14:48 | |
| *** bobh has quit IRC | 14:53 | |
| ml | do new implementations require the use of tenant_id or can I completely drop it in favor or project_id? | 14:54 |
| mordred | ml: please use project_id - I would prefer the word tenant showed up nowhere :) | 14:58 |
| ml | excellent, less work | 14:58 |
| *** bobh has joined #openstack-sdks | 14:59 | |
| *** bobh has quit IRC | 15:03 | |
| openstackgerrit | Merged openstack/openstacksdk master: baremetal: implement validate_node https://review.openstack.org/599533 | 15:03 |
| openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | 15:07 |
| *** bobh has joined #openstack-sdks | 15:10 | |
| *** bobh has quit IRC | 15:14 | |
| *** bobh has joined #openstack-sdks | 15:22 | |
| *** bobh has quit IRC | 15:27 | |
| *** dtantsur is now known as dtantsur|brb | 15:28 | |
| *** bobh has joined #openstack-sdks | 15:31 | |
| *** bobh has quit IRC | 15:36 | |
| *** bobh has joined #openstack-sdks | 15:37 | |
| *** bobh has quit IRC | 15:42 | |
| *** dtantsur|brb is now known as dtantsur | 15:46 | |
| *** bobh has joined #openstack-sdks | 15:47 | |
| *** bobh has quit IRC | 15:47 | |
| *** bobh has joined #openstack-sdks | 15:47 | |
| mordred | dtantsur: ok. I think I figure out the bifrost test failure | 15:56 |
| dtantsur | \o/ | 15:56 |
| dtantsur | so, what's it? | 15:57 |
| mordred | dtantsur: tl;dr - when noauth and endpoint are used, things are weird - so I've got some code adding a conditional for "did the user set overrides, if so just trust the user" | 15:57 |
| dtantsur | mordred: how does it play with our looooong text for api-sig? | 15:58 |
| dtantsur | I remember thinking really hard on a few bits around endpoint-override.. | 15:58 |
| mordred | I *think* it's correct - but we should definitely walk through it and make sure | 15:58 |
| mordred | also - I think I found a minor-ish bug in keystoneauth ... but maybe it's not one | 15:59 |
| mordred | kmalloc, cmurphy: in keystoneauth's discovery cache, we cache http://localhost:6385/ separately from http://localhost:6385 | 15:59 |
| mordred | so if someone puts in an endpoint_override with http://localhost:6385 and then the discovery document or catalog has http://localhost:6385/ - I *think* we're ignoring the cache and making a second call | 16:00 |
| kmalloc | *blink* | 16:00 |
| cmurphy | yay | 16:00 |
| * cmurphy wonders who wrote that discovery cache | 16:01 | |
| mordred | cmurphy: not me not me | 16:02 |
| dtantsur | Oo | 16:02 |
| kmalloc | mordred: so... Fwiw, the URL with the trailing slash is often different. With exception of the first slash | 16:02 |
| kmalloc | So sounds like we need an rstrip | 16:02 |
| kmalloc | mordred: it was either you or jamielennox I think. | 16:03 |
| kmalloc | mordred: :P | 16:03 |
| mordred | kmalloc: I'm going to blame jamielennox because he's not going to defend himself which is safer for me | 16:03 |
| kmalloc | Heheheh | 16:03 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | 16:03 |
| mordred | dtantsur, kmalloc, cmurphy: https://review.openstack.org/#/c/600292/5..6/openstack/service_description.py - wit that change, openstack.baremetal is the correct proxy object in a devstack | 16:04 |
| kmalloc | My eyes are bleeding. | 16:05 |
| mordred | they should be | 16:06 |
| mordred | that means you still have some amount of sanity | 16:06 |
| * dtantsur is worried about his sanity as this patch makes some sense to him | 16:06 | |
| kmalloc | Oh it's ok, I lost most of my sanity in the 10000 lines for flask | 16:06 |
| kmalloc | My eyes are only bleeding because of the number of files changed and the number of loc. | 16:07 |
| kmalloc | Not the content of the patch. | 16:07 |
| *** e0ne has joined #openstack-sdks | 16:07 | |
| dtantsur | mordred: will it work with an unversioned endpoint? | 16:08 |
| mordred | kmalloc: MOST of it is removing useless lines from tests | 16:08 |
| kmalloc | mordred: exactly | 16:08 |
| mordred | dtantsur: I believe so | 16:08 |
| mordred | dtantsur: well, I say that ... but actually no, I think it still won't | 16:09 |
| mordred | I think there is still one more "bug" preventing that from working | 16:09 |
| dtantsur | \o/ | 16:09 |
| mordred | which is - in most of the codepaths in keystoneauth if someone gives an endpoint_override we treat it as unassailable fact | 16:09 |
| mordred | that means we don't do discovery on it | 16:09 |
| mordred | since the user gave us the endpoint | 16:10 |
| dtantsur | aha, and this is something that I think I changed in the API SIG patch ;) | 16:10 |
| mordred | heh | 16:10 |
| dtantsur | because that means we're still stuck with versioned endpoints everywhere | 16:10 |
| dtantsur | aaaand microversion discovery also? | 16:10 |
| mordred | yeah- we need to figure out when it is the correct thing to do to do the discovery logic even with an endpoint override | 16:10 |
| dtantsur | man we need to dedicate a whole day in Berlin in November just to sit down and go through this mess | 16:11 |
| mordred | for microversion discovery it will do the fetch on the endpoint override | 16:11 |
| mordred | dtantsur: ++ | 16:11 |
| mordred | seriouslyu | 16:11 |
| mordred | because therea re like 12 competing use cases | 16:11 |
| dtantsur | yep | 16:11 |
| dtantsur | ideally, I'd prefer to endpoint-override to be unversioned (but support versioned ones too) | 16:12 |
| dtantsur | my head hurts just from thinking about it.. | 16:12 |
| * dtantsur remembers doing gross hacks for rust-openstack | 16:12 | |
| * dtantsur particularly loves https://github.com/dtantsur/rust-openstack/blob/master/src/common/protocol.rs#L128-L137 | 16:14 | |
| mordred | dtantsur: yeah. such awesomeness | 16:16 |
| dtantsur | the next two functions are also awesome: they handle the fact that some API use empty string instead of null :) | 16:16 |
| *** e0ne has quit IRC | 16:17 | |
| dtantsur | which is not an issue for Python but a big issue for a statically typed language :D | 16:17 |
| *** dtantsur is now known as dtantsur|afk | 16:23 | |
| *** olivierb has quit IRC | 16:26 | |
| *** e0ne has joined #openstack-sdks | 16:31 | |
| mordred | dtantsur|afk: ok. I think I have the versioned/unversioned thing handled | 16:32 |
| *** jpich has quit IRC | 16:32 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | 16:35 |
| *** e0ne has quit IRC | 16:35 | |
| *** bobh has quit IRC | 16:43 | |
| openstackgerrit | Merged openstack/openstacksdk master: Explicitly set logging levels for external libs https://review.openstack.org/602085 | 19:33 |
| *** cdent has quit IRC | 20:27 | |
| *** Leo_m_ has joined #openstack-sdks | 21:04 | |
| *** Leo_m has quit IRC | 21:06 | |
| *** Leo_m_ has quit IRC | 21:12 | |
| *** Leo_m has joined #openstack-sdks | 21:12 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Run all tasks through the threadpool https://review.openstack.org/603739 | 21:33 |
| *** Leo_m_ has joined #openstack-sdks | 21:48 | |
| *** Leo_m has quit IRC | 21:51 | |
| *** Leo_m_ has quit IRC | 22:54 | |
| *** Leo_m has joined #openstack-sdks | 22:55 | |
| *** Leo_m has quit IRC | 22:55 | |
| *** bobh has joined #openstack-sdks | 23:11 | |
| *** slaweq has quit IRC | 23:19 | |
| *** mriedem is now known as mriedem_away | 23:44 | |
| openstackgerrit | Merged openstack/openstacksdk master: Update baremetal objects with fields added up to Rocky https://review.openstack.org/599420 | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!