)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4d6d84e00d8a421e328bf2531017bd55268e7ea8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bbea9933_922266a2","updated":"2024-11-06 12:17:02.000000000","message":"Should we consider adding locking around this instead of re-instantiating the key manager on every call?","commit_id":"757c55004090d2e49db99a4e6f1202b1e3a5e17c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"87256669ca1dbd297bf6fa4912d7daf98d845e03","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"67c42ba3_2a308d88","updated":"2025-07-10 15:04:52.000000000","message":"The problem is caused by the incorrect cachming mechanism in castellan which was fixed by https://review.opendev.org/c/openstack/castellan/+/952669 so we don\u0027t have to update the service code calling castellan.","commit_id":"757c55004090d2e49db99a4e6f1202b1e3a5e17c"},{"author":{"_account_id":37436,"name":"Si Snow","display_name":"silh","email":"silh@chinatelecom.cn","username":"silh"},"change_message_id":"aa03041e14aa3b319c896020001f0fec8eecb669","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"99e4d347_0c01f483","updated":"2024-11-11 06:07:12.000000000","message":"recheck - failed test cinder-plugin-ceph-tempest-mn-aa","commit_id":"757c55004090d2e49db99a4e6f1202b1e3a5e17c"},{"author":{"_account_id":37436,"name":"Si Snow","display_name":"silh","email":"silh@chinatelecom.cn","username":"silh"},"change_message_id":"b3e144b46b8bad21448cdfbdbdaf4881245cf2b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a35d5da3_641b942d","updated":"2024-11-11 07:21:49.000000000","message":"recheck - failed test openstack-tox-py39-arm64","commit_id":"757c55004090d2e49db99a4e6f1202b1e3a5e17c"},{"author":{"_account_id":37436,"name":"Si Snow","display_name":"silh","email":"silh@chinatelecom.cn","username":"silh"},"change_message_id":"59c830e773910ff2d296f8daa82826fda65adfb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"392123d7_2994cb7d","in_reply_to":"b78d61b5_2735cbe1","updated":"2024-11-08 11:54:39.000000000","message":"Fix it by adding locking: https://review.opendev.org/c/openstack/castellan/+/934421\n\nBut I think we should circumvent this problem in cinder.","commit_id":"757c55004090d2e49db99a4e6f1202b1e3a5e17c"},{"author":{"_account_id":37436,"name":"Si Snow","display_name":"silh","email":"silh@chinatelecom.cn","username":"silh"},"change_message_id":"7b732f015837b1ece83effedabdf5927a1935ddf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b78d61b5_2735cbe1","in_reply_to":"bbea9933_922266a2","updated":"2024-11-07 01:22:27.000000000","message":"Yes, you are right, I thought so at beginning. Later I checked the code of cinder, nova, and castellan-ui, and most of them directly re-instantiated the key manager on every calle. \nI think we should circumvent this problem first, and then consider solving it at the root(castllen).","commit_id":"757c55004090d2e49db99a4e6f1202b1e3a5e17c"}]}