| *** slaweq_ has joined #openstack-sdks | 00:11 | |
| *** slaweq_ has quit IRC | 00:15 | |
| *** ricolin has joined #openstack-sdks | 01:03 | |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 01:44 |
|---|---|---|
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 01:46 |
| openstackgerrit | Merged openstack/keystoneauth master: Add remove_service to token fixtures https://review.opendev.org/675174 | 01:47 |
| *** slaweq_ has joined #openstack-sdks | 02:10 | |
| *** slaweq_ has quit IRC | 02:15 | |
| *** adriant has quit IRC | 02:54 | |
| *** whoami-rajat has joined #openstack-sdks | 03:12 | |
| *** gtema has joined #openstack-sdks | 03:24 | |
| *** gtema has quit IRC | 03:29 | |
| *** gkadam has joined #openstack-sdks | 03:31 | |
| *** gkadam has quit IRC | 03:40 | |
| *** slaweq_ has joined #openstack-sdks | 04:10 | |
| *** slaweq_ has quit IRC | 04:15 | |
| *** adriant has joined #openstack-sdks | 04:45 | |
| *** Luzi has joined #openstack-sdks | 05:00 | |
| *** dave-mccowan has quit IRC | 05:03 | |
| *** slaweq_ has joined #openstack-sdks | 06:11 | |
| *** slaweq_ has quit IRC | 06:16 | |
| *** ralonsoh has joined #openstack-sdks | 06:25 | |
| *** slaweq_ has joined #openstack-sdks | 06:59 | |
| *** tosky has joined #openstack-sdks | 07:17 | |
| *** gtema has joined #openstack-sdks | 07:26 | |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 07:27 |
| *** gtema has quit IRC | 07:30 | |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 07:30 |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 07:31 |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 07:33 |
| *** dtantsur|afk is now known as dtantsur | 07:43 | |
| *** jpich has joined #openstack-sdks | 08:02 | |
| *** e0ne has joined #openstack-sdks | 09:04 | |
| *** cdent has joined #openstack-sdks | 09:07 | |
| *** gtema has joined #openstack-sdks | 09:28 | |
| *** gtema has quit IRC | 09:32 | |
| stingrayza | Hi folks. I have a question about the os_zone ansible module. is this the right place, or should I take it over to one of the ansible channels? | 09:38 |
| frickler | stingrayza: I've seen people discuss ansible pull requests here, so you aren't too much off here I'd say | 09:51 |
| *** dmellado has quit IRC | 09:53 | |
| stingrayza | not so much a PR discussion, but I just want to sound something out before I log an issue... | 09:54 |
| stingrayza | let me create a pastebin quick | 09:54 |
| *** dmellado has joined #openstack-sdks | 09:55 | |
| frickler | stingrayza: you can use paste.openstack.org instead of some advertisement-heavy pastebin | 09:55 |
| stingrayza | frickler: thanks ;) exactly where I was. sorry - call it pastebin out of habit :) | 09:57 |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 10:03 |
| stingrayza | When I (try) create a zone using os_zone, I get an error "Zone xxxx not found" - but creating it via the cli works (and then os_zone passes) http://paste.openstack.org/show/755646/ | 10:06 |
| stingrayza | just wanted to check I'm not doing anything silly in my os_zone command | 10:07 |
| stingrayza | (triple checked the docs) | 10:07 |
| frickler | stingrayza: ya, I can reproduce this, seems os_zone tries to update the zone instead of creating it: http://paste.openstack.org/show/755648/ | 10:49 |
| frickler | mordred: mugsie: ^^ | 10:49 |
| stingrayza | frickler: thanks! I'll log an issue - and take a squiz at the code to see if I can submit a PR to fix ;) | 10:51 |
| frickler | stingrayza: mordred: fyi, this looks like a regression in openstacksdk, with 0.31.0 I don't see the error, just with 0.32.0 | 11:00 |
| *** jpich has quit IRC | 11:02 | |
| frickler | conn.get_zone('missing.com.') now returns False instead of None, failing the check in https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/cloud/openstack/os_zone.py#L197 | 11:16 |
| frickler | looks like gtema's patch introduced this https://review.opendev.org/665435 Switch cloud layer to use proxy for DNS | 11:17 |
| *** tobiash has quit IRC | 11:18 | |
| *** tobiash has joined #openstack-sdks | 11:20 | |
| mordred | frickler: poo. we have another patch up to fix an ansible regression - lets fix this one too | 11:23 |
| *** tobiash has quit IRC | 11:24 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix dns return values https://review.opendev.org/675298 | 11:25 |
| mordred | frickler: ^^ | 11:25 |
| mordred | Shrews, dtantsur: ^^ I can respin the release request with that included (which will also get us a release note for the other fix) | 11:25 |
| *** tobiash has joined #openstack-sdks | 11:26 | |
| frickler | mordred: commented, you've uncovered more inconsistencies | 11:29 |
| mordred | frickler: replied - want me to update the docstring there real quick? | 11:30 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix dns return values https://review.opendev.org/675298 | 11:30 |
| mordred | there ya go :) | 11:30 |
| frickler | mordred: if this is really a doc bug instead of a behaviour change, then I'm fine with it | 11:30 |
| frickler | mordred: let me do a quick crosscheck against 0.31.0 | 11:31 |
| frickler | mordred: o.k., 0.31.0 returned None there, too, so that's o.k., then | 11:33 |
| mordred | cool | 11:37 |
| frickler | mordred: but I'd actually tend to agree with Thierry on that the release might better be 0.33.0 than 0.32.1 | 11:39 |
| stingrayza | frickler, mordred: fantastic! thanks gents :D | 11:44 |
| stingrayza | at least I know I wasn't going mad ;D | 11:44 |
| stingrayza | ^ my default position when I find a bug. | 11:45 |
| openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: WIP: Add server migration list CLI https://review.opendev.org/675304 | 11:49 |
| openstackgerrit | Maxim Babushkin proposed openstack/openstacksdk master: Add 'tag' support to compute with supported microversion https://review.opendev.org/675306 | 11:50 |
| openstackgerrit | Maxim Babushkin proposed openstack/openstacksdk master: Add 'tag' support to compute with supported microversion https://review.opendev.org/675306 | 11:52 |
| openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | 11:53 |
| mordred | frickler: yeah - agree re: 0.33 | 11:58 |
| *** jpich has joined #openstack-sdks | 12:02 | |
| *** cdent has quit IRC | 12:04 | |
| *** ricolin_ has joined #openstack-sdks | 12:05 | |
| *** ricolin has quit IRC | 12:07 | |
| *** jpich has quit IRC | 12:09 | |
| *** jpich has joined #openstack-sdks | 12:10 | |
| *** jpich has quit IRC | 12:11 | |
| *** jpich has joined #openstack-sdks | 12:11 | |
| mordred | kmalloc: whence you are awakened, I have a question for you on keystone v2 admin interface things | 12:29 |
| mordred | kmalloc: using the token fixture from ksa I've got a catalog in the sdk tests now that uses differnt hostnames for admin and public endpoints - this is flushing out some places where things were off but we weren't noticing because a requests-mock uri registered for either would work (same hostname) | 12:30 |
| mordred | in a few places I'm getting: No mock address: GET https://identity.example.com/v2.0/OS-KSADM/roles | 12:30 |
| mordred | I would have thought that OS-KSADM would really only ever be on the admin interface | 12:31 |
| mordred | but that's telling me the code is making a request to OS-KSADM on the public interface | 12:31 |
| mordred | is the code wrong? or the test wrong? | 12:31 |
| mordred | cmurphy: ^^ in case you know | 12:31 |
| *** Luzi has quit IRC | 12:32 | |
| mordred | OR - should I put a pin in it for later since at this point keystonev2 is so old it makes us all want to die inside | 12:34 |
| *** jpich has quit IRC | 12:35 | |
| *** jpich has joined #openstack-sdks | 12:36 | |
| *** jpich has quit IRC | 12:51 | |
| *** jpich has joined #openstack-sdks | 12:51 | |
| openstackgerrit | Andrey Kurilin proposed openstack/osc-lib master: Fix usage of NotFound exception https://review.opendev.org/675328 | 13:00 |
| *** cdent has joined #openstack-sdks | 13:05 | |
| mordred | kmalloc:, cmurphy: I think the right answer for now is to just skip it for now - I don't think it's worth the effort to cleanup keystonev2 unless someone actually complains | 13:05 |
| *** mriedem has joined #openstack-sdks | 13:16 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | 13:25 |
| dtantsur | mordred: approved | 13:25 |
| mordred | dtantsur: woot! thanks | 13:25 |
| *** slaweq_ is now known as slaweq | 13:29 | |
| *** gtema has joined #openstack-sdks | 13:29 | |
| *** gtema has quit IRC | 13:34 | |
| *** stephenfin has quit IRC | 13:35 | |
| *** stephenfin has joined #openstack-sdks | 13:36 | |
| *** ricolin_ is now known as ricolin | 13:55 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | 14:20 |
| *** cdent has quit IRC | 14:31 | |
| mordred | Shrews, dtantsur, kmalloc: ^^ that one is ready for review - although it needs a ksa release so it can't be landed and the tests won't go totally green until that one exists | 14:35 |
| openstackgerrit | Andrey Kurilin proposed openstack/osc-lib master: Fix usage of NotFound exception https://review.opendev.org/675328 | 14:42 |
| openstackgerrit | Merged openstack/openstacksdk master: Fix dns return values https://review.opendev.org/675298 | 15:01 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove container-infra.example.com hack https://review.opendev.org/675378 | 15:10 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Remove the block-storage catalog hack https://review.opendev.org/675382 | 15:26 |
| kmalloc | mordred: o/ | 15:30 |
| *** gtema has joined #openstack-sdks | 15:31 | |
| *** dave-mccowan has joined #openstack-sdks | 15:32 | |
| kmalloc | mordred: so, in v3 admin/public are the same. really the only thing anything knows is it should talk to keystone. the split between admin and public was really a "do I get a 404 or not", and people could run both on the same endpoint in reality if thye wanted (some did even in v2) | 15:32 |
| kmalloc | mordred: soooo possibly the test or fixture are subtly wrong. | 15:32 |
| kmalloc | and OS-XXXX extensions were weird. | 15:33 |
| mordred | kmalloc: well - for v3 we're fine - it's all v2 cases where we're messed up | 15:33 |
| kmalloc | mordred: if someone complains about v2 we can clean it up. | 15:33 |
| mordred | (for v3 nothing is coded to request admin endpoints) | 15:33 |
| mordred | cool | 15:33 |
| mordred | yeah - that's where I got to | 15:33 |
| mordred | kmalloc: in fact- what if I don't even have admin endpoints for anything in the v3 catalog? | 15:34 |
| kmalloc | otherwise i would let v2 just kindof sit. my guess is we will rapidly get to a point where no one will be using v2 even with SDK. | 15:34 |
| mordred | (so that we catch places where people are doing something that they probably shouldnt be) | 15:34 |
| kmalloc | someone was working on fixing that in the general case (no admin endpoints in gate in v3) recently | 15:34 |
| kmalloc | i am not sure what landed. | 15:34 |
| kmalloc | but some things were still a little broken. i support going to no admin endpoints in v3 | 15:35 |
| kmalloc | esp. in SDK testing. | 15:35 |
| kmalloc | no explicit admin endpoints* | 15:35 |
| *** gtema has quit IRC | 15:35 | |
| mordred | ++ | 15:35 |
| *** ricolin_ has joined #openstack-sdks | 15:43 | |
| *** ricolin_ has quit IRC | 15:45 | |
| *** ricolin has quit IRC | 15:46 | |
| *** ricolin_ has joined #openstack-sdks | 15:46 | |
| *** ricolin_ is now known as ricolin | 15:47 | |
| edleafe | What's that you say? The API-SIG Office Hour is upon us? | 16:00 |
| elmiko | \o/ | 16:00 |
| * dtantsur lurks while on another meeting | 16:08 | |
| *** e0ne has quit IRC | 16:16 | |
| *** jpich has quit IRC | 16:29 | |
| *** cdent has joined #openstack-sdks | 16:29 | |
| *** jangutter has quit IRC | 16:44 | |
| *** tosky has quit IRC | 16:49 | |
| *** dtantsur is now known as dtantsur|afk | 16:57 | |
| edleafe | Guess all that lurking wore dtantsur|afk out! | 16:57 |
| elmiko | haha, yeah! | 16:58 |
| elmiko | we can blame it on the /other/ meeting XD | 16:58 |
| elmiko | i won't be around next week, i'm on conference duty ;) | 17:00 |
| elmiko | have a good weekend edleafe o/ | 17:00 |
| edleafe | you too, elmiko! | 17:01 |
| *** e0ne has joined #openstack-sdks | 17:05 | |
| *** ricolin_ has joined #openstack-sdks | 17:16 | |
| *** ricolin has quit IRC | 17:19 | |
| *** gtema has joined #openstack-sdks | 17:32 | |
| *** ricolin_ is now known as ricolin | 17:32 | |
| *** e0ne has quit IRC | 17:35 | |
| *** gtema has quit IRC | 17:36 | |
| *** cdent has quit IRC | 17:53 | |
| *** ralonsoh has quit IRC | 17:54 | |
| *** bobh has joined #openstack-sdks | 18:35 | |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134 | 18:49 |
| frickler | mordred: kmalloc: for what happens with no admin endpoint see https://review.opendev.org/651492 , heat blows up when using the OS-TRUST interface, so that might be related to your v2 issue | 18:50 |
| frickler | anyway help with fixing that remaining issue would be greatly appreciated, I dug through the tracebacks and didn't find anything helpful | 18:51 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134 | 19:00 |
| *** bobh has quit IRC | 19:06 | |
| *** e0ne has joined #openstack-sdks | 19:16 | |
| *** bobh has joined #openstack-sdks | 19:19 | |
| Shrews | mordred: omg, i got the test working | 19:21 |
| Shrews | mordred: i'm assuming we want the same retry logic on dlo object manifests, yeah? Would it also need to delete the segments? | 19:23 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Retry large object manifest upload https://review.opendev.org/675134 | 19:44 |
| Shrews | mordred: I just realized that we have the same problem if any of the segment uploads fail. :/ So no limited to manifest failure. | 19:45 |
| Shrews | So not* | 19:45 |
| *** bobh has quit IRC | 19:56 | |
| *** bobh has joined #openstack-sdks | 19:56 | |
| *** bobh has quit IRC | 20:01 | |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 20:27 |
| *** e0ne has quit IRC | 20:37 | |
| mordred | Shrews: this is quite the big beastie | 20:43 |
| mordred | Shrews: but yay! | 20:43 |
| mordred | Shrews: I believe with DLO objects we just pass the "delete your manifests" parameter and deleting the manifest object will cause the segment objects to be deleted | 20:44 |
| Shrews | mordred: but that requires actually uploading the manifest, yeah? | 20:45 |
| mordred | oh - joy. yes | 20:46 |
| openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | 20:46 |
| mordred | Shrews: you have the smart brain | 20:46 |
| Shrews | can't delete it if you can't upload it :-P | 20:46 |
| efried | mordred: I loaded up your patch from yesterday and was able to get nova talking to cyborg just fine. | 20:53 |
| mordred | efried: woot | 20:53 |
| efried | I've got the setup running in a public cloud, do you want to poke at anything? | 20:53 |
| *** whoami-rajat has quit IRC | 21:01 | |
| efried | mordred: ...but afaict it doesn't look any different than the ksa version. | 21:02 |
| mordred | efried: I think as long as it worked like you expected that's good enough for me | 21:03 |
| mordred | Shrews: if you have a sec in between bouts of yelling at object-store apis ... https://review.opendev.org/#/c/675135/ is easy-ish | 21:04 |
| *** rm_work has quit IRC | 21:05 | |
| mordred | Shrews: (and this is the patch efried is mentioning that he tested in the scrollback just above that makes cyborg and placement integration in nova work better) | 21:05 |
| *** rm_work has joined #openstack-sdks | 21:05 | |
| efried | mordred: We expected that to get rid of a bunch of warnings to placement as well, let me see if I can verify that... | 21:06 |
| mordred | efried: if it doesn't, something is horribly wrong | 21:06 |
| efried | mordred: Okay, I confirmed. Went back through the logs before I applied your patch and found those UnsupportedServiceVersion warnings, and they then went away once the patch was applied. | 21:27 |
| mordred | efried: woot! | 21:28 |
| mordred | senseless warnings in logs are like fungus | 21:28 |
| mordred | you gotta get rid of them before the spread | 21:28 |
| efried | left a +1 accordingly. | 21:29 |
| efried | mordred: though a guy wonders if there should be some unit test for this.... | 21:29 |
| mordred | efried: some days I feel like you and Shrews are the same person what with your "shouldn't there be a test for this" ;) | 21:30 |
| efried | I can think of worse company to be in. | 21:31 |
| efried | I mean, I'm happy having tested this live, BUT I only tested it with a nice clean default devstack service catalog and conf | 21:31 |
| mordred | efried: oh - I don't know if you saw - patch one on on the path to getting you a reusable fixture is actually working now | 21:31 |
| efried | nice | 21:31 |
| mordred | efried: people use openstack in some configuration that isn't a nice clean default devstack? | 21:31 |
| efried | ...I didn't put it through the paces of endpoint_override, weird service catalog entries, non-default interfaces, weird version requests.... | 21:32 |
| *** gtema has joined #openstack-sdks | 21:33 | |
| mordred | yeah. well - so - the nice thing here (and we should do all of those things) - but the nice thing about this patch is it basically says "if we don't know anything, just defer 100% to the existing and already tested elsewhere adapter creation logic" | 21:35 |
| mordred | so all the other combos *should* be covered already | 21:35 |
| mordred | but that's me rationalizing a scenario that clearly needs some coverage :) | 21:35 |
| *** gtema has quit IRC | 21:38 | |
| Shrews | mordred: i insist on a test for the test as well | 21:47 |
| *** slaweq has quit IRC | 22:15 | |
| *** whoami-rajat has joined #openstack-sdks | 22:24 | |
| *** ricolin_ has joined #openstack-sdks | 23:02 | |
| *** ricolin has quit IRC | 23:05 | |
| *** slaweq has joined #openstack-sdks | 23:11 | |
| *** slaweq has quit IRC | 23:15 | |
| *** gtema has joined #openstack-sdks | 23:35 | |
| *** gtema has quit IRC | 23:39 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!