| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-08-31 | |||
| 13:06:46 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Prevent routers to be always updated if no shared public network https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893367 | |
| 13:09:15 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Fix usage of subnet_id key for router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893193 | |
| 13:23:14 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Ensure that double-stack routers are not always updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893370 | |
| 13:34:01 | Uggla | Hello guys, I'm using the sdk to connect to a manila service. I need to use service token, it seams that the sdk does not handle them (I could be wrong). Any advices on that topic ? | |
| 13:39:44 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Ensure that double-stack routers are not always updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893370 | |
| 13:54:50 | andrewbogott_ | stephenfin, gtema, here's the patch resulting from our conversation on Monday. Turns out it doesn't take much! https://review.opendev.org/c/openstack/openstacksdk/+/893283 | |
| 13:55:48 | gtema | yes, I have noticed it. Thanks andrewbogott_ I had already a very brief look at it, and the simplicity of it pushes me to look deeper on potential side effects | |
| 13:56:26 | andrewbogott_ | yeah, the simplicity worried me too :) | |
| 15:13:19 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Introduce OpenAPI specs for diverse compute methods https://review.opendev.org/c/openstack/openstacksdk/+/893365 | |
| 15:13:19 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Start using OpenAPI for diverse image operations https://review.opendev.org/c/openstack/openstacksdk/+/893366 | |
| 16:45:49 | opendevreview | Merged openstack/python-openstackclient master: Propocol release versions properly https://review.opendev.org/c/openstack/python-openstackclient/+/893343 | |
| 17:17:07 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 17:18:56 | opendevreview | Merged openstack/os-client-config master: replace "-" with "_" in setup.cfg https://review.opendev.org/c/openstack/os-client-config/+/893132 | |
| 17:18:57 | opendevreview | Merged openstack/os-client-config master: Remove ceilometerclient https://review.opendev.org/c/openstack/os-client-config/+/874366 | |
| #openstack-sdks - 2023-09-01 | |||
| 02:35:51 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume attachment support https://review.opendev.org/c/openstack/openstacksdk/+/884167 | |
| 02:44:54 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate volume attachment commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/888234 | |
| 02:45:27 | opendevreview | Merged openstack/python-openstackclient master: Migrate backup commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/889748 | |
| 07:03:24 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for default security group rule CRUDs https://review.opendev.org/c/openstack/python-openstackclient/+/887364 | |
| 07:20:29 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Fix usage of subnet_id key for router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893193 | |
| 07:23:06 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Fix usage of subnet_id key for router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893193 | |
| 08:44:54 | slaweq | gtema stephenfin hi, can one of You maybe check https://review.opendev.org/c/openstack/openstacksdk/+/887262 and (later) https://review.opendev.org/c/openstack/python-openstackclient/+/887364 - we have all neutron patches merged already :) | |
| 08:44:58 | slaweq | thx in advance | |
| 08:45:22 | gtema | slaweq - yupp, looking now | |
| 08:45:34 | slaweq | @thx a lot | |
| 08:45:44 | slaweq | gtema thx a lot :) | |
| 08:46:08 | gtema | I hope it is not urgent, cause SDK release went out just yesterday ;-) | |
| 08:46:29 | slaweq | no, it's not | |
| 08:46:36 | gtema | oki | |
| 08:46:57 | slaweq | I'm just pinging now about it as yesterday last missing part in neutron was finally merged | |
| 08:47:08 | slaweq | so I would like to finally finish all of this work :) | |
| 08:47:15 | slaweq | but it isn't urgent really | |
| 08:48:43 | gtema | great, thanks for info | |
| 09:35:23 | opendevreview | Jakob Meng proposed openstack/openstacksdk master: Use find_user because get_user has issues with filtering by domain_id https://review.opendev.org/c/openstack/openstacksdk/+/874541 | |
| 10:15:49 | Uggla | stephenfin, I try to use service token with the sdk. So far I changed the auth to use a ServiceTokenAuthWrapper plugin. Looking at the headers, it looks good, but the proxy discovery fails to find version. Any idea about the reason ? | |
| 10:16:24 | stephenfin | I have never worked with service tokens in SDK, so no. We're going to have to figure this out together. | |
| 10:16:32 | stephenfin | Could you put together an absolutely minimum reproducer? | |
| 10:17:06 | stephenfin | (I assume this is related to the manila-nova integration? I don't want to have to start nova to test this, heh) | |
| 10:17:40 | Uggla | stephenfin, yep you are right, this is for manila/nova integration. | |
| 10:18:46 | Uggla | stephenfin, but having a simple reproducer sounds difficult. | |
| 10:19:03 | stephenfin | Okay sweet | |
| 10:19:06 | stephenfin | If you recall, I added docs on how to configure a Connection from oslo.conf in https://review.opendev.org/c/openstack/openstacksdk/+/889892/. Perhaps you can start from there? | |
| 10:19:32 | stephenfin | andrewbogott_: I left some hopefully useful comments on that review | |
| 10:29:47 | Uggla | stephenfin, I'm gonna try to do a small reproducer. | |
| 10:36:14 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix usage of subnet_id key for router https://review.opendev.org/c/openstack/ansible-collections-openstack/+/893193 | |
| 10:43:31 | opendevreview | Merged openstack/python-openstackclient master: tox: Bump min_version to 4.3.0 https://review.opendev.org/c/openstack/python-openstackclient/+/882813 | |
| 11:01:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Rewrite caching docs https://review.opendev.org/c/openstack/openstacksdk/+/893491 | |
| 11:01:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove remnants of port caching https://review.opendev.org/c/openstack/openstacksdk/+/893492 | |
| 11:01:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove remnants of floating IP caching https://review.opendev.org/c/openstack/openstacksdk/+/893493 | |
| 11:01:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove remnants of server caching https://review.opendev.org/c/openstack/openstacksdk/+/893494 | |
| 11:01:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove floating network cache https://review.opendev.org/c/openstack/openstacksdk/+/893495 | |
| 11:01:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Don't use dangerous argument defaults https://review.opendev.org/c/openstack/openstacksdk/+/893496 | |
| 11:01:13 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove old cloud-layer caching functionality https://review.opendev.org/c/openstack/openstacksdk/+/893497 | |
| 11:01:18 | stephenfin | gtema: We can kill that now, yeah? ^ | |
| 11:05:13 | gtema | Right, I guess so | |
| 11:05:38 | stephenfin | Sweet | |
| 11:12:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove references to shade https://review.opendev.org/c/openstack/openstacksdk/+/893498 | |
| 11:12:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Replace use of aliased exceptions https://review.opendev.org/c/openstack/openstacksdk/+/893499 | |
| 11:35:58 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Adds stateful parameter to security groups https://review.opendev.org/c/openstack/ansible-collections-openstack/+/891975 | |
| 11:47:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: connection: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/893501 | |
| 12:15:08 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add docs on configuration of a service user https://review.opendev.org/c/openstack/openstacksdk/+/893505 | |
| 12:49:15 | opendevreview | kim woo seok proposed openstack/python-openstackclient master: Boot From Volume print image column in server info https://review.opendev.org/c/openstack/python-openstackclient/+/892833 | |
| 13:03:56 | opendevreview | kim woo seok proposed openstack/openstacksdk master: [WIP]Migration stack adopt in openstacksdk https://review.opendev.org/c/openstack/openstacksdk/+/891250 | |
| 13:04:41 | opendevreview | kim woo seok proposed openstack/openstacksdk master: [WIP]Migration stack adopt in openstacksdk https://review.opendev.org/c/openstack/openstacksdk/+/891250 | |
| 14:33:27 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Implemented heat 'stack suspend' and 'stack resume' function https://review.opendev.org/c/openstack/openstacksdk/+/892156 | |
| 15:54:05 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Ensure coe_cluster_template compare labels properly https://review.opendev.org/c/openstack/ansible-collections-openstack/+/882089 | |
| 15:56:28 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: VPN ike policy module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880531 | |
| 15:58:22 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: VPN ike policy module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880531 | |
| 16:01:25 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: VPN ike policy module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880531 | |
| 16:05:40 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: VPN ipsec policy module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880534 | |
| 16:12:01 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Vpn service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880565 | |
| 16:12:40 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Vpn service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880565 | |
| 16:13:20 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: VPN ipsec policy module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880534 | |
| 16:13:33 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Vpn service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880565 | |
| 16:14:00 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Vpn service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880565 | |
| 16:17:02 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Vpn service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880565 | |
| 16:19:05 | opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-collections-openstack master: Vpn service module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/880565 | |
| 17:53:36 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Adds stateful parameter to security groups https://review.opendev.org/c/openstack/ansible-collections-openstack/+/891975 | |
| 17:58:32 | opendevreview | Jakob Meng proposed openstack/openstacksdk master: Use find_user because get_user has issues with filtering by domain_id https://review.opendev.org/c/openstack/openstacksdk/+/874541 | |
| 20:33:35 | opendevreview | Merged openstack/ansible-collections-openstack master: Adds stateful parameter to security groups https://review.opendev.org/c/openstack/ansible-collections-openstack/+/891975 | |
| #openstack-sdks - 2023-09-02 | |||
| 00:00:56 | opendevreview | Andrew Bogott proposed openstack/openstacksdk master: config loader: Prefer cli/env over clouds.yaml args for some args https://review.opendev.org/c/openstack/openstacksdk/+/893283 | |
| #openstack-sdks - 2023-09-04 | |||
| 11:39:46 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 11:39:46 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Use OpenAPI for object_store download_object https://review.opendev.org/c/openstack/openstacksdk/+/893624 | |
| 11:46:45 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Replace use of aliased exceptions https://review.opendev.org/c/openstack/openstacksdk/+/893499 | |
| 13:06:35 | Uggla | Hi stephenfin, I need to come back with the service token issue. Last time I was too fast to answer. Basically I did what you suggested in the latest snipet. What I had done before was almost the same. I'm still stuggling the same issue -> https://paste.opendev.org/show/bckoNCvr83oGpR72fP4M. It seems that as soon as we use the ServiceTokenAuthWrapper the SDK discovery is falling the retrieve the revision. temp_adapter.get_api_major_version() in | |
| 13:06:35 | Uggla | service_description.py returns nothing. I spent a lot of time trying to figure why but haven't find it yet. So if you have a hint it will be great. | |
| 14:10:42 | opendevreview | Merged openstack/openstacksdk master: Add support for default security group rules in SDK https://review.opendev.org/c/openstack/openstacksdk/+/887262 | |
| 14:12:18 | frickler | stephenfin: gtema: ^^ shouldn't we obey feature freeze now until we branch? | |
| 14:12:36 | stephenfin | damn, yes, we should. that's my fault | |
| 14:13:22 | gtema | right, I also missed that | |
| 14:13:31 | frickler | I mean, I don't think this particular patch is critical, so we can likely live with that, just a general reminder | |
| 14:14:07 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 14:17:26 | gtema | uggla: can you pls give a short tl;dr version with snippet how you auth and what is the request/response when version discovery is triggered? I dislike message in your trace: "The shared-file-system service for :None..." | |
| 14:18:10 | gtema | I assume that you maybe do not get the service endpoints information with your auth and as such everything else is failing | |
| 14:20:11 | Uggla | gtema, I'm doing more or less what was provided/documented by Stephen --> https://review.opendev.org/c/openstack/openstacksdk/+/893505 | |
| 14:22:36 | Uggla | gtema, but to sumerise I think the idea is to "replace" the auth plugin with service_token.ServiceTokenAuthWrapper. | |
| 14:24:41 | gtema | can you pls gather http requests/responses? | |
| 14:25:02 | Uggla | gtema, with my code I can see the both tokens (user and service) in the headers of the request. But I think something has changed in the session that prevent "service discovery" by the sdk. | |
| 14:25:43 | gtema | I do not need the token in headers, but I am rather interested where the version discovery request goes and what is the response | |
| 14:27:16 | gtema | basically I assume that ServiceTokenAuthWrapper (as auth) does not contain any endpoints information and therefore everything fails | |
| 14:30:24 | Uggla | gtema, you may be right, as I can't see any request flying in tcpdump. | |