)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"ae4b23820e5c81c3d5832e4b87a15e78d76484d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a395f34_58a719bb","updated":"2022-02-02 07:45:07.000000000","message":"Good catch!","commit_id":"dd52caf68e3f1912352041e7797cac236c54d41f"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"3a11581014cf40c429e459a34b218cf044075acf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"de04c08f_2e347515","updated":"2022-02-02 08:09:05.000000000","message":"I hope that zuul will be happy with this but for me looks good. Thx","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ab9797f7bcc3f87fe41ae08ba1f163dc19815a82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4fa0f37e_869103c6","updated":"2022-02-02 09:03:14.000000000","message":"Nice","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"18dbac59f66c190cf18989b2637df67a6c139f2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"80e58a1a_92ff57f0","updated":"2022-02-02 10:22:22.000000000","message":"No issues with the change itself (not that I\u0027m qualified to have any), but my paranoid self would really like to see maximal CI coverage for this. Can we wait until [1] lands in Nova, and then I\u0027ll push a DNM Nova patch that Depends-on this one, just to exercise this code with all the conceivable backend configurations that we can support upstream?\n\n[1] https://review.opendev.org/c/openstack/nova/+/819494","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"127af2d78df5036776f736cfe4d6960d9585c4e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5b47d720_ad784a07","updated":"2022-02-04 19:57:18.000000000","message":"Now that 0b0f40d1b308b29da537859b72080488560c23d4 has landed in Nova, https://review.opendev.org/c/openstack/nova/+/827917 is the DNM change to run the Nova gate on this patch. Thanks for your patience and collaboration!","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"716bf33e6b0c657d5a3ca67ba99bfe31d1375e70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a25e1536_2550d567","updated":"2022-02-04 21:53:10.000000000","message":"The run technically failed, but nothing related to Nova/Neutron interactions. LGTM!","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2b795c58208783d5d5867e301ae935c2e2cbd550","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"85c9a3ce_f604880f","updated":"2022-02-02 10:18:05.000000000","message":"recheck","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b6ae54191f26ba7fd0ad7fd3b9fa1c8ff66a3e79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bd40183f_fa609507","updated":"2022-02-07 11:04:42.000000000","message":"recheck","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"f7a27566827df149a78d9a9817ba42c36928b768","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b0d7d133_9c554534","updated":"2022-02-07 11:07:13.000000000","message":"waiting for CI to +W","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"32aeb6a8e58dee6778166766a09654136f4f3d82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6c74883e_d39c8524","in_reply_to":"80e58a1a_92ff57f0","updated":"2022-02-02 11:24:56.000000000","message":"Right, I\u0027ll stop it.\n\nWe have enough coverage, IMO. The problem is, as with race conditions, this is difficult to spot out and catch. This issue is randomly affecting the CI jobs. Of course, that should improve the CI health.\n\nI\u0027ll -W this patch until you test it. I\u0027ll be in touch.","commit_id":"507989fc622a6633926c4288827f482bae6054dd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c4120352ad0b83a50e6ca3dfd59dc2fb2e71898d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6959a82d_ddf99634","in_reply_to":"a25e1536_2550d567","updated":"2022-02-07 11:04:33.000000000","message":"Thanks for feedback. I\u0027ll re-activate the patch again.","commit_id":"507989fc622a6633926c4288827f482bae6054dd"}]}
