Change in ovirt-engine[master]: core: Set emulated machines after capabilities

ahadas at redhat.com ahadas at redhat.com
Mon Dec 15 07:22:43 UTC 2014


Arik Hadas has submitted this change and it was merged.

Change subject: core: Set emulated machines after capabilities
......................................................................


core: Set emulated machines after capabilities

After incorrectly adding a ppc64 host in an empty x86-64 cluster and
removing it, the cluster cannot accept new x86-64 hosts, accusing
incompatibility between the emulatedMachines property of the cluster
and the hosts being added.
To fix the issue, the place where emulated machines is set checks if
the host is operational before being executed.

Change-Id: Ibc25acf8e9914e382a5ebf66f9c5d895ed4dab2a
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1061320
Signed-off-by: Leonardo Bianconi <leonardo.bianconi at eldorado.org.br>
Signed-off-by: Vitor de Lima <vdelima at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VirtMonitoringStrategy.java
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Vitor de Lima: Verified
  Arik Hadas: Looks good to me, approved
  Michal Skrivanek: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc25acf8e9914e382a5ebf66f9c5d895ed4dab2a
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Leonardo Bianconi <leonardo.bianconi at eldorado.org.br>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa at eldorado.org.br>
Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi at eldorado.org.br>
Gerrit-Reviewer: Liran Zelkha <lzelkha at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Vitor de Lima <vdelima at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list