Roy Golan has posted comments on this change. Change subject: core: Add supportedEmulatedMachines to Host model cluster ......................................................................
Patch Set 1: (1 inline comment) >>I'm missing why you don't set the new default value per cluster based on the >>current first value in the EmulatedMachine config per cluster version This means that if we want auto-setting by the first host, the first value must be '' (empty or null). '','rhel6.3.0','pc-1.2' So the auto-set feature becomes optional. Maybe this is what you meant in the first place but I missed that. >> (then delete that value which is no longer supported, which i also missed >> seeing). I didn't supply an update yet, just a read-only field in the UI. .................................................... File backend/manager/modules/dal/src/test/resources/fixtures.xml Line 528: <value>6d849ebf-755f-4552-ad09-9a090cda105d</value> Line 529: <value>200</value> Line 530: <value>2.3</value> Line 531: <value>0</value> Line 532: <value>t/rue</value> ... :( Line 533: <value>false</value> Line 534: <value>false</value> Line 535: <value>false</value> Line 536: <value>pc-1.0</value> -- To view, visit http://gerrit.ovirt.org/15864 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I25df8376026f3e5f57ce85d8e7a20e5000ce7eed Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
