)]}'
{"nova/tests/unit/virt/libvirt/test_driver.py":[{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"95aa3db5c3cd4582c5416426a9d3a1b398c5ca25","unresolved":false,"context_lines":[{"line_number":10440,"context_line":"                                      mkfs\u003dTrue)"},{"line_number":10441,"context_line":""},{"line_number":10442,"context_line":"    def test_create_image_initrd(self):"},{"line_number":10443,"context_line":"        kernel_id \u003d uuids.kernel_id"},{"line_number":10444,"context_line":"        ramdisk_id \u003d uuids.ramdisk_id"},{"line_number":10445,"context_line":""},{"line_number":10446,"context_line":"        kernel_fname \u003d imagecache.get_cache_fname(kernel_id)"},{"line_number":10447,"context_line":"        ramdisk_fname \u003d imagecache.get_cache_fname(ramdisk_id)"}],"source_content_type":"text/x-python","patch_set":11,"id":"dada55a8_01583989","line":10444,"range":{"start_line":10443,"start_character":8,"end_line":10444,"end_character":37},"updated":"2016-07-20 17:41:34.000000000","message":"It looks like these ids could just be inlined below since they aren\u0027t otherwise used. For example:\n\n        kernel_fname \u003d imagecache.get_cache_fname(uuids.kernel_id)\n        ramdisk_fname \u003d imagecache.get_cache_fname(uuids.ramdisk_id)","commit_id":"79d8292c4b2981c687c54c6299db1ba72aad9a38"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"93afb68d753c0efbe9386aea3dda04c23c0416ea","unresolved":false,"context_lines":[{"line_number":9390,"context_line":"            fetch_image_mock.assert_has_calls(["},{"line_number":9391,"context_line":"                mock.call(context\u003dself.context,"},{"line_number":9392,"context_line":"                          target\u003dbackfile_path,"},{"line_number":9393,"context_line":"                          image_id\u003dself.test_instance[\u0027image_ref\u0027]),"},{"line_number":9394,"context_line":"                mock.call(self.context, kernel_path, instance.kernel_id),"},{"line_number":9395,"context_line":"                mock.call(self.context, ramdisk_path, instance.ramdisk_id)"},{"line_number":9396,"context_line":"            ])"}],"source_content_type":"text/x-python","patch_set":14,"id":"bacf61ea_23a4fa55","line":9393,"range":{"start_line":9393,"start_character":35,"end_line":9393,"end_character":53},"updated":"2016-07-27 20:11:14.000000000","message":"Seems like it would  be good to clean this up too, since it\u0027s image-related and you convert the following two lines to use instance instead of self.test_instance","commit_id":"8e13a00b880bb0b694811d81f09596937b871886"}]}