Xiaolei Shi has posted comments on this change. Change subject: common: Numa feature entities ......................................................................
Patch Set 26: (1 comment) http://gerrit.ovirt.org/#/c/23702/26/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java: Line 173: public void setAutoRecoverable(boolean autoRecoverable) { Line 174: this.autoRecoverable = autoRecoverable; Line 175: } Line 176: Line 177: private boolean numaSupported; We collect numa infomation in getCapabilities command. This command is called in VdsManager. But in VdsManager, it will not update VdsStatic, only update VdsDynamic and VdsStatistics. That means if we put this field in VdsStatic, we need to modify current ovirt process to update VdsStatic periodly. I remembered in previous community discussion, VdsStatic only holds the data which user configured, but numa data is from vdsm, it should be placed in VdsDynamic. Is that right Line 178: Line 179: public VdsStatic() { Line 180: serverSslEnabled = false; Line 181: vdsStrength = 100; -- To view, visit http://gerrit.ovirt.org/23702 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifefade432e7955503980bdc6fc5d73ea32818a95 Gerrit-PatchSet: 26 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <[email protected]> Gerrit-Reviewer: Jiří Moskovčák <[email protected]> Gerrit-Reviewer: Kobi Ianko <[email protected]> Gerrit-Reviewer: Leonardo Bianconi <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Vitor de Lima <[email protected]> Gerrit-Reviewer: Xiaolei Shi <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
