Michael Kublin has posted comments on this change.
Change subject: core: when running initVdsOnUp inActive domain can be selected
as master
......................................................................
Patch Set 4: (1 inline comment)
patch look ok to me, will be acked with whole series together
....................................................
File
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
Line 1333: result = ResourceManager.getInstance()
Line 1334:
.getEventListener().storageDomainNotOperational(domainId, _storagePoolId);
Line 1335: } else {
Line 1336: log.warnFormat("Domain {0} was reported by all
hosts in status UP as problematic. Not moving the domain to NonOperational
because it is being reconstructed now.",
Line 1337: domainIdTuple);
This log message can be changed, not related to patch but....
Line 1338: result = ResourceManager.getInstance()
Line 1339:
.getEventListener().masterDomainNotOperational(domainId, _storagePoolId, false);
Line 1340: }
Line 1341: }
--
To view, visit http://gerrit.ovirt.org/13160
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9031e2fe0afc161c1a3acdddc9b1e7a58626a159
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
