)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"eaeb613532154f5c1014b2fcf40d72fe4cc7ce9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e946364b_be248b22","updated":"2022-05-17 22:22:34.000000000","message":"This will pass pylint, but I\u0027m not sure it will actually work.  Stephen has a patch that refactors this file to use the non-legacy enginefacade, but it\u0027s kind of high up in the dependency chain and hasn\u0027t merged yet: https://review.opendev.org/c/openstack/cinder/+/837540\n\nSo we may want to hold off on this (though it does make it more important to get the other patches merged soon).","commit_id":"52e0c48e1b752bd68eaa4be312332f0b629bffc1"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1198e72258025f085f9ac739846cbb91da323f08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e7b7ab86_c9b06602","updated":"2022-05-18 16:18:27.000000000","message":"Yeah, this is borked until [1] merges. There mustn\u0027t be test coverage for this function to say I missed this. If you want to fix this immediately, we\u0027d need to partially revert bc9397073e and make all those \u0027_sync_foo\u0027 helpers optionally take a \u0027session\u0027 argument (as opposed to it being mandatory). I\u0027d *much* rather just get [1] merged faster though\n\n[1] https://review.opendev.org/c/openstack/cinder/+/837540","commit_id":"52e0c48e1b752bd68eaa4be312332f0b629bffc1"}]}