)]}'
{"nova/tests/api/ec2/test_cloud.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"903085e19a958b572f1b40c1c2351758348ffeae","unresolved":false,"context_lines":[{"line_number":2115,"context_line":"        self.stubs.Set(fake._FakeImageService, \u0027show\u0027, fake_show)"},{"line_number":2116,"context_line":""},{"line_number":2117,"context_line":"        def fake_block_device_mapping_get_all_by_instance(context, inst_id):"},{"line_number":2118,"context_line":"            from nova.db.sqlalchemy import models"},{"line_number":2119,"context_line":""},{"line_number":2120,"context_line":"            return [models.BlockDeviceMapping("},{"line_number":2121,"context_line":"                id\u003d1,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAOn%2F%2F13k%3D","line":2118,"updated":"2013-04-18 16:38:30.000000000","message":"I think this import should be at the top.","commit_id":"3b0de63089348c76271cf7a73f215f667853b44f"}],"nova/tests/api/openstack/compute/test_server_actions.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"903085e19a958b572f1b40c1c2351758348ffeae","unresolved":false,"context_lines":[{"line_number":785,"context_line":"        image_service.create(None, original_image)"},{"line_number":786,"context_line":""},{"line_number":787,"context_line":"        def fake_block_device_mapping_get_all_by_instance(context, inst_id):"},{"line_number":788,"context_line":"            from nova.db.sqlalchemy import models"},{"line_number":789,"context_line":""},{"line_number":790,"context_line":"            return [models.BlockDeviceMapping("},{"line_number":791,"context_line":"                volume_id\u003d_fake_id(\u0027a\u0027),"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAOn%2F%2F13s%3D","line":788,"updated":"2013-04-18 16:38:30.000000000","message":"I don\u0027t see a reason to have this import local, rather than at the top with the others.","commit_id":"3b0de63089348c76271cf7a73f215f667853b44f"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"107f604be8244b38b1566914bc26b4e3e9eff338","unresolved":false,"context_lines":[{"line_number":773,"context_line":"        image_service.create(None, original_image)"},{"line_number":774,"context_line":""},{"line_number":775,"context_line":"        def fake_block_device_mapping_get_all_by_instance(context, inst_id):"},{"line_number":776,"context_line":"            return [db.sqlalchemy.models.BlockDeviceMapping("},{"line_number":777,"context_line":"                volume_id\u003d_fake_id(\u0027a\u0027),"},{"line_number":778,"context_line":"                virtual_name\u003dNone,"},{"line_number":779,"context_line":"                volume_size\u003d1,"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAOn%2F%2Ft5I%3D","line":776,"updated":"2013-04-26 13:13:07.000000000","message":"I appreciate the test coverage, but I think we need to avoid leaking the sqlalchemy models even further.  There is a very strong chance that in this cycle we will be fixing this so the db API doesn\u0027t return these models anymore.  This kind of thing will have to be changed at that point, so I\u0027d rather avoid it now.\n\nI don\u0027t necessarily have a bright idea here, other than defining your own custom type here that won\u0027t allow del, with a comment explaning why you\u0027re not using dict.","commit_id":"a39d004536b1950e14f5244e155d3ba950a5d95e"}]}