)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"01414e8991b13d3b505133e5e6954eb729189df2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a48a1ba4_d64a7dbb","updated":"2023-03-20 11:14:16.000000000","message":"Good start. Two requests inline. Could we also get a release note, please?","commit_id":"35a35108a15e0f49b57d7f84dc298b874e36a19d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"17d01d72502227716817b288eefd5423a1037ef9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"39bde837_73304903","updated":"2023-03-20 11:16:23.000000000","message":"You also need to document this new parameter in the docs [1]\n\n[1] doc/source/user/proxies/image_v2.rst","commit_id":"35a35108a15e0f49b57d7f84dc298b874e36a19d"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"42e639b99dc0b76276d6c2d8a0520d2b8415a1eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"0839f42e_c8c93141","updated":"2023-04-05 07:12:56.000000000","message":"re-approving already processed change","commit_id":"27e66984bf699cce8b34016595ee156281aa4ede"}],"openstack/image/v2/cache.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"01414e8991b13d3b505133e5e6954eb729189df2","unresolved":true,"context_lines":[{"line_number":35,"context_line":"                                  list_type\u003dCachedImage)"},{"line_number":36,"context_line":"    queued_images \u003d resource.Body(\u0027queued_images\u0027, type\u003dlist)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def queue(self, session, image):"},{"line_number":39,"context_line":"        image_id \u003d resource.Resource._get_id(image)"},{"line_number":40,"context_line":"        url \u003d utils.urljoin(self.base_path, image_id)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f8473ce4_2c1b5062","line":38,"updated":"2023-03-20 11:14:16.000000000","message":"Can you add a docstring, please?","commit_id":"35a35108a15e0f49b57d7f84dc298b874e36a19d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"01414e8991b13d3b505133e5e6954eb729189df2","unresolved":true,"context_lines":[{"line_number":35,"context_line":"                                  list_type\u003dCachedImage)"},{"line_number":36,"context_line":"    queued_images \u003d resource.Body(\u0027queued_images\u0027, type\u003dlist)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def queue(self, session, image):"},{"line_number":39,"context_line":"        image_id \u003d resource.Resource._get_id(image)"},{"line_number":40,"context_line":"        url \u003d utils.urljoin(self.base_path, image_id)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"56dcef3b_14637e49","line":38,"updated":"2023-03-20 11:14:16.000000000","message":"This should probably have a microversion parameter.\n\n  def queue(self, session, image, *, microversion\u003dNone):\n      if microversion is None:\n          microversion \u003d self._get_microversion(session, action\u003d\u0027commit\u0027)\n      ...\n      response \u003d session.put(url, microversion\u003dmicroversion)\n\nYou don\u0027t necessarily need the microversion parameter (though I see no reason not to provide it) but you should be passing a microversion to the \u0027session.put\u0027 call.","commit_id":"35a35108a15e0f49b57d7f84dc298b874e36a19d"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"1c9b3ed43cc4ae17d3b524519b3c189389252d83","unresolved":false,"context_lines":[{"line_number":35,"context_line":"                                  list_type\u003dCachedImage)"},{"line_number":36,"context_line":"    queued_images \u003d resource.Body(\u0027queued_images\u0027, type\u003dlist)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def queue(self, session, image):"},{"line_number":39,"context_line":"        image_id \u003d resource.Resource._get_id(image)"},{"line_number":40,"context_line":"        url \u003d utils.urljoin(self.base_path, image_id)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ef0f66d3_a6e9ccc7","line":38,"in_reply_to":"56dcef3b_14637e49","updated":"2023-03-23 12:50:48.000000000","message":"Ack","commit_id":"35a35108a15e0f49b57d7f84dc298b874e36a19d"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"1c9b3ed43cc4ae17d3b524519b3c189389252d83","unresolved":false,"context_lines":[{"line_number":35,"context_line":"                                  list_type\u003dCachedImage)"},{"line_number":36,"context_line":"    queued_images \u003d resource.Body(\u0027queued_images\u0027, type\u003dlist)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def queue(self, session, image):"},{"line_number":39,"context_line":"        image_id \u003d resource.Resource._get_id(image)"},{"line_number":40,"context_line":"        url \u003d utils.urljoin(self.base_path, image_id)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"a181fc1d_5ba4410d","line":38,"in_reply_to":"f8473ce4_2c1b5062","updated":"2023-03-23 12:50:48.000000000","message":"Done","commit_id":"35a35108a15e0f49b57d7f84dc298b874e36a19d"}]}