)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"830425fd93329e5662ecfdd5091549e747bba59a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ad103a95_fa023bf8","updated":"2023-03-17 18:21:10.000000000","message":"Ninja-approving to keep the gate moving.","commit_id":"fccd6b347e35c8a08163cf7450d5cd9ec3fad014"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"29d7e4fe80cacf9f1570b15221f680d198c823c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aec978cc_ea5c9fc3","updated":"2023-03-17 19:30:27.000000000","message":"looks like the extra specs are modified before we are doing the deepcopy\nbetter to create a local copy","commit_id":"fccd6b347e35c8a08163cf7450d5cd9ec3fad014"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"f1d0deda74fca3e7fdbbf8d4f6b232fc51971d8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"32054e02_8d56bece","updated":"2023-03-17 19:47:16.000000000","message":"I think this fix is going to be moot if we take the deepcopy() fix,\nhttps://review.opendev.org/c/openstack/cinder/+/877717\n\nI\u0027m not against it, but look: the test re-uses a ton of self.data things. The self.data.extra_specs is not special, so setting up a special treatment for it obviously masks the issue. If other tests do something to damage any of these pre-created constants, not only extra_specs, this kind of failure is going to continue.","commit_id":"b466bb3c1ba1eb0c7431fff3fa8239aec0ce6ebc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dcba02ec56503e790ed68b8b22e87f9639d8280c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e91a4f89_e064e89c","updated":"2023-03-17 20:23:44.000000000","message":"It\u0027s not going to hurt anything, so let\u0027s unclog the gate.","commit_id":"b466bb3c1ba1eb0c7431fff3fa8239aec0ce6ebc"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f82e5f148007fcd58baa33feb457f6ce834ebb18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"84bd77d6_fcd94c29","in_reply_to":"32054e02_8d56bece","updated":"2023-03-17 20:19:48.000000000","message":"Yes i agree. this isn\u0027t the best way to resolve the whole mess we\u0027re into with these tests but refactoring all of the UTs in the most optimized way can be a future effort (as mentioned in the commit message). Currently I\u0027m just trying to unblock the gate but yes, this isn\u0027t a preferred solution for all the UTs and we need to find something better there.","commit_id":"b466bb3c1ba1eb0c7431fff3fa8239aec0ce6ebc"}]}