)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"d2e714fcde014fd014b9bb077e4c298974a57b08","unresolved":true,"context_lines":[{"line_number":18,"context_line":"assert the behaviour of the associated libvirt volume driver."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Minor conflict in nova/tests/fixtures.py around the new"},{"line_number":21,"context_line":"fake connection_info. Small adjustements in test_bug_1939545.py to"},{"line_number":22,"context_line":"account for location of fakelibvirt and removal of useless imports."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Related-Bug: #1939545"},{"line_number":25,"context_line":"Change-Id: I3f1073d70e4332737e33f3c9c00cf129d0ef76b4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"8a876626_63a22f7d","line":22,"range":{"start_line":21,"start_character":22,"end_line":22,"end_character":67},"updated":"2022-06-14 00:07:55.000000000","message":"I don\u0027t see any differences in imports from the stable/wallaby change, so it seems like this sentence is not correct?","commit_id":"79e7dc3889a1aea0c72838ffe22980004c6ee493"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"da8391c6632d23cef2536920abd82a539cd87afb","unresolved":true,"context_lines":[{"line_number":24,"context_line":"Related-Bug: #1939545"},{"line_number":25,"context_line":"Change-Id: I3f1073d70e4332737e33f3c9c00cf129d0ef76b4"},{"line_number":26,"context_line":"(cherry-picked from commit 268b7169554f871d3813d78e79861934d086484d)"},{"line_number":27,"context_line":"(cherry picked from commit 57cab3144860b2a34b661dba8290b3687d9ce365)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"f37cc114_d9001d31","line":27,"range":{"start_line":27,"start_character":27,"end_line":27,"end_character":67},"updated":"2022-06-14 00:11:28.000000000","message":"This will also need to be updated to 92433d898360478337b6b434319e4c871ab2f863 in order to merge (pass the CI cherry pick check).","commit_id":"79e7dc3889a1aea0c72838ffe22980004c6ee493"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"507d34f3706cec4141fc140e0170d181085ec2f6","unresolved":false,"context_lines":[{"line_number":17,"context_line":"connection_info listing the iscsi volume type in order to correctly"},{"line_number":18,"context_line":"assert the behaviour of the associated libvirt volume driver."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Minor conflict in nova/tests/fixtures.py around the new"},{"line_number":21,"context_line":"fake connection_info. Small adjustements in test_bug_1939545.py to"},{"line_number":22,"context_line":"account for location of fakelibvirt and removal of useless imports."},{"line_number":23,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"0c33a164_1e8f0f7d","line":20,"updated":"2022-07-14 13:00:28.000000000","message":"nit: I find it helpful to prefix these with \u0027NOTE\u0027 so it\u0027s clear that this is an addition to the original commit message. I also tend to uncomment and include the \u0027Conflicts:\u0027 section git auto-generates (and add my own \u0027Changes:\u0027 section for anything I changed that wasn\u0027t the result on a conflict)","commit_id":"02854cc4458f1a68deaf840a49f7cab7aed0cb05"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"d2e714fcde014fd014b9bb077e4c298974a57b08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"be3ae4c3_a3eead4e","updated":"2022-06-14 00:07:55.000000000","message":"Change itself looks good, fixture conflict is documented correctly in the commit message. Appears to be an extra sentence about changes that were not done (not needed) but other than that, LGTM","commit_id":"79e7dc3889a1aea0c72838ffe22980004c6ee493"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a025c326b06f7dc70792dacfcb94717133916f8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"02dd19f8_c055544e","updated":"2022-06-15 19:13:52.000000000","message":"nova-tox-validate-backport\tFAILURE\t5m 22s (non-voting)","commit_id":"79e7dc3889a1aea0c72838ffe22980004c6ee493"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"777fb4cf99f12b4c656a0576b19010e41aa02d88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"051dbf9d_7c63208b","in_reply_to":"02dd19f8_c055544e","updated":"2022-06-15 19:15:09.000000000","message":"Note: this is expected to fail bc wallaby change isn\u0027t merged yet but I think the job will still fail after wallaby merges bc of the manual \"cherry-picked\".","commit_id":"79e7dc3889a1aea0c72838ffe22980004c6ee493"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"507d34f3706cec4141fc140e0170d181085ec2f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a0589dc4_250904e5","updated":"2022-07-14 13:00:28.000000000","message":"Mostly clean backport with changes called out. Mel\u0027s comment about some of that note being incorrect are valid but meh, I know and no one will ever look at this again","commit_id":"02854cc4458f1a68deaf840a49f7cab7aed0cb05"}]}