)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"a92da3bd6cc8c67576d6069469f54f02a4570d0a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"28185ebb_7b4b23fd","updated":"2023-02-03 17:05:52.000000000","message":"This looks OK to me to backport. On the other hand there is file conflict, which is not indicated in the commit message. The conflict is due to the bug reproducer test-only patch [1] is not backported.\n\n[1] https://review.opendev.org/c/openstack/nova/+/847000","commit_id":"5932ea76152aac19cd392b624cd74de3c664933a"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"c8bc0158c789bf7ea17fbc98beb98e1e589adb3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"51ac0562_8d294ddf","in_reply_to":"28185ebb_7b4b23fd","updated":"2023-02-03 17:09:16.000000000","message":"I\u0027ve cherry picked the above mentioned test-only patch and re-applied the cherry pick of this patch, which is now a clean cherry pick from stable/zed (was master at merge time). LGTM.","commit_id":"5932ea76152aac19cd392b624cd74de3c664933a"},{"author":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"change_message_id":"c45f3208cda6bf99db5b3148d8f7deea4cc4be73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"02086a76_517ed8fc","updated":"2023-02-13 12:26:10.000000000","message":"Hi Elod, I\u0027m a bit confused because the diff between my patch (1) and yours (2) says that the only thing that changed is the commit message - https://review.opendev.org/c/openstack/nova/+/867989/1..2 - and the message itself is unchanged. Unless I am missing something I assume that you forgot to add/apply something. In any case I will take a another look at this backport to make sure your concerns are addressed.","commit_id":"f2121c7551c0a2de134bf469f66c058ffbdd628d"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"a63554a76dfdd8b92705aa745a98c4d8d0f1ad9d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0d15235c_f38a2330","in_reply_to":"02086a76_517ed8fc","updated":"2023-02-13 13:21:54.000000000","message":"Hi Edward. The difference is that I have cherry picked the related test reproducer patch [1] as well. With that, the cherry pick became a \"clean\" cherry pick, no conflict needed to be resolved. The difference in the commit message is that which patch is the \u0027parent\u0027 patch (as the parent became the mentioned test patch [1]).\n\nAnd about why we don\u0027t see any other diff between Patch Set 1 and Patch Set 2: it is a \u0027gerrit feature\u0027, as basically the code became the same. if you compare them with git, there will be difference, and it will be exactly the test reproducer patch.\n\nAll beyond these, could you re-apply please the cherry pick on top of the test reprodcer patch [1], and remove the \u0027conflict\u0027 part from the commit message? Thanks in advance o:)\n\n[1] https://review.opendev.org/c/openstack/nova/+/872663","commit_id":"f2121c7551c0a2de134bf469f66c058ffbdd628d"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"ca4d9e37ec66b798b5918efe631fa6404a875a55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5c2a2247_d74239d7","updated":"2023-03-02 15:59:49.000000000","message":"Clean backport straight from master when it was zed. Objects calls were existing, no regression found. LGTM.","commit_id":"7934b9ec57d7060fbcf27706aa98ebf5a83f920a"},{"author":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"change_message_id":"8a09fe760516471dc5535b544d15f29ab4729653","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1115ec06_142c9f5e","updated":"2023-02-13 20:09:19.000000000","message":"Thanks for clearing that up Elod, makes sense now. I have resubmitted the backport on top of your patch and indeed no merge conflicts to resolve. I also now see that gerrit recognising the dependency chain i.e. with the \"Relation Chain\" and \"Submitted Together\" fields. So now I will wait for your patch to land first.","commit_id":"7934b9ec57d7060fbcf27706aa98ebf5a83f920a"},{"author":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"change_message_id":"501ba7b363720b5e293ffc78cfcf6314928d7ca5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4819de56_b8b2df93","updated":"2023-03-11 09:05:14.000000000","message":"failure possibly a result of https://wiki.openstack.org/wiki/Infrastructure_Status:\n\n2023-03-09 17:25:25 UTC Yesterday\u0027s change to Gerrit configs to use submit-requirements had a boolean logic bug. This has now been corrected and any changes that did not merge as a result can be rechecked. We have reenqueued the changes we identified as being affected.\n\nTherefore I will try a recheck.","commit_id":"7934b9ec57d7060fbcf27706aa98ebf5a83f920a"},{"author":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"change_message_id":"a070f51d2a6f1bef665266e6a3045565a7f2dc78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"55aa05d8_54d910ed","updated":"2023-03-11 09:05:26.000000000","message":"recheck","commit_id":"7934b9ec57d7060fbcf27706aa98ebf5a83f920a"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"34714f95dd679a13644ab17cc5d84ca29187d909","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8f9ab3b3_20f957be","updated":"2023-03-07 12:56:04.000000000","message":"recheck - nova-ceph-multistore TIMED_OUT","commit_id":"7934b9ec57d7060fbcf27706aa98ebf5a83f920a"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"701a8e4a36a8265433c1c320b6789c96ede4f790","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0fb8f9a0_2aecdf5f","in_reply_to":"1115ec06_142c9f5e","updated":"2023-02-14 12:02:39.000000000","message":"No problem, thanks for the update! The patch looks good to me now.","commit_id":"7934b9ec57d7060fbcf27706aa98ebf5a83f920a"}]}