| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | 11:39 |
|---|---|---|
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Use OpenAPI for object_store download_object https://review.opendev.org/c/openstack/openstacksdk/+/893624 | 11:39 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Replace use of aliased exceptions https://review.opendev.org/c/openstack/openstacksdk/+/893499 | 11:46 |
| 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 |
| 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 |
| opendevreview | Merged openstack/openstacksdk master: Add support for default security group rules in SDK https://review.opendev.org/c/openstack/openstacksdk/+/887262 | 14:10 |
| frickler | stephenfin: gtema: ^^ shouldn't we obey feature freeze now until we branch? | 14:12 |
| stephenfin | damn, yes, we should. that's my fault | 14:12 |
| gtema | right, I also missed that | 14:13 |
| frickler | I mean, I don't think this particular patch is critical, so we can likely live with that, just a general reminder | 14:13 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: [WIP] Add OpenAPI invocation interface https://review.opendev.org/c/openstack/openstacksdk/+/892161 | 14:14 |
| 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:17 |
| gtema | I assume that you maybe do not get the service endpoints information with your auth and as such everything else is failing | 14:18 |
| Uggla | gtema, I'm doing more or less what was provided/documented by Stephen --> https://review.opendev.org/c/openstack/openstacksdk/+/893505 | 14:20 |
| Uggla | gtema, but to sumerise I think the idea is to "replace" the auth plugin with service_token.ServiceTokenAuthWrapper. | 14:22 |
| gtema | can you pls gather http requests/responses? | 14:24 |
| 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 |
| 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:25 |
| gtema | basically I assume that ServiceTokenAuthWrapper (as auth) does not contain any endpoints information and therefore everything fails | 14:27 |
| Uggla | gtema, you may be right, as I can't see any request flying in tcpdump. | 14:30 |
| gtema | before you establish connection add "openstack.enable_logging(debug=True)" | 14:31 |
| gtema | that will spit all the requests flying around | 14:31 |
| 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:39 |
| 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. | 14:42 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: Fix pre-commit issues https://review.opendev.org/c/openstack/cliff/+/893668 | 15:18 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: Add pre-commit https://review.opendev.org/c/openstack/cliff/+/893669 | 15:18 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: Blacken code base https://review.opendev.org/c/openstack/cliff/+/893670 | 15:18 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: Ignore blacken patch https://review.opendev.org/c/openstack/cliff/+/893671 | 15:18 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: tox: Use pre-commit for linter checks https://review.opendev.org/c/openstack/cliff/+/893672 | 15:18 |
| *** whoami-r- is now known as whoami-rajat | 15:54 | |
| opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [1/2] https://review.opendev.org/c/openstack/openstacksdk/+/887037 | 19:43 |
| opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | 19:43 |
| -opendevstatus- NOTICE: Some Gerrit changes that update Zuul configuration may fail with no response from Zuul. A fix is in progress. | 20:06 | |
| opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | 21:43 |
| -opendevstatus- NOTICE: Gerrit changes with updates to Zuul's configuration should now be handled correctly. Recheck any changes to Zuul configuration which did not report results. | 22:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!