)]}'
{"nova/api/ec2/cloud.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"763f3ed0cd1a42fd6adb5944d26a4cd757c22685","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        not_available_zones \u003d []"},{"line_number":257,"context_line":"        for zone in [service[\u0027available_zones\u0027] for service in"},{"line_number":258,"context_line":"                    disabled_services if not service[\u0027available_zones\u0027] in"},{"line_number":259,"context_line":"                    available_zones]:"},{"line_number":260,"context_line":"            if not zone in not_available_zones:"},{"line_number":261,"context_line":"                not_available_zones.append(zone)"},{"line_number":262,"context_line":"        return (available_zones, not_available_zones)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FzX0%3D","line":259,"updated":"2013-01-04 16:18:31.000000000","message":"This is kind of dense.  Maybe split into separate lines?","commit_id":"ab3d96f481c6e9ff7d8dff0029f02b65f111c383"},{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"763f3ed0cd1a42fd6adb5944d26a4cd757c22685","unresolved":false,"context_lines":[{"line_number":257,"context_line":"        for zone in [service[\u0027available_zones\u0027] for service in"},{"line_number":258,"context_line":"                    disabled_services if not service[\u0027available_zones\u0027] in"},{"line_number":259,"context_line":"                    available_zones]:"},{"line_number":260,"context_line":"            if not zone in not_available_zones:"},{"line_number":261,"context_line":"                not_available_zones.append(zone)"},{"line_number":262,"context_line":"        return (available_zones, not_available_zones)"},{"line_number":263,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FzX8%3D","line":260,"updated":"2013-01-04 16:18:31.000000000","message":"\"zone not in\" is more readable","commit_id":"ab3d96f481c6e9ff7d8dff0029f02b65f111c383"}],"nova/availability_zones.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"763f3ed0cd1a42fd6adb5944d26a4cd757c22685","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    services \u003d [dict(service.iteritems()) for service in services]"},{"line_number":43,"context_line":"    metadata \u003d db.aggregate_host_get_by_metadata_key(context,"},{"line_number":44,"context_line":"            key\u003d\u0027availability_zone\u0027)"},{"line_number":45,"context_line":"    all_defaults \u003d True"},{"line_number":46,"context_line":"    for service in services:"},{"line_number":47,"context_line":"        az \u003d CONF.internal_service_availability_zone"},{"line_number":48,"context_line":"        if service[\u0027topic\u0027] \u003d\u003d \"compute\":"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FzYA%3D","line":45,"updated":"2013-01-04 16:18:31.000000000","message":"all_defaults appears to be unused.","commit_id":"ab3d96f481c6e9ff7d8dff0029f02b65f111c383"}],"nova/db/api.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"c1a680a10473193ffd406df65a1f5aec74ff0065","unresolved":false,"context_lines":[{"line_number":690,"context_line":"    return rv"},{"line_number":691,"context_line":""},{"line_number":692,"context_line":""},{"line_number":693,"context_line":"def instance_update_and_get_original(context, instance_uuid, values):"},{"line_number":694,"context_line":"    \"\"\"Set the given properties on an instance and update it. Return"},{"line_number":695,"context_line":"    a shallow copy of the original instance reference, as well as the"},{"line_number":696,"context_line":"    updated one."}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2Fwdg%3D","line":693,"updated":"2013-01-07 14:19:59.000000000","message":"Should this take an update_cells argument like the others?","commit_id":"fe9c18ae1ae0d0a14b4d93932ef413b83f5a7013"}],"nova/tests/api/ec2/test_cloud.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"763f3ed0cd1a42fd6adb5944d26a4cd757c22685","unresolved":false,"context_lines":[{"line_number":714,"context_line":"                {\u0027name\u0027: \u0027agg2\u0027}, {\u0027availability_zone\u0027: \u0027zone2\u0027})"},{"line_number":715,"context_line":"        db.aggregate_host_add(self.context, agg.id, \u0027host2_zones\u0027)"},{"line_number":716,"context_line":"        result \u003d self.cloud.describe_availability_zones(self.context)"},{"line_number":717,"context_line":"        print result[\u0027availabilityZoneInfo\u0027]"},{"line_number":718,"context_line":"        self.assertEqual(len(result[\u0027availabilityZoneInfo\u0027]), 3)"},{"line_number":719,"context_line":"        admin_ctxt \u003d context.get_admin_context(read_deleted\u003d\"no\")"},{"line_number":720,"context_line":"        result \u003d self.cloud.describe_availability_zones(admin_ctxt,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FzYw%3D","line":717,"updated":"2013-01-04 16:18:31.000000000","message":"Do we really want these prints in the checked-in tests?","commit_id":"ab3d96f481c6e9ff7d8dff0029f02b65f111c383"}],"nova/tests/test_xenapi.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"c1a680a10473193ffd406df65a1f5aec74ff0065","unresolved":false,"context_lines":[{"line_number":2264,"context_line":"        result \u003d db.aggregate_get(self.context, aggregate[\u0027id\u0027])"},{"line_number":2265,"context_line":"        self.assertTrue(fake_init_pool.called)"},{"line_number":2266,"context_line":"        print result.metadetails"},{"line_number":2267,"context_line":"        print self.fake_metadata"},{"line_number":2268,"context_line":"        self.assertThat(self.fake_metadata,"},{"line_number":2269,"context_line":"                        matchers.DictMatches(result[\u0027metadetails\u0027]))"},{"line_number":2270,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FwfM%3D","line":2267,"updated":"2013-01-07 14:19:59.000000000","message":"Can we clean up the prints?","commit_id":"fe9c18ae1ae0d0a14b4d93932ef413b83f5a7013"}]}