)]}'
{"nova/db/sqlalchemy/migrate_repo/versions/066_preload_instance_info_cache_table.py":[{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"0317f2c6afb6680d73fef9c82de838939429ef72","unresolved":false,"context_lines":[{"line_number":26,"context_line":"meta \u003d MetaData()"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"def upgrade(migrate_engine):"},{"line_number":30,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":31,"context_line":"    # grab tables"},{"line_number":32,"context_line":"    instance_info_caches \u003d Table(\u0027instance_info_caches\u0027, meta, autoload\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADX%2F%2F%2F8U%3D","line":29,"updated":"2012-01-05 19:17:39.000000000","message":"This migration is so complicated is almost needs a unit test.","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"de2e2dbd09120e378f7fec63b03a2737c99742eb","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        if ip_string:"},{"line_number":100,"context_line":"            ip \u003d {\u0027address\u0027: ip_string,"},{"line_number":101,"context_line":"                  \u0027type\u0027: type}"},{"line_number":102,"context_line":"            if \u0027.\u0027 in ip_string:"},{"line_number":103,"context_line":"                ip[\u0027version\u0027] \u003d 4"},{"line_number":104,"context_line":"            else:"},{"line_number":105,"context_line":"                ip[\u0027version\u0027] \u003d 6"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2B4g%3D","line":102,"updated":"2012-01-03 19:43:09.000000000","message":"::ffff:127.0.0.1—that\u0027s an IPv6 address, but this test would tag it as a v4 address.  Probably not important—I can\u0027t imagine that this would actually show up as an existing entry in the db!—but it should probably be commented on because someone else is likely to raise the issue…","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"},{"author":{"_account_id":191,"name":"Trey Morris","email":"trey.morris@rackspace.com","username":"tr3buchet"},"change_message_id":"4a813a3feb198cb118b56175685dab5e5f8f55b2","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        if ip_string:"},{"line_number":100,"context_line":"            ip \u003d {\u0027address\u0027: ip_string,"},{"line_number":101,"context_line":"                  \u0027type\u0027: type}"},{"line_number":102,"context_line":"            if \u0027.\u0027 in ip_string:"},{"line_number":103,"context_line":"                ip[\u0027version\u0027] \u003d 4"},{"line_number":104,"context_line":"            else:"},{"line_number":105,"context_line":"                ip[\u0027version\u0027] \u003d 6"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2BjQ%3D","line":102,"in_reply_to":"AAAADH%2F%2F%2B4g%3D","updated":"2012-01-04 16:39:54.000000000","message":"seems odd, i\u0027ll switch by checking for colons to mean v6 just in case it matters","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"}],"nova/network/manager.py":[{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"0317f2c6afb6680d73fef9c82de838939429ef72","unresolved":false,"context_lines":[{"line_number":785,"context_line":"        # TODO(tr3buchet): return model"},{"line_number":786,"context_line":"        return network_info"},{"line_number":787,"context_line":""},{"line_number":788,"context_line":"    def build_network_info_model(self, context, vifs, fixed_ips,"},{"line_number":789,"context_line":"                                                 instance_type):"},{"line_number":790,"context_line":"        \"\"\"Returns a NetworkInfo object containing all network information"},{"line_number":791,"context_line":"        for an instance\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADX%2F%2F%2F8Q%3D","line":788,"updated":"2012-01-05 19:17:39.000000000","message":"Where\u0027s the explicit test for this method?","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"31b38455207a4b6475cc60601fd9f1b8a08254ac","unresolved":false,"context_lines":[{"line_number":797,"context_line":"            # if rxtx_cap data are not set everywhere, set to none"},{"line_number":798,"context_line":"            try:"},{"line_number":799,"context_line":"                rxtx_cap \u003d network[\u0027rxtx_base\u0027] * instance_type[\u0027rxtx_factor\u0027]"},{"line_number":800,"context_line":"            except TypeError:"},{"line_number":801,"context_line":"                rxtx_cap \u003d None"},{"line_number":802,"context_line":"            except KeyError:"},{"line_number":803,"context_line":"                rxtx_cap \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2Bk0%3D","line":800,"updated":"2012-01-04 16:18:34.000000000","message":"Just combine these two except blocks with \u0027except (TypeError, KeyError):\u0027","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"},{"author":{"_account_id":191,"name":"Trey Morris","email":"trey.morris@rackspace.com","username":"tr3buchet"},"change_message_id":"4a813a3feb198cb118b56175685dab5e5f8f55b2","unresolved":false,"context_lines":[{"line_number":797,"context_line":"            # if rxtx_cap data are not set everywhere, set to none"},{"line_number":798,"context_line":"            try:"},{"line_number":799,"context_line":"                rxtx_cap \u003d network[\u0027rxtx_base\u0027] * instance_type[\u0027rxtx_factor\u0027]"},{"line_number":800,"context_line":"            except TypeError:"},{"line_number":801,"context_line":"                rxtx_cap \u003d None"},{"line_number":802,"context_line":"            except KeyError:"},{"line_number":803,"context_line":"                rxtx_cap \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2BkA%3D","line":800,"in_reply_to":"AAAADH%2F%2F%2Bk0%3D","updated":"2012-01-04 16:39:54.000000000","message":"sure, why not","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"}],"nova/tests/api/openstack/v2/test_servers.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"c176256325405c742325fff13d24833b5becb962","unresolved":false,"context_lines":[{"line_number":1201,"context_line":"        self.assertEqual(self.server_delete_called, True)"},{"line_number":1202,"context_line":""},{"line_number":1203,"context_line":"    @test.skip_test(\u0027test fails in python 2.6: assertRaises() as context \u0027"},{"line_number":1204,"context_line":"                    \u0027manager was added in 2.7\u0027)"},{"line_number":1205,"context_line":"    def test_delete_server_instance_while_building(self):"},{"line_number":1206,"context_line":"        req \u003d fakes.HTTPRequest.blank(\u0027/v2/fake/servers/%s\u0027 % FAKE_UUID)"},{"line_number":1207,"context_line":"        req.method \u003d \u0027DELETE\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAADX%2F%2F%2F5o%3D","line":1204,"updated":"2012-01-05 20:10:59.000000000","message":"Rebase onto master and pull these skips out; a patch to fix this problem just went in.","commit_id":"05d7b3835bcfde10b93f1190f82c1c44ab6b7e2c"}],"nova/tests/test_compute.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"de2e2dbd09120e378f7fec63b03a2737c99742eb","unresolved":false,"context_lines":[{"line_number":817,"context_line":"                          instance[\u0027uuid\u0027])"},{"line_number":818,"context_line":"        self.compute.terminate_instance(self.context, instance[\u0027uuid\u0027])"},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"    @test.skip_test(\u0027lp892005\u0027)"},{"line_number":821,"context_line":"    def test_instance_set_to_error_on_uncaught_exception(self):"},{"line_number":822,"context_line":"        \"\"\"Test that instance is set to error state when exception is raised\"\"\""},{"line_number":823,"context_line":"        instance \u003d self._create_fake_instance()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAADH%2F%2F%2B4s%3D","line":820,"updated":"2012-01-03 19:43:09.000000000","message":"Could you expand this a little bit?  It\u0027s a bit cryptic to me.","commit_id":"04dddd42ab1d92602e4a254ceeb1fb1acaa9c9dd"}]}