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 <[email protected]>
Signed-off-by: Vitor de Lima <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa 
<[email protected]>
Gerrit-Reviewer: Leonardo Bianconi <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Shahar Havivi <[email protected]>
Gerrit-Reviewer: Vitor de Lima <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to