)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9ae22c84b8545e39b90bc1d2892cac23a72655e5","unresolved":false,"context_lines":[{"line_number":235,"context_line":"            return function(self, context, *args, **kwargs)"},{"line_number":236,"context_line":"        except Exception:"},{"line_number":237,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":238,"context_line":"                if \u0027migration\u0027 in kwargs:"},{"line_number":239,"context_line":"                    migration \u003d kwargs[\u0027migration\u0027]"},{"line_number":240,"context_line":"                    status \u003d migration.status"},{"line_number":241,"context_line":"                    if status not in [\u0027migrating\u0027, \u0027post-migrating\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F90Fo%3D","line":238,"updated":"2014-04-01 23:27:08.000000000","message":"Is this needed if you\u0027re decorating with expects_fun_args?  See line 272 where reverts_task_state doesn\u0027t check if instance is in kwargs for that decorator.","commit_id":"1d1f6e7564725a9b878fc5686111a8066c3fd686"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9ae22c84b8545e39b90bc1d2892cac23a72655e5","unresolved":false,"context_lines":[{"line_number":237,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":238,"context_line":"                if \u0027migration\u0027 in kwargs:"},{"line_number":239,"context_line":"                    migration \u003d kwargs[\u0027migration\u0027]"},{"line_number":240,"context_line":"                    status \u003d migration.status"},{"line_number":241,"context_line":"                    if status not in [\u0027migrating\u0027, \u0027post-migrating\u0027]:"},{"line_number":242,"context_line":"                        return"},{"line_number":243,"context_line":"                    migration.status \u003d \u0027error\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F90HY%3D","line":240,"updated":"2014-04-01 23:27:08.000000000","message":"Do you no longer need to check if migration is an object?","commit_id":"1d1f6e7564725a9b878fc5686111a8066c3fd686"}],"nova/tests/compute/test_compute.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9ae22c84b8545e39b90bc1d2892cac23a72655e5","unresolved":false,"context_lines":[{"line_number":3786,"context_line":"    def test_state_revert(self):"},{"line_number":3787,"context_line":"        # ensure that task_state is reverted after a failed operation."},{"line_number":3788,"context_line":"        migration \u003d migration_obj.Migration()"},{"line_number":3789,"context_line":"        migration.instance_uuid \u003d \u0027\u0027"},{"line_number":3790,"context_line":"        migration.new_instance_type_id \u003d \u00271\u0027"},{"line_number":3791,"context_line":""},{"line_number":3792,"context_line":"        actions \u003d ["}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F90HA%3D","line":3789,"updated":"2014-04-01 23:27:08.000000000","message":"Use a fake uuid, not just empty string since that wouldn\u0027t happen.","commit_id":"1d1f6e7564725a9b878fc5686111a8066c3fd686"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"be54e9d2913b3d6b2742ed85a313091ac0da1731","unresolved":false,"context_lines":[{"line_number":3786,"context_line":"    def test_state_revert(self):"},{"line_number":3787,"context_line":"        # ensure that task_state is reverted after a failed operation."},{"line_number":3788,"context_line":"        migration \u003d migration_obj.Migration()"},{"line_number":3789,"context_line":"        migration.instance_uuid \u003d \u0027b48316c5-71e8-45e4-9884-6c78055b9b13\u0027"},{"line_number":3790,"context_line":"        migration.new_instance_type_id \u003d \u00271\u0027"},{"line_number":3791,"context_line":""},{"line_number":3792,"context_line":"        actions \u003d ["}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWn%2F%2F4B4%3D","line":3789,"updated":"2014-04-08 19:48:23.000000000","message":"nit: I\u0027d just use uuidutils.generate_uuid() here.","commit_id":"72cd430e971589f7872a9d4e8f173c68b4955375"}]}