| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-09-01 | |||
| 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: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: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:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove floating network cache https://review.opendev.org/c/openstack/openstacksdk/+/893495 | |
| 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: cloud: Replace use of aliased exceptions https://review.opendev.org/c/openstack/openstacksdk/+/893499 | |
| 11:12:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove references to shade https://review.opendev.org/c/openstack/openstacksdk/+/893498 | |
| 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: Use OpenAPI for object_store download_object https://review.opendev.org/c/openstack/openstacksdk/+/893624 | |
| 11:39:46 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | |
| 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 | 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. | |
| 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 | |
| 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. | |
| 14:31:23 | gtema | before you establish connection add "openstack.enable_logging(debug=True)" | |
| 14:31:25 | gtema | that will spit all the requests flying around | |
| 14:39:54 | frickler | stephenfin: gtema: I just noticed that the branching patch has already been proposed, so if there is no need for a bugfix patch and release until Friday, all should be fine anyway https://review.opendev.org/c/openstack/releases/+/893619 | |
| 14:42:22 | Uggla | gtema, without service token, I can see the requests. Not the case passing the ServiceTokenAuthWrapper. I think something bad arrives before and I'm fooled by the log. | |
| 15:18:05 | opendevreview | Stephen Finucane proposed openstack/cliff master: Add pre-commit https://review.opendev.org/c/openstack/cliff/+/893669 | |
| 15:18:05 | opendevreview | Stephen Finucane proposed openstack/cliff master: Fix pre-commit issues https://review.opendev.org/c/openstack/cliff/+/893668 | |
| 15:18:06 | opendevreview | Stephen Finucane proposed openstack/cliff master: Ignore blacken patch https://review.opendev.org/c/openstack/cliff/+/893671 | |
| 15:18:06 | opendevreview | Stephen Finucane proposed openstack/cliff master: Blacken code base https://review.opendev.org/c/openstack/cliff/+/893670 | |
| 15:18:07 | opendevreview | Stephen Finucane proposed openstack/cliff master: tox: Use pre-commit for linter checks https://review.opendev.org/c/openstack/cliff/+/893672 | |
| 19:43:25 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | |
| 19:43:25 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [1/2] https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |
| 21:43:13 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | |
| #openstack-sdks - 2023-09-05 | |||
| 03:36:46 | opendevreview | Ke Niu proposed openstack/python-openstackclient master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/python-openstackclient/+/850988 | |
| 09:17:50 | opendevreview | Stephen Finucane proposed openstack/cliff master: Add pre-commit https://review.opendev.org/c/openstack/cliff/+/893669 | |
| 09:17:50 | opendevreview | Stephen Finucane proposed openstack/cliff master: Fix pre-commit issues https://review.opendev.org/c/openstack/cliff/+/893668 | |
| 09:17:51 | opendevreview | Stephen Finucane proposed openstack/cliff master: Ignore blacken patch https://review.opendev.org/c/openstack/cliff/+/893671 | |
| 09:17:51 | opendevreview | Stephen Finucane proposed openstack/cliff master: Blacken code base https://review.opendev.org/c/openstack/cliff/+/893670 | |
| 09:17:52 | opendevreview | Stephen Finucane proposed openstack/cliff master: tox: Use pre-commit for linter checks https://review.opendev.org/c/openstack/cliff/+/893672 | |
| 09:33:57 | stephenfin | frickler: gtema: So be clear, we can merge away on the assumption that we can always fix issues with the release once the stable branch is cut? | |
| 09:34:05 | stephenfin | *to be | |
| 10:36:53 | gtema | It should be. I have +1ed the branching change, so there should be no conflicts theoretically | |
| 11:00:03 | opendevreview | Merged openstack/openstacksdk master: docs: Use asterix for Network resources doc https://review.opendev.org/c/openstack/openstacksdk/+/892668 | |
| 11:00:23 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Improve support for quota sets https://review.opendev.org/c/openstack/openstacksdk/+/784704 | |
| 11:31:43 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add "hardware_offload_type" attribute to "port" https://review.opendev.org/c/openstack/openstacksdk/+/892771 | |
| 11:50:34 | opendevreview | Merged openstack/openstacksdk master: Unused param in stack unit test https://review.opendev.org/c/openstack/openstacksdk/+/892834 | |
| 12:14:18 | opendevreview | Merged openstack/openstacksdk master: Implement stack export in openstacksdk https://review.opendev.org/c/openstack/openstacksdk/+/891249 | |
| 14:17:18 | opendevreview | Pavlo Shchelokovskyy proposed openstack/cliff master: Handle complex objects in yaml formatter better https://review.opendev.org/c/openstack/cliff/+/893741 | |
| 14:24:45 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove unused hook https://review.opendev.org/c/openstack/openstacksdk/+/893743 | |
| 14:31:41 | opendevreview | Pavlo Shchelokovskyy proposed openstack/cliff master: Handle complex objects in yaml formatter better https://review.opendev.org/c/openstack/cliff/+/893741 | |
| 16:11:16 | opendevreview | Merged openstack/cliff master: Fix flake8 violation E721 https://review.opendev.org/c/openstack/cliff/+/893047 | |
| 18:47:50 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate resource filter commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/893802 | |
| 18:50:41 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate resource filter commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/893802 | |
| 19:01:09 | opendevreview | Merged openstack/openstacksdk master: fix block storage resource registry https://review.opendev.org/c/openstack/openstacksdk/+/890398 | |
| 19:30:02 | opendevreview | Merged openstack/python-openstackclient master: Migrate 'availability zone list' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/885135 | |
| 19:30:04 | opendevreview | Merged openstack/python-openstackclient master: Migrate 'extension list' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/885139 | |
| 19:30:06 | opendevreview | Merged openstack/python-openstackclient master: Remove project purge image commands https://review.opendev.org/c/openstack/python-openstackclient/+/885528 | |
| 19:30:07 | opendevreview | Merged openstack/python-openstackclient master: tests: Remove FakeNetworkV2Client https://review.opendev.org/c/openstack/python-openstackclient/+/885526 | |
| 19:35:50 | opendevreview | Merged openstack/python-openstackclient master: tests: Use consistent shortcut to fake network client https://review.opendev.org/c/openstack/python-openstackclient/+/892788 | |
| 22:36:20 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Fix: volume request body for scheduler hints https://review.opendev.org/c/openstack/openstacksdk/+/890593 | |
| #openstack-sdks - 2023-09-06 | |||
| 10:34:26 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Implemented heat 'stack suspend' and 'stack resume' function https://review.opendev.org/c/openstack/openstacksdk/+/892156 | |
| 10:59:40 | 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 | |
| 11:40:36 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 12:08:35 | opendevreview | Merged openstack/python-openstackclient master: tests: Add network v2 FakeClientMixin https://review.opendev.org/c/openstack/python-openstackclient/+/885527 | |
| 12:16:35 | opendevreview | Merged openstack/python-openstackclient master: tests: Remove FakeImagev1Client, FakeImagev2Client https://review.opendev.org/c/openstack/python-openstackclient/+/885529 | |
| 12:26:00 | opendevreview | Merged openstack/python-openstackclient master: tests: Use consistent shortcut to fake image client https://review.opendev.org/c/openstack/python-openstackclient/+/892789 | |
| 12:29:15 | opendevreview | Merged openstack/python-openstackclient master: tests: Add image v1, v2 FakeClientMixin https://review.opendev.org/c/openstack/python-openstackclient/+/885530 | |
| 12:46:50 | 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 | |