)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"81a6bee961518e075fd70c5d59ac13d5df91d468","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a4d743b6_569f10a4","updated":"2025-05-15 10:37:12.000000000","message":"Patchset 4 reproduces the error from the bug report see the test results [1] from the python  3.12 job [2], but the fix in the follow-up patch fixes all of them, see the tests results [3] from the python 3.12 job there [4]. In the next patchset I\u0027ll comment out the failing tests and re-enable them in the follow-up.\n\n[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a54/openstack/a545d18107be422ea7fc7ea82db8566d/testr_results.html\n[2] https://zuul.opendev.org/t/openstack/build/a545d18107be422ea7fc7ea82db8566d\n[3] https://564427108cabbad8b493-532d928235cdca0f1f7d2e9417dacd2f.ssl.cf1.rackcdn.com/openstack/8002cdd0adc34c129aab04f4e3e5f416/testr_results.html\n[4] https://zuul.opendev.org/t/openstack/build/8002cdd0adc34c129aab04f4e3e5f416","commit_id":"f89a0e55a31706110a9c2bbe823c451ee353de49"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"1e5dcae713ec5a44233d946fc3887f8f9b4b5e0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4ac6f98d_eeadd085","updated":"2025-05-22 17:55:00.000000000","message":"LGTM, thanks Joan","commit_id":"7d90a079b06a5367704cf696259b08cf192024c9"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f492ec5387d295819c185d193b7d4e1a3360944c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"faadeee1_5e4726a0","updated":"2025-05-15 16:16:25.000000000","message":"let wait to merge this untile we are happy with the follow up (im goign to reveiw that next) as i woudl liek to minise the time we have the temproy assert chagnes so we can come back an +w this wonce we have +w\u0027d the follow up","commit_id":"7d90a079b06a5367704cf696259b08cf192024c9"}],"watcher/tests/common/test_cinder_helper.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"1e5dcae713ec5a44233d946fc3887f8f9b4b5e0c","unresolved":true,"context_lines":[{"line_number":154,"context_line":"        volume_type \u003d self.fake_volume_type()"},{"line_number":155,"context_line":"        cinder_util.cinder.volume_types.list.return_value \u003d [volume_type]"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        # the logging in this function has a bug, temporarily changing the"},{"line_number":158,"context_line":"        # assert to catch the exception"},{"line_number":159,"context_line":"        # https://review.opendev.org/c/openstack/watcher/+/822559 merges"},{"line_number":160,"context_line":"        self.assertRaises(TypeError, cinder_util.migrate,"}],"source_content_type":"text/x-python","patch_set":5,"id":"ffb3c9d4_dcac44e4","line":157,"range":{"start_line":157,"start_character":8,"end_line":157,"end_character":10},"updated":"2025-05-22 17:55:00.000000000","message":"something that we usually do is to add the TODO(user): revert once X merges...\nit is easy to grep afterwards and avoid leaving things behind\nbut this is fine, you are proposing here and already reverting in another patch","commit_id":"7d90a079b06a5367704cf696259b08cf192024c9"}]}