Change in vdsm[master]: sp: setDomainRegularRole is implemented by backend

danken at redhat.com danken at redhat.com
Thu Mar 27 16:36:54 UTC 2014


Dan Kenigsberg has posted comments on this change.

Change subject: sp: setDomainRegularRole is implemented by backend
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/24495/1/vdsm/storage/sp.py
File vdsm/storage/sp.py:

Line 811:             self.log.exception('migration to new master failed')
Line 812:             try:
Line 813:                 self._backend.setDomainRegularRole(newmsd)
Line 814:             except Exception:
Line 815:                 self.log.exception('unable to mark domain %s as regular',
Yet another reminder that the "role" field in the metadata is untrustworthy...

How about fixing this bug by removing the attempt to amend that field? If we failed to migrate master, it's probably because we lost access to storage, so we cannot dream to amend it.
Line 816:                                    newmsd.sdUUID)
Line 817: 
Line 818:             # Do not release the cluster lock if unmount fails. The lock
Line 819:             # will prevent other hosts from mounting the master filesystem


-- 
To view, visit http://gerrit.ovirt.org/24495
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia49e45ccab0b933a8f1abc988ea0bd60e7b99b7d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list