Change in vdsm[master]: BZ#733669 Return specific error for invalid host

danken at redhat.com danken at redhat.com
Wed Sep 14 05:32:26 UTC 2011


Dan Kenigsberg has posted comments on this change.

Change subject: BZ#733669 Return specific error for invalid host
......................................................................


Patch Set 4: I would prefer that you didn't submit this

(1 inline comment)

....................................................
Commit Message
Line 7: BZ#733669 Return specific error for invalid host
Sorry for only noticing this now, but the commit message is not good enough for a piece of code we would like to revert when we start using virDomainMigrateToURI2. (well, almost revert. I'd like to keep the bubbling of the error code)

Please describe the problem at had, and mention that this restriction is to be dropped

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaf0ae2d62772b01d05d9d5e62d2433c1758d69b1
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken 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