Update patch set 1

Patch Set 1:

(1 comment)

Patch-set: 1
This commit is contained in:
Gerrit User 28522 2025-08-29 13:37:28 +00:00 committed by Gerrit Code Review
parent 15c5104406
commit 49788a8a3a

View file

@ -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"
}
]
}