)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"bc4428564072d1d16c9b7cd4b563f0bd6071e660","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2753d4c8_2cc2c313","updated":"2025-04-24 16:16:43.000000000","message":"Do we want to recheck this a bunch of times before merging, or should we go ahead and merge it? :)","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"278ed9c785e68ef305de3e5dcf1b219fe57e1318","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1c4846e6_0a1756b8","updated":"2025-04-25 15:21:04.000000000","message":"Doesn\u0027t make the problem any worse, and is using the same wait pattern we have elsewhere in these tests.  LGTM.","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"98883244a539a387b819dbdcbec1c2ccffd52e98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"52a13810_ae786d38","updated":"2025-04-25 09:16:10.000000000","message":"Let\u0027s run one more check","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"7882f1f7dcebc061296950902fe7635fd64f74ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"53340d2c_47d65a73","updated":"2025-04-25 15:24:51.000000000","message":"Looks good to go","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"1786ab71cdbe9474198c51d2078ed3dc912406fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9ebd0885_1173640e","updated":"2025-04-25 08:10:09.000000000","message":"recheck grenade failure, copy image test worked fine","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"4127328f0ca12b38d68c333b8df1468a67a9dcc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6d5750f0_60d2dd12","updated":"2025-04-25 07:02:08.000000000","message":"recheck one more loop to confirm copy image test works as expected","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f56558c4e77d1edc36fe9c31fec4cf38b97d792e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d5df62d0_af8d814c","in_reply_to":"2753d4c8_2cc2c313","updated":"2025-04-24 16:42:52.000000000","message":"This is more right than it was before, so I\u0027d just merge and see if the sporadic problems go away, but your call.","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"}],"glance/tests/functional/v2/test_images_import_locking.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"9bed0c8dc18a76a20d5e9db3e0a071e6f9636891","unresolved":true,"context_lines":[{"line_number":101,"context_line":"            for i in range(0, 10):"},{"line_number":102,"context_line":"                if \u0027running\u0027 in state:"},{"line_number":103,"context_line":"                    break"},{"line_number":104,"context_line":"                time.sleep(0.1)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        self.addDetail(\u0027Second import response\u0027,"},{"line_number":107,"context_line":"                       ttc.text_content(str(resp)))"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ed8c6c0_daf255d9","line":104,"range":{"start_line":104,"start_character":26,"end_line":104,"end_character":27},"updated":"2025-04-24 16:21:32.000000000","message":"Should we add:\n\nelse: self.fail(\u0027The data stream never started\u0027)\n\nhere? Mostly in order to hel debugging if this ever fails again?","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f56558c4e77d1edc36fe9c31fec4cf38b97d792e","unresolved":true,"context_lines":[{"line_number":101,"context_line":"            for i in range(0, 10):"},{"line_number":102,"context_line":"                if \u0027running\u0027 in state:"},{"line_number":103,"context_line":"                    break"},{"line_number":104,"context_line":"                time.sleep(0.1)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        self.addDetail(\u0027Second import response\u0027,"},{"line_number":107,"context_line":"                       ttc.text_content(str(resp)))"}],"source_content_type":"text/x-python","patch_set":1,"id":"a3875750_7059b630","line":104,"range":{"start_line":104,"start_character":26,"end_line":104,"end_character":27},"in_reply_to":"7ed8c6c0_daf255d9","updated":"2025-04-24 16:42:52.000000000","message":"We don\u0027t do that above on L77 because we\u0027ll check that the task was running. We\u0027ll check that the thing actually started running below on L131. This is really not confirming anything just trying to delay, but if you want me to add it in both places I guess I can do that.","commit_id":"caac5fc12c550a72848b0ded652d67311cb708f6"}]}