[NEW PATCH] BZ#733669 Return specific error for invalid host (via gerrit-bot)

[Federico Simoncelli]

New patch submitted by Federico Simoncelli (fsimonce(a)redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/924

commit 5c34b8a2b26533c19a336acd934054bc9254f02f
Author: Federico Simoncelli <fsimonce(a)redhat.com&gt;
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'}},