New patch submitted by Federico Simoncelli (fsimonce@redhat.com)
You can review this change at: http://gerrit.usersys.redhat.com/924
commit 5c34b8a2b26533c19a336acd934054bc9254f02f Author: Federico Simoncelli fsimonce@redhat.com Date: Fri Sep 9 17:01:26 2011 +0000
BZ#733669 Return specific error for invalid host
Change-Id: Iaf0ae2d62772b01d05d9d5e62d2433c1758d69b1
diff --git a/vdsm/clientIF.py b/vdsm/clientIF.py index 3dde1c0..fdc401b 100644 --- a/vdsm/clientIF.py +++ b/vdsm/clientIF.py @@ -778,7 +778,7 @@ class clientIF:
if not utils.validLocalHostname(): self.log.error('Migration failed: local hostname is not correct') - return errCode['createErr'] + return errCode['wrongHost']
response = self.create(params) if response['status']['code']: diff --git a/vdsm/define.py b/vdsm/define.py index ca30ddf..78e7563 100644 --- a/vdsm/define.py +++ b/vdsm/define.py @@ -36,6 +36,7 @@ errCode = { 'noVM': {'status': {'code': 1, 'message': 'Virtual machine d 'ticketErr': {'status': {'code':18, 'message': 'Error while setting spice ticket'}}, 'nonresp': {'status': {'code':19, 'message': 'Guest agent non-responsive'}}, # codes 20-29 are reserved for add/delNetwork + 'wrongHost': {'status': {'code':39, 'message': 'Migration destination has an invalid hostname'}}, 'unavail': {'status': {'code':40, 'message': 'Resource unavailable'}}, 'changeDisk': {'status': {'code':41, 'message': 'Failed to change disk image'}}, 'destroyErr': {'status': {'code':42, 'message': 'Virtual machine destroy error'}},