| Posted | Nick | Remark | |
|---|---|---|---|
| #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. | |
| 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: Fix pre-commit issues https://review.opendev.org/c/openstack/cliff/+/893668 | |
| 15:18:05 | opendevreview | Stephen Finucane proposed openstack/cliff master: Add pre-commit https://review.opendev.org/c/openstack/cliff/+/893669 | |
| 15:18:06 | opendevreview | Stephen Finucane proposed openstack/cliff master: Blacken code base https://review.opendev.org/c/openstack/cliff/+/893670 | |
| 15:18:06 | opendevreview | Stephen Finucane proposed openstack/cliff master: Ignore blacken patch https://review.opendev.org/c/openstack/cliff/+/893671 | |
| 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 [1/2] https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |
| 19:43:25 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | |
| 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: Fix pre-commit issues https://review.opendev.org/c/openstack/cliff/+/893668 | |
| 09:17:50 | opendevreview | Stephen Finucane proposed openstack/cliff master: Add pre-commit https://review.opendev.org/c/openstack/cliff/+/893669 | |
| 09:17:51 | opendevreview | Stephen Finucane proposed openstack/cliff master: Blacken code base https://review.opendev.org/c/openstack/cliff/+/893670 | |
| 09:17:51 | opendevreview | Stephen Finucane proposed openstack/cliff master: Ignore blacken patch https://review.opendev.org/c/openstack/cliff/+/893671 | |
| 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 | |
| 12:55:01 | opendevreview | BAESUKJU proposed openstack/python-openstackclient master: Add new glance cache command - image cache clear - image cache delete - image cache list - image cache queue https://review.opendev.org/c/openstack/python-openstackclient/+/893905 | |
| 12:56:14 | opendevreview | BAESUKJU proposed openstack/python-openstackclient master: Add new glance cache command https://review.opendev.org/c/openstack/python-openstackclient/+/893905 | |
| 13:49:55 | opendevreview | BAESUKJU proposed openstack/python-openstackclient master: Add new glance cache command https://review.opendev.org/c/openstack/python-openstackclient/+/893905 | |
| 14:51:18 | diablo_rojo | gtema: stephenfin things are going well with Antonia I assume? Did you need me in the meeting today? | |
| 14:51:50 | gtema | we need you always ;-) | |
| 14:52:02 | gtema | but feel free to skip today | |
| 14:52:05 | diablo_rojo | Lol | |
| 14:52:25 | diablo_rojo | I am pingable if there is anything I can help with :) | |
| 14:52:39 | diablo_rojo | Seems like she is doing well though. | |
| 14:53:51 | diablo_rojo | One patch merged, another currently in merge conflict, but maybe that will get fixed before the meeting. | |
| 14:54:01 | gtema | yupp | |
| 16:13:47 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | |
| 16:19:44 | opendevreview | kim woo seok proposed openstack/python-openstackclient master: Fix server list command with -c option https://review.opendev.org/c/openstack/python-openstackclient/+/892830 | |
| 16:36:19 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | |
| 16:40:51 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | |
| 17:39:59 | opendevreview | Antonia Gaete proposed openstack/openstacksdk master: Add support for other metadef property operations https://review.opendev.org/c/openstack/openstacksdk/+/857727 | |
| #openstack-sdks - 2023-09-07 | |||
| 01:21:46 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | |
| 02:12:06 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack event list' https://review.opendev.org/c/openstack/openstacksdk/+/892836 | |
| 09:49:04 | opendevreview | OpenStack Release Bot proposed openstack/cliff stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/cliff/+/894146 | |
| 09:49:05 | opendevreview | OpenStack Release Bot proposed openstack/cliff stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/cliff/+/894148 | |
| 09:49:20 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/openstacksdk/+/894149 | |
| 09:49:22 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/openstacksdk/+/894150 | |
| 09:49:24 | opendevreview | OpenStack Release Bot proposed openstack/openstacksdk master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/openstacksdk/+/894151 | |
| 09:49:32 | opendevreview | OpenStack Release Bot proposed openstack/osc-lib stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/osc-lib/+/894152 | |
| 09:49:34 | opendevreview | OpenStack Release Bot proposed openstack/osc-lib stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/osc-lib/+/894153 | |
| 09:49:35 | opendevreview | OpenStack Release Bot proposed openstack/osc-lib master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/osc-lib/+/894154 | |
| 10:27:55 | opendevreview | Takashi Kajinami proposed openstack/microversion-parse master: Fix warnings in doc build https://review.opendev.org/c/openstack/microversion-parse/+/894167 | |
| 10:48:22 | opendevreview | Takashi Kajinami proposed openstack/microversion-parse master: Fix warnings in doc build https://review.opendev.org/c/openstack/microversion-parse/+/894167 | |
| 11:19:18 | opendevreview | Merged openstack/microversion-parse master: Fix warnings in doc build https://review.opendev.org/c/openstack/microversion-parse/+/894167 | |
| 11:36:01 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate resource filter commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/893802 | |
| 11:39:04 | opendevreview | Merged openstack/microversion-parse master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/microversion-parse/+/890632 | |
| 11:49:23 | opendevreview | Merged openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 14:58:32 | opendevreview | SongSuheon proposed openstack/openstacksdk master: Implement heat 'stack resource signal' https://review.opendev.org/c/openstack/openstacksdk/+/894203 | |