From 49788a8a3a5714c0a08f7001ecf0fb3ce664f723 Mon Sep 17 00:00:00 2001 From: Gerrit User 28522 <28522@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Date: Fri, 29 Aug 2025 13:37:28 +0000 Subject: [PATCH] Update patch set 1 Patch Set 1: (1 comment) Patch-set: 1 --- ee58ef4d0084b01a1bbfe193626642bfe57b80c7 | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/ee58ef4d0084b01a1bbfe193626642bfe57b80c7 b/ee58ef4d0084b01a1bbfe193626642bfe57b80c7 index 0348ddb34..8707fea60 100644 --- a/ee58ef4d0084b01a1bbfe193626642bfe57b80c7 +++ b/ee58ef4d0084b01a1bbfe193626642bfe57b80c7 @@ -16,6 +16,24 @@ "message": "As python-keystoneclient is imported here and there in various services [1] I think we should consider it as oslo libraries and first ensure that calling top level layers are migrated before merging these patches. That would avoid mixing execution models. Internal usage of Eventlet in python-keystonclient seems limited to certificat signifing functions, all the rest inherit from importer environment.\n\nThoughts?\n\n[1] https://codesearch.openstack.org/?q\u003dimport%20keystoneclient\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d", "revId": "ee58ef4d0084b01a1bbfe193626642bfe57b80c7", "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" + }, + { + "unresolved": false, + "key": { + "uuid": "095dc449_e1e529a7", + "filename": "/PATCHSET_LEVEL", + "patchSetId": 1 + }, + "lineNbr": 0, + "author": { + "id": 28522 + }, + "writtenOn": "2025-08-29T13:37:28Z", + "side": 1, + "message": "That\u0027s why I submitted this patch as WIP.\n\nMagnum seems, at least, yet requiring Eventlet and python-keystoneclient, so IMO that\u0027s a blocker:\nhttps://codesearch.openstack.org/?q\u003deventlet\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003dopenstack/magnum", + "parentUuid": "072a51f6_b008274f", + "revId": "ee58ef4d0084b01a1bbfe193626642bfe57b80c7", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543" } ] } \ No newline at end of file