Change in vdsm[master]: BZ#733669 Improve error reporting for wrong local hostname

danken at redhat.com danken at redhat.com
Thu Sep 15 10:54:20 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#733669 Improve error reporting for wrong local hostname
......................................................................


BZ#733669 Improve error reporting for wrong local hostname

This patch improves the error reporting when migration fails.
It is mainly designed to return the new error 'wrongHost' to
the manager when the migration destination has a wrong local
hostname (which will cause the migration to fail without any
informative message).
Additionally it checks the local hostname during the bootstrap
process and produces a warning if needed.

In this patch:
* add local hostname check during bootstrap (report a warning)
* return the new error 'wrongHost' if migration destination has
  a wrong local hostname
* source node must return a specific error when migration fails

Change-Id: Iaf0ae2d62772b01d05d9d5e62d2433c1758d69b1
---
M vds_bootstrap/vds_bootstrap.py
M vdsm/clientIF.py
M vdsm/define.py
M vdsm/libvirtvm.py
M vdsm/vm.py
5 files changed, 43 insertions(+), 2 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  David Naori: Verified


--
To view, visit http://gerrit.usersys.redhat.com/924
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf0ae2d62772b01d05d9d5e62d2433c1758d69b1
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Naori <dnaori at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list