)]}'
{"tempest/api/volume/test_volumes_snapshots.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a5e87d6b5a50be1589ad5d14e08193f52d9718b1","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        # Delete the snapshot"},{"line_number":131,"context_line":"        self.delete_snapshot(snapshot[\u0027id\u0027])"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _volume_from_snapshot(self, extra_size\u003dNone):"},{"line_number":134,"context_line":"        src_size \u003d CONF.volume.volume_size"},{"line_number":135,"context_line":"        size \u003d src_size + extra_size"},{"line_number":136,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_351c1c76","line":133,"range":{"start_line":133,"start_character":47,"end_line":133,"end_character":51},"updated":"2018-09-03 00:31:29.000000000","message":"Default value needs to be int for L135. make it 0 as default.","commit_id":"3f4e40314eed178d28b1b20f52e58d620df38b3e"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"270b916e5b6664824bb307b129c7ca1f43edf24a","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        # Delete the snapshot"},{"line_number":131,"context_line":"        self.delete_snapshot(snapshot[\u0027id\u0027])"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _volume_from_snapshot(self, extra_size\u003dNone):"},{"line_number":134,"context_line":"        src_size \u003d CONF.volume.volume_size"},{"line_number":135,"context_line":"        size \u003d src_size + extra_size"},{"line_number":136,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_d87cff73","line":133,"range":{"start_line":133,"start_character":8,"end_line":133,"end_character":29},"updated":"2018-09-03 01:42:18.000000000","message":"better to name it as _create_volume_from_snapshot or something like this","commit_id":"3f4e40314eed178d28b1b20f52e58d620df38b3e"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"ff46714ff3f14a4114738ed026d530c5ad7276a9","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        # Delete the snapshot"},{"line_number":131,"context_line":"        self.delete_snapshot(snapshot[\u0027id\u0027])"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _volume_from_snapshot(self, extra_size\u003dNone):"},{"line_number":134,"context_line":"        src_size \u003d CONF.volume.volume_size"},{"line_number":135,"context_line":"        size \u003d src_size + extra_size"},{"line_number":136,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_fb71b5a9","line":133,"range":{"start_line":133,"start_character":47,"end_line":133,"end_character":51},"in_reply_to":"3f79a3b5_351c1c76","updated":"2018-09-03 03:43:11.000000000","message":"Done","commit_id":"3f4e40314eed178d28b1b20f52e58d620df38b3e"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"ff46714ff3f14a4114738ed026d530c5ad7276a9","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        # Delete the snapshot"},{"line_number":131,"context_line":"        self.delete_snapshot(snapshot[\u0027id\u0027])"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _volume_from_snapshot(self, extra_size\u003dNone):"},{"line_number":134,"context_line":"        src_size \u003d CONF.volume.volume_size"},{"line_number":135,"context_line":"        size \u003d src_size + extra_size"},{"line_number":136,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_9b68c1ba","line":133,"range":{"start_line":133,"start_character":8,"end_line":133,"end_character":29},"in_reply_to":"3f79a3b5_d87cff73","updated":"2018-09-03 03:43:11.000000000","message":"Done","commit_id":"3f4e40314eed178d28b1b20f52e58d620df38b3e"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"4dfc7bc2d4c0d15ad25c4f3658c4ce58eb457fd2","unresolved":false,"context_lines":[{"line_number":157,"context_line":"    def test_volume_from_snapshot(self):"},{"line_number":158,"context_line":"        # Creates a volume from a snapshot passing a size"},{"line_number":159,"context_line":"        # different from the source"},{"line_number":160,"context_line":"        self._volume_from_snapshot(extra_size\u003d1)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    @decorators.idempotent_id(\u0027053d8870-8282-4fff-9dbb-99cb58bb5e0a\u0027)"},{"line_number":163,"context_line":"    def test_volume_from_snapshot_no_size(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_bb91fd9c","line":160,"range":{"start_line":160,"start_character":13,"end_line":160,"end_character":34},"updated":"2018-09-03 03:56:32.000000000","message":"forgot to modify this:)","commit_id":"303be9739fff25bba98d0de9c8f0454e18a3ccf1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"bb3074caf65db8204934544c20c9e9a3f3180441","unresolved":false,"context_lines":[{"line_number":157,"context_line":"    def test_volume_from_snapshot(self):"},{"line_number":158,"context_line":"        # Creates a volume from a snapshot passing a size"},{"line_number":159,"context_line":"        # different from the source"},{"line_number":160,"context_line":"        self._volume_from_snapshot(extra_size\u003d1)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    @decorators.idempotent_id(\u0027053d8870-8282-4fff-9dbb-99cb58bb5e0a\u0027)"},{"line_number":163,"context_line":"    def test_volume_from_snapshot_no_size(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_bbd6dd52","line":160,"range":{"start_line":160,"start_character":13,"end_line":160,"end_character":34},"in_reply_to":"3f79a3b5_bb91fd9c","updated":"2018-09-03 04:02:50.000000000","message":"It\u0027s very early now understand me :)\nThanks","commit_id":"303be9739fff25bba98d0de9c8f0454e18a3ccf1"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"26f89185599494671e7c431da45fa36a3cc6f263","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def _create_volume_from_snapshot(self, extra_size\u003d0):"},{"line_number":134,"context_line":"        src_size \u003d CONF.volume.volume_size"},{"line_number":135,"context_line":"        size \u003d src_size + extra_size"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        src_vol \u003d self.create_volume(size\u003dsrc_size)"},{"line_number":138,"context_line":"        src_snap \u003d self.create_snapshot(src_vol[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_e6427c0f","line":135,"updated":"2018-09-03 06:58:50.000000000","message":"nit: how about changing the name to `dst_size`?","commit_id":"91d5fe0573f8374ee7170bc5ca286a7dd8c3d07d"}]}