| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-08-07 | |||
| 20:46:12 | Shrews | i found nothing meaningful :( thx | |
| 20:46:36 | mordred | Shrews: are you saying we should document our test framework? | |
| 20:47:21 | mordred | Shrews: if so - you're in luck - because I'm working on extracting all this requests-mock/url stuff into a reusable fixture that openstacksdk consumers can also use - which means we'll necessarily need to give it a proper api and document that API | |
| 20:47:30 | mordred | if that's not what your'e saying, you're still in luck, for the same reason ;) | |
| 21:43:06 | openstackgerrit | Morgan Fainberg proposed openstack/keystoneauth master: Add remove_service to token fixtures https://review.opendev.org/675174 | |
| 22:05:02 | efried | mordred: Sundar has been pretty busy today, so only sparsely working with your patch, but we seem to have at least gotten past that initial crash. | |
| 22:49:34 | mordred | efried: sweet | |
| 22:51:03 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | |
| 23:08:52 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | |
| #openstack-sdks - 2019-08-08 | |||
| 01:44:36 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 01:46:40 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 01:47:05 | openstackgerrit | Merged openstack/keystoneauth master: Add remove_service to token fixtures https://review.opendev.org/675174 | |
| 07:27:47 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 07:30:53 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 07:31:29 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 07:33:00 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 09:38:07 | 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:51:40 | frickler | stingrayza: I've seen people discuss ansible pull requests here, so you aren't too much off here I'd say | |
| 09:54:43 | stingrayza | not so much a PR discussion, but I just want to sound something out before I log an issue... | |
| 09:54:51 | stingrayza | let me create a pastebin quick | |
| 09:55:53 | frickler | stingrayza: you can use paste.openstack.org instead of some advertisement-heavy pastebin | |
| 09:57:08 | stingrayza | frickler: thanks ;) exactly where I was. sorry - call it pastebin out of habit :) | |
| 10:03:55 | openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | |
| 10:06:52 | 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:07:14 | stingrayza | just wanted to check I'm not doing anything silly in my os_zone command | |
| 10:07:22 | stingrayza | (triple checked the docs) | |
| 10:49:08 | 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:22 | frickler | mordred: mugsie: ^^ | |
| 10:51:50 | 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 ;) | |
| 11:00:57 | 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:16:27 | 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:17:34 | frickler | looks like gtema's patch introduced this https://review.opendev.org/665435 Switch cloud layer to use proxy for DNS | |
| 11:23:16 | mordred | frickler: poo. we have another patch up to fix an ansible regression - lets fix this one too | |
| 11:25:07 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix dns return values https://review.opendev.org/675298 | |
| 11:25:11 | mordred | frickler: ^^ | |
| 11:25:49 | 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:29:05 | frickler | mordred: commented, you've uncovered more inconsistencies | |
| 11:30:08 | mordred | frickler: replied - want me to update the docstring there real quick? | |
| 11:30:38 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix dns return values https://review.opendev.org/675298 | |
| 11:30:45 | mordred | there ya go :) | |
| 11:30:59 | frickler | mordred: if this is really a doc bug instead of a behaviour change, then I'm fine with it | |
| 11:31:19 | frickler | mordred: let me do a quick crosscheck against 0.31.0 | |
| 11:33:57 | frickler | mordred: o.k., 0.31.0 returned None there, too, so that's o.k., then | |
| 11:37:53 | mordred | cool | |
| 11:39:21 | 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:44:23 | stingrayza | frickler, mordred: fantastic! thanks gents :D | |
| 11:44:40 | stingrayza | at least I know I wasn't going mad ;D | |
| 11:45:05 | stingrayza | ^ my default position when I find a bug. | |
| 11:49:26 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: WIP: Add server migration list CLI https://review.opendev.org/675304 | |
| 11:50:29 | openstackgerrit | Maxim Babushkin proposed openstack/openstacksdk master: Add 'tag' support to compute with supported microversion https://review.opendev.org/675306 | |
| 11:52:41 | openstackgerrit | Maxim Babushkin proposed openstack/openstacksdk master: Add 'tag' support to compute with supported microversion https://review.opendev.org/675306 | |
| 11:53:25 | openstackgerrit | Loan Harrouin proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/675113 | |
| 11:58:52 | mordred | frickler: yeah - agree re: 0.33 | |
| 12:29:30 | mordred | kmalloc: whence you are awakened, I have a question for you on keystone v2 admin interface things | |
| 12:30:38 | 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:48 | mordred | in a few places I'm getting: No mock address: GET https://identity.example.com/v2.0/OS-KSADM/roles | |
| 12:31:03 | mordred | I would have thought that OS-KSADM would really only ever be on the admin interface | |
| 12:31:24 | mordred | but that's telling me the code is making a request to OS-KSADM on the public interface | |
| 12:31:31 | mordred | is the code wrong? or the test wrong? | |
| 12:31:38 | mordred | cmurphy: ^^ in case you know | |
| 12:34:18 | 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 | |
| 13:00:01 | openstackgerrit | Andrey Kurilin proposed openstack/osc-lib master: Fix usage of NotFound exception https://review.opendev.org/675328 | |
| 13:05:37 | 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:25:33 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | |
| 13:25:40 | dtantsur | mordred: approved | |
| 13:25:44 | mordred | dtantsur: woot! thanks | |
| 14:20:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | |
| 14:35:19 | 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:42:26 | openstackgerrit | Andrey Kurilin proposed openstack/osc-lib master: Fix usage of NotFound exception https://review.opendev.org/675328 | |
| 15:01:17 | openstackgerrit | Merged openstack/openstacksdk master: Fix dns return values https://review.opendev.org/675298 | |
| 15:10:20 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove container-infra.example.com hack https://review.opendev.org/675378 | |
| 15:26:54 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Remove the block-storage catalog hack https://review.opendev.org/675382 | |
| 15:30:40 | kmalloc | mordred: o/ | |
| 15:32:30 | 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:54 | kmalloc | mordred: soooo possibly the test or fixture are subtly wrong. | |
| 15:33:25 | kmalloc | and OS-XXXX extensions were weird. | |
| 15:33:29 | mordred | kmalloc: well - for v3 we're fine - it's all v2 cases where we're messed up | |
| 15:33:42 | kmalloc | mordred: if someone complains about v2 we can clean it up. | |
| 15:33:45 | mordred | (for v3 nothing is coded to request admin endpoints) | |
| 15:33:47 | mordred | cool | |
| 15:33:51 | mordred | yeah - that's where I got to | |
| 15:34:16 | mordred | kmalloc: in fact- what if I don't even have admin endpoints for anything in the v3 catalog? | |
| 15:34:17 | 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:40 | mordred | (so that we catch places where people are doing something that they probably shouldnt be) | |
| 15:34:46 | kmalloc | someone was working on fixing that in the general case (no admin endpoints in gate in v3) recently | |
| 15:34:50 | kmalloc | i am not sure what landed. | |
| 15:35:08 | kmalloc | but some things were still a little broken. i support going to no admin endpoints in v3 | |
| 15:35:14 | kmalloc | esp. in SDK testing. | |
| 15:35:24 | kmalloc | no explicit admin endpoints* | |
| 15:35:44 | mordred | ++ | |
| 16:00:00 | edleafe | What's that you say? The API-SIG Office Hour is upon us? | |
| 16:00:21 | elmiko | \o/ | |
| 16:57:51 | edleafe | Guess all that lurking wore dtantsur|afk out! | |
| 16:58:19 | elmiko | haha, yeah! | |
| 16:58:34 | elmiko | we can blame it on the /other/ meeting XD | |
| 17:00:47 | elmiko | i won't be around next week, i'm on conference duty ;) | |
| 17:00:54 | elmiko | have a good weekend edleafe o/ | |
| 17:01:03 | edleafe | you too, elmiko! | |
| 18:49:23 | openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: WIP: Retry SLO manifest upload https://review.opendev.org/675134 | |
| 18:50:24 | 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 | |