)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"4ea187c51223da6af579951249f07d0cbfb57a9c","unresolved":false,"context_lines":[{"line_number":4998,"context_line":"    def add_aggregate_host(self, context, host, slave_info\u003dNone,"},{"line_number":4999,"context_line":"                           aggregate\u003dNone, aggregate_id\u003dNone):"},{"line_number":5000,"context_line":"        \"\"\"Notify hypervisor of change (for hypervisor pools).\"\"\""},{"line_number":5001,"context_line":"        if not aggregate:"},{"line_number":5002,"context_line":"            aggregate \u003d self.conductor_api.aggregate_get(context, aggregate_id)"},{"line_number":5003,"context_line":""},{"line_number":5004,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Fo%2Bo%3D","side":"PARENT","line":5001,"updated":"2013-10-11 21:40:05.000000000","message":"It looks like aggregate_id was something meant to be deprecated before... and that aggregate will always !\u003d None now.  Maybe a patch should be submitted to fix this first.","commit_id":"27802731748ebae2c77ef74d684641b2d811a659"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"299549feb14e4689eb6eead97223ad241886cd9e","unresolved":false,"context_lines":[{"line_number":4998,"context_line":"    def add_aggregate_host(self, context, host, slave_info\u003dNone,"},{"line_number":4999,"context_line":"                           aggregate\u003dNone, aggregate_id\u003dNone):"},{"line_number":5000,"context_line":"        \"\"\"Notify hypervisor of change (for hypervisor pools).\"\"\""},{"line_number":5001,"context_line":"        if not aggregate:"},{"line_number":5002,"context_line":"            aggregate \u003d self.conductor_api.aggregate_get(context, aggregate_id)"},{"line_number":5003,"context_line":""},{"line_number":5004,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FkW0%3D","side":"PARENT","line":5001,"in_reply_to":"AAAATH%2F%2Fo%2Bo%3D","updated":"2013-10-14 13:01:54.000000000","message":"But it was a change in 2.14/2.15 so I think we still need this for compatibility reasons, until we move to 3.x","commit_id":"27802731748ebae2c77ef74d684641b2d811a659"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"4ea187c51223da6af579951249f07d0cbfb57a9c","unresolved":false,"context_lines":[{"line_number":5018,"context_line":"    def remove_aggregate_host(self, context, host, slave_info\u003dNone,"},{"line_number":5019,"context_line":"                              aggregate\u003dNone, aggregate_id\u003dNone):"},{"line_number":5020,"context_line":"        \"\"\"Removes a host from a physical hypervisor pool.\"\"\""},{"line_number":5021,"context_line":"        if not aggregate:"},{"line_number":5022,"context_line":"            aggregate \u003d self.conductor_api.aggregate_get(context, aggregate_id)"},{"line_number":5023,"context_line":""},{"line_number":5024,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Fo%2Bg%3D","side":"PARENT","line":5021,"updated":"2013-10-11 21:40:05.000000000","message":"Same.","commit_id":"27802731748ebae2c77ef74d684641b2d811a659"}]}