)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"cb2c9462a576c319ebf26a91e4a73241af528ba9","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Related to blueprint volume-backed-server-build"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Co-Authored-By: Rajat Dhasmana \u003crajatdhasmana@gmail.com\u003e"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2fcf73e0e6f7fba47bd70fded241f2101eae6f10"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"91051172_33bdb642","line":12,"updated":"2022-02-24 19:03:03.000000000","message":"This should have:\n\n Depends-On: https://review.opendev.org/c/openstack/python-cinderclient/+/606891\n\nThe patch that adds the reimage method is not merged yet.","commit_id":"a7134889afbb503d27d86f49da340d19827690cd"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1c74410561de24a0c46941e6f6d2f47ef399553c","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Related to blueprint volume-backed-server-build"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Co-Authored-By: Rajat Dhasmana \u003crajatdhasmana@gmail.com\u003e"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2fcf73e0e6f7fba47bd70fded241f2101eae6f10"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bb6f659c_9f037954","line":12,"in_reply_to":"91051172_33bdb642","updated":"2022-02-24 20:57:48.000000000","message":"Hi Melwitt,\n\nThe cinderclient change is merged so we won\u0027t require the depends on tag anymore right?","commit_id":"a7134889afbb503d27d86f49da340d19827690cd"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"63b75dea6d4c1cca39aa103d495e05a930fd6942","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Related to blueprint volume-backed-server-build"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Co-Authored-By: Rajat Dhasmana \u003crajatdhasmana@gmail.com\u003e"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2fcf73e0e6f7fba47bd70fded241f2101eae6f10"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6523f691_dc7fe1d4","line":12,"in_reply_to":"bb6f659c_9f037954","updated":"2022-02-24 22:32:50.000000000","message":"That is correct, now that it is merged, Depends-On is no longer needed.","commit_id":"a7134889afbb503d27d86f49da340d19827690cd"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"acc9c0540404618313a66a884a94bb50505de1c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"acbffde2_2caa2b1c","updated":"2022-02-24 18:23:09.000000000","message":"I\u0027m not really the author of this, I just split it for Rajat.\n\nThis is a really simple thing we can merge ahead of the rest of the blueprint to reduce the amount of stuff that has to go with the rest of the functionality.","commit_id":"a7134889afbb503d27d86f49da340d19827690cd"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"cb2c9462a576c319ebf26a91e4a73241af528ba9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0b0dcc0d_ae7cfc81","updated":"2022-02-24 19:03:03.000000000","message":"This looks good but should be Depends-On the client patch it requires.","commit_id":"a7134889afbb503d27d86f49da340d19827690cd"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"63b75dea6d4c1cca39aa103d495e05a930fd6942","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"42dfccdc_97aed48b","updated":"2022-02-24 22:32:50.000000000","message":"Looks good and matches the new cinderclient function, LGTM","commit_id":"f320e59a8a36940528640e03986294bceb06e637"}],"nova/volume/cinder.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"cb2c9462a576c319ebf26a91e4a73241af528ba9","unresolved":true,"context_lines":[{"line_number":929,"context_line":"    def reimage_volume(self, context, volume_id, image_id,"},{"line_number":930,"context_line":"                       reimage_reserved\u003dFalse):"},{"line_number":931,"context_line":"        cinderclient(context, \u00273.68\u0027).volumes.reimage("},{"line_number":932,"context_line":"            volume_id, image_id, reimage_reserved)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bb64dd47_f12b4878","line":932,"updated":"2022-02-24 19:03:03.000000000","message":"Microversion for os-reimage API is correct:\n\nhttps://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded\u003dreimage-a-volume-detail#reimage-a-volume","commit_id":"a7134889afbb503d27d86f49da340d19827690cd"}]}