)]}'
{"nova/tests/unit/compute/test_compute_api.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"3ccd85661e4a8978521444f157d04143a2d056c9","unresolved":false,"context_lines":[{"line_number":3115,"context_line":"            return obj_base.obj_make_list(context, cls(),"},{"line_number":3116,"context_line":"                    objects.BlockDeviceMapping, instance_bdms)"},{"line_number":3117,"context_line":""},{"line_number":3118,"context_line":"        def fake_image_create(_self, context, image_meta, data\u003dNone):"},{"line_number":3119,"context_line":"            self.assertThat(image_meta, matchers.DictMatches(expect_meta))"},{"line_number":3120,"context_line":""},{"line_number":3121,"context_line":"        def fake_volume_create_snapshot(self, context, volume_id, name,"}],"source_content_type":"text/x-python","patch_set":10,"id":"3f79a3b5_655a492b","line":3118,"range":{"start_line":3118,"start_character":30,"end_line":3118,"end_character":35},"updated":"2018-08-17 01:40:29.000000000","message":"Why do we need to use _self instead of self here?\nThis question comes from the following line 3121 which is at the same class.","commit_id":"b8acfd14953bbc4510678d2716bfadf7d94b5f5e"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"8e21755ca871e2bc225213243e18b8dfa2481901","unresolved":false,"context_lines":[{"line_number":3115,"context_line":"            return obj_base.obj_make_list(context, cls(),"},{"line_number":3116,"context_line":"                    objects.BlockDeviceMapping, instance_bdms)"},{"line_number":3117,"context_line":""},{"line_number":3118,"context_line":"        def fake_image_create(_self, context, image_meta, data\u003dNone):"},{"line_number":3119,"context_line":"            self.assertThat(image_meta, matchers.DictMatches(expect_meta))"},{"line_number":3120,"context_line":""},{"line_number":3121,"context_line":"        def fake_volume_create_snapshot(self, context, volume_id, name,"}],"source_content_type":"text/x-python","patch_set":10,"id":"3f79a3b5_c2ebde0f","line":3118,"range":{"start_line":3118,"start_character":30,"end_line":3118,"end_character":35},"in_reply_to":"3f79a3b5_655a492b","updated":"2018-08-20 13:47:57.000000000","message":"\u0027self\u0027 is used in line 3119. It should be a subclass of \u0027TestCase\u0027 class, not \u0027nova.image.api.API\u0027 class.\nSo _self is used here.","commit_id":"b8acfd14953bbc4510678d2716bfadf7d94b5f5e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"a5ddc9727519b076ae431ea2d86cdff19c16b278","unresolved":false,"context_lines":[{"line_number":3115,"context_line":"            return obj_base.obj_make_list(context, cls(),"},{"line_number":3116,"context_line":"                    objects.BlockDeviceMapping, instance_bdms)"},{"line_number":3117,"context_line":""},{"line_number":3118,"context_line":"        def fake_image_create(_self, context, image_meta, data\u003dNone):"},{"line_number":3119,"context_line":"            self.assertThat(image_meta, matchers.DictMatches(expect_meta))"},{"line_number":3120,"context_line":""},{"line_number":3121,"context_line":"        def fake_volume_create_snapshot(self, context, volume_id, name,"}],"source_content_type":"text/x-python","patch_set":10,"id":"3f79a3b5_4148faca","line":3118,"range":{"start_line":3118,"start_character":30,"end_line":3118,"end_character":35},"in_reply_to":"3f79a3b5_c2ebde0f","updated":"2018-08-22 03:44:11.000000000","message":"Oh, I got it.\nThanks for your explanation.","commit_id":"b8acfd14953bbc4510678d2716bfadf7d94b5f5e"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"8e21755ca871e2bc225213243e18b8dfa2481901","unresolved":false,"context_lines":[{"line_number":3116,"context_line":"                    objects.BlockDeviceMapping, instance_bdms)"},{"line_number":3117,"context_line":""},{"line_number":3118,"context_line":"        def fake_image_create(_self, context, image_meta, data\u003dNone):"},{"line_number":3119,"context_line":"            self.assertThat(image_meta, matchers.DictMatches(expect_meta))"},{"line_number":3120,"context_line":""},{"line_number":3121,"context_line":"        def fake_volume_create_snapshot(self, context, volume_id, name,"},{"line_number":3122,"context_line":"                                        description):"}],"source_content_type":"text/x-python","patch_set":10,"id":"3f79a3b5_1f760fda","line":3119,"range":{"start_line":3119,"start_character":12,"end_line":3119,"end_character":16},"updated":"2018-08-20 13:47:57.000000000","message":"Here","commit_id":"b8acfd14953bbc4510678d2716bfadf7d94b5f5e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"8be7d450b05e2bd7236e536cb7458c012f72182d","unresolved":false,"context_lines":[{"line_number":3140,"context_line":"            return obj_base.obj_make_list(context, cls(),"},{"line_number":3141,"context_line":"                    objects.BlockDeviceMapping, instance_bdms)"},{"line_number":3142,"context_line":""},{"line_number":3143,"context_line":"        def fake_image_create(_self, context, image_meta, data\u003dNone):"},{"line_number":3144,"context_line":"            self.assertThat(image_meta, matchers.DictMatches(expect_meta))"},{"line_number":3145,"context_line":""},{"line_number":3146,"context_line":"        def fake_volume_create_snapshot(self, context, volume_id, name,"}],"source_content_type":"text/x-python","patch_set":13,"id":"3f79a3b5_447205e5","line":3143,"updated":"2018-09-19 05:51:19.000000000","message":"Is _self intentional here?","commit_id":"500b1e3e7f7168e1136aecf64f4a30389b792563"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"65c99a0195aa75d743f75093b3aef4d57a6570de","unresolved":false,"context_lines":[{"line_number":3140,"context_line":"            return obj_base.obj_make_list(context, cls(),"},{"line_number":3141,"context_line":"                    objects.BlockDeviceMapping, instance_bdms)"},{"line_number":3142,"context_line":""},{"line_number":3143,"context_line":"        def fake_image_create(_self, context, image_meta, data\u003dNone):"},{"line_number":3144,"context_line":"            self.assertThat(image_meta, matchers.DictMatches(expect_meta))"},{"line_number":3145,"context_line":""},{"line_number":3146,"context_line":"        def fake_volume_create_snapshot(self, context, volume_id, name,"}],"source_content_type":"text/x-python","patch_set":13,"id":"3f79a3b5_bd551852","line":3143,"in_reply_to":"3f79a3b5_447205e5","updated":"2018-09-19 22:50:49.000000000","message":"Yes. Because \u0027self\u0027 is used in line 3144.","commit_id":"500b1e3e7f7168e1136aecf64f4a30389b792563"}]}
