Bug 1061320 - Cluster emulatedMachines property is set even if the host is non-operational
Summary: Cluster emulatedMachines property is set even if the host is non-operational
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-core
Version: 3.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.6.0
Assignee: Martin Polednik
QA Contact: bugs@ovirt.org
URL:
Whiteboard: virt
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-02-04 15:55 UTC by Vitor de Lima
Modified: 2016-02-10 19:47 UTC (History)
11 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-08-10 11:06:41 UTC
oVirt Team: Virt
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 25699 0 master MERGED core: Set emulated machines after capabilities Never

Description Vitor de Lima 2014-02-04 15:55:59 UTC
Description of problem:
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.

Version-Release number of selected component (if applicable):
3.4

How reproducible:
Always

Steps to Reproduce:
1. Create a cluster with any x86-64 processor name
2. Add a ppc64 host into it
3. Remove this ppc64 host
4. Add a x86-64 host

Actual results:
The x86-64 host won't be added into the cluster, resulting in an error due to the emulatedMachines property of the cluster being incorrectly set.

Expected results:
The x86-64 host is added into the cluster.

Additional info:
This bug also occurs in other cases where the HandleVdsCpuFlagsOrClusterChangedCommand sets a host an non-operational but the emulatedMachines property of this host is used to populate the same field in the cluster, enven if the host was reject.

Comment 1 Itamar Heim 2014-02-09 08:52:35 UTC
Setting target release to current version for consideration and review. please
do not push non-RFE bugs to an undefined target release to make sure bugs are
reviewed for relevancy, fix, closure, etc.

Comment 2 Omer Frenkel 2014-05-25 12:14:06 UTC
i think this was already solved by http://gerrit.ovirt.org/#/c/27539

can you please verify problem solved?

Comment 3 Roy Golan 2014-06-25 12:51:16 UTC
emulated machine config value should be tight to architecture. this will be easier to manage and it won't be coupled with where you invoke it in the code


   x86_64 {rhel6.5, pc}, ppc {pseries}

same as we did with other cross architecure config values

Comment 4 Michal Skrivanek 2014-12-17 09:36:53 UTC
fix was reverted and have some issues, moving to 3.5.2

Comment 5 Sandro Bonazzola 2015-03-03 12:53:09 UTC
Re-targeting to 3.5.3 since this bug has not been marked as blocker for 3.5.2 and we have already released 3.5.2 Release Candidate.

Comment 7 Michal Skrivanek 2015-08-10 10:13:09 UTC
would be nice to clear the machine type when last host is removed from the cluster

Comment 8 Martin Polednik 2015-08-10 11:06:41 UTC
The machine type can be cleared by right-clicking the cluster row, pressing "Reset Emulated Machine" and confirming the operation. Because of this functionality, implicit machine type reset is not needed.


Note You need to log in before you can comment on or make changes to this bug.