)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"74888bd532f75b5ed9d6bc25fb3ac261177b8698","unresolved":false,"context_lines":[{"line_number":13,"context_line":"MigrationError will be raised. It will cause the instance to be"},{"line_number":14,"context_line":"ERROR state. This patch modify the exception raised."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug : #1289174"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ib3f1a810ed38bc42743e7572c2b278cfbfca1787"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAVn%2F%2FHGY%3D","line":16,"updated":"2014-03-09 13:04:33.000000000","message":"Closes-Bug :  \u003d\u003e Closes-Bug:","commit_id":"fad088371a3f542093f8393e60d3025b7b5a56e8"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"6a85f7cde553466d1278c0610557d28a599502e9","unresolved":false,"context_lines":[{"line_number":13,"context_line":"MigrationError will be raised. It will cause the instance to be"},{"line_number":14,"context_line":"ERROR state. This patch modify the exception raised."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug : #1289174"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ib3f1a810ed38bc42743e7572c2b278cfbfca1787"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAVn%2F%2FFgM%3D","line":16,"in_reply_to":"AAAAVn%2F%2FHGY%3D","updated":"2014-03-10 02:15:23.000000000","message":"Done","commit_id":"fad088371a3f542093f8393e60d3025b7b5a56e8"}],"nova/compute/manager.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1c143d434bb4b809bd243ce92e99fec17490e85d","unresolved":false,"context_lines":[{"line_number":3154,"context_line":""},{"line_number":3155,"context_line":"        if not instance[\u0027host\u0027]:"},{"line_number":3156,"context_line":"            msg \u003d _(\u0027Instance has no source host\u0027)"},{"line_number":3157,"context_line":"            raise exception.NoValidHost(reason\u003dmsg)"},{"line_number":3158,"context_line":""},{"line_number":3159,"context_line":"        same_host \u003d instance[\u0027host\u0027] \u003d\u003d self.host"},{"line_number":3160,"context_line":"        if same_host and not CONF.allow_resize_to_same_host:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FeO4%3D","line":3157,"updated":"2014-04-14 11:54:49.000000000","message":"instance[\u0027host\u0027] is source host, not destination host. When this exception happens, that would be unexpected I guess. So previous behavior seems right.","commit_id":"dd14b8c2cc672e82634154be067bd1ecb93f5ad5"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"b39ce8614ad4782cc6fb968fbda79b8cdeceeb3b","unresolved":false,"context_lines":[{"line_number":3154,"context_line":""},{"line_number":3155,"context_line":"        if not instance[\u0027host\u0027]:"},{"line_number":3156,"context_line":"            msg \u003d _(\u0027Instance has no source host\u0027)"},{"line_number":3157,"context_line":"            raise exception.NoValidHost(reason\u003dmsg)"},{"line_number":3158,"context_line":""},{"line_number":3159,"context_line":"        same_host \u003d instance[\u0027host\u0027] \u003d\u003d self.host"},{"line_number":3160,"context_line":"        if same_host and not CONF.allow_resize_to_same_host:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FjMA%3D","line":3157,"in_reply_to":"AAAAW3%2F%2FeO4%3D","updated":"2014-04-19 16:41:30.000000000","message":"+1","commit_id":"dd14b8c2cc672e82634154be067bd1ecb93f5ad5"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1c143d434bb4b809bd243ce92e99fec17490e85d","unresolved":false,"context_lines":[{"line_number":3157,"context_line":"            raise exception.NoValidHost(reason\u003dmsg)"},{"line_number":3158,"context_line":""},{"line_number":3159,"context_line":"        same_host \u003d instance[\u0027host\u0027] \u003d\u003d self.host"},{"line_number":3160,"context_line":"        if same_host and not CONF.allow_resize_to_same_host:"},{"line_number":3161,"context_line":"            msg \u003d _(\u0027destination same as source!\u0027)"},{"line_number":3162,"context_line":"            raise exception.NoValidHost(reason\u003dmsg)"},{"line_number":3163,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FeMI%3D","line":3160,"updated":"2014-04-14 11:54:49.000000000","message":"At API layer, innore_hosts filter is set like the following:\n\n 2433         filter_properties \u003d {\u0027ignore_hosts\u0027: []}\n 2434\n 2435         if not CONF.allow_resize_to_same_host:\n 2436             filter_properties[\u0027ignore_hosts\u0027].append(instance[\u0027host\u0027])\n\nThis exception happens, that means the above filter did not work correctly. I feel previous code seems right.","commit_id":"dd14b8c2cc672e82634154be067bd1ecb93f5ad5"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"b39ce8614ad4782cc6fb968fbda79b8cdeceeb3b","unresolved":false,"context_lines":[{"line_number":3157,"context_line":"            raise exception.NoValidHost(reason\u003dmsg)"},{"line_number":3158,"context_line":""},{"line_number":3159,"context_line":"        same_host \u003d instance[\u0027host\u0027] \u003d\u003d self.host"},{"line_number":3160,"context_line":"        if same_host and not CONF.allow_resize_to_same_host:"},{"line_number":3161,"context_line":"            msg \u003d _(\u0027destination same as source!\u0027)"},{"line_number":3162,"context_line":"            raise exception.NoValidHost(reason\u003dmsg)"},{"line_number":3163,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FjLY%3D","line":3160,"in_reply_to":"AAAAW3%2F%2FeMI%3D","updated":"2014-04-19 16:41:30.000000000","message":"I do not know why do we need to have such checking in compute side, this checking seems useless as I cannot think up a case can cause this error.\n\nThe API checking can make sure \"failure fast\".","commit_id":"dd14b8c2cc672e82634154be067bd1ecb93f5ad5"}]}
