)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b3c063b9c0c1c6a1e8dcf69da20687850262be28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3a1fcdfb_7cf8e530","updated":"2024-07-19 12:58:58.000000000","message":"@Ber1in: You mentioned on the bug that you had encountered this issue; can you leave a comment here that you have verified that the patch works?  (Obviously the unit tests/current tempest tests are not sufficient to detect this issue, though the success of the cinder-plugin-ceph-tempest job does give us some confidence that your patch does not cause a regression).  Thanks!","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"eaeae01ed4f2427026ce0e2b901711b86fe34d09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"56e33c56_0a684eb4","updated":"2024-07-19 14:46:48.000000000","message":"I ran some tests and bytearray is okay in either speed or memory consumption.","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"},{"author":{"_account_id":36978,"name":"Wu","display_name":"Berlin","email":"1652124020@qq.com","username":"Ber1in.Wu"},"change_message_id":"a3b444ef1ad250492f2d8412fd7b8c5301efebbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"226dc201_dd0fe952","updated":"2024-05-23 02:51:47.000000000","message":"recheck","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"},{"author":{"_account_id":36978,"name":"Wu","display_name":"Berlin","email":"1652124020@qq.com","username":"Ber1in.Wu"},"change_message_id":"9b036078b1f7f18b3c8258b3df2a5598e9b66487","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b4841847_88c3adae","updated":"2024-05-22 03:30:59.000000000","message":"recheck","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"},{"author":{"_account_id":36978,"name":"Wu","display_name":"Berlin","email":"1652124020@qq.com","username":"Ber1in.Wu"},"change_message_id":"bf389a80690de5bb277af18e7dfaa78267c42f42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e52d42ea_bf9b038d","updated":"2024-07-25 09:03:47.000000000","message":"recheck tempest-integrated-storage timed out, not related to this patch","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"},{"author":{"_account_id":36978,"name":"Wu","display_name":"Berlin","email":"1652124020@qq.com","username":"Ber1in.Wu"},"change_message_id":"62ffbd58993d3e574e6ccb1739f1e14bb9c3b775","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fab19ebb_a747415d","in_reply_to":"3a1fcdfb_7cf8e530","updated":"2024-07-19 13:36:39.000000000","message":"Yes I\u0027ve verified it and it\u0027s ready for production","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"}],"cinder/backup/drivers/ceph.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"d9aeabea1fa10f2ce58f8933c7a418e662ff0cbd","unresolved":true,"context_lines":[{"line_number":399,"context_line":"                    eventlet.tpool.Proxy(volume.rbd_image).discard("},{"line_number":400,"context_line":"                        offset + chunks * limit, rem)"},{"line_number":401,"context_line":"            else:"},{"line_number":402,"context_line":"                zeroes \u003d bytearray(self.chunk_size)"},{"line_number":403,"context_line":"                chunks \u003d int(length / self.chunk_size)"},{"line_number":404,"context_line":"                for chunk in range(0, chunks):"},{"line_number":405,"context_line":"                    LOG.debug(\"Writing zeroes chunk %d\", chunk)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2d3af6a9_d1efda64","line":402,"updated":"2024-07-19 14:26:13.000000000","message":"Why bytearray and not b\u0027\\0\u0027*self.chunk_size? Is it faster? Bytearray was mutable, I thought.","commit_id":"c13370cdec32ae6b74f3653981519af567be40de"}]}