Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-08-07
20:28:23 mordred so the json you return from list_object in your requests_mock should feed which segments it fires deletes for, yeah?
20:28:49 mordred (and you could probably make it a small list of segments so you don't go crazy)
20:29:23 Shrews i think that's right, just trying to figure out how to make that happen. thus far, it doesn't like my list objects uri i'm creating there
20:31:58 Shrews '{endpoint}/{container}?format=json&prefix={prefix}' <<-- that one
20:44:00 mordred Shrews: oh - so ... theres a thing with url parameters where you have to pass a thing
20:44:15 mordred Shrews: complete_qs=True
20:44:42 mordred Shrews: if the query string is meaningful, complete_qs causes it to be considered
20:45:35 Shrews mordred: i was wondering what that meant
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/

Earlier   Later