Yaniv Bronhaim has posted comments on this change. Change subject: Adding system information to getCapabilities from host ......................................................................
Patch Set 1: (3 inline comments) It is bios information. Those are the parameters that are requested here https://bugzilla.redhat.com/show_bug.cgi?id=867543, if you think we need to add something else update the bz please. .................................................... File backend/manager/dbscripts/create_tables.sql Line 432: CREATE TABLE vds_dynamic Line 433: ( Line 434: vds_id UUID NOT NULL, Line 435: status INTEGER NOT NULL, Line 436: host_manufacturer VARCHAR(40), So should I make new upgrade script for that? Don't we want those attributes to be defaults when we create the table? Line 437: host_product_name VARCHAR(40), Line 438: host_version VARCHAR(40), Line 439: host_serial_number VARCHAR(40), Line 440: host_uuid VARCHAR(40), .................................................... File backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java Line 157: return true; Line 158: } Line 159: Line 160: public VDS(Guid vds_group_id, String vds_group_name, String vds_group_description, Guid vds_id, String vds_name, Line 161: String ip, String host_name, int port, int status, String host_manufacturer, String host_product_name, This is the standard here... I won't change all variable names. not in the same patch at least.. don't you agree? Line 162: String host_version, String host_serial_number, String host_uuid, String host_family, Line 163: Integer cpu_cores, String cpu_model, Line 164: Double cpu_speed_mh, String if_total_speed, Boolean kvm_enabled, Integer physical_mem_mb, Line 165: Double cpu_idle, Double cpu_load, Double cpu_sys, .................................................... File backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java Line 303: vds.sethost_manufacturer(AssignStringValue(xmlRpcStruct, VdsProperties.host_manufacturer)); Line 304: vds.sethost_product_name(AssignStringValue(xmlRpcStruct, VdsProperties.host_product_name)); Line 305: vds.sethost_version(AssignStringValue(xmlRpcStruct, VdsProperties.host_version)); Line 306: vds.sethost_serial_number(AssignStringValue(xmlRpcStruct, VdsProperties.host_serial_number)); Line 307: vds.sethost_uuid(AssignStringValue(xmlRpcStruct, VdsProperties.host_uuid)); fix to what? Line 308: vds.sethost_family(AssignStringValue(xmlRpcStruct, VdsProperties.host_family)); Line 309: vds.setcpu_cores(AssignIntValue(xmlRpcStruct, VdsProperties.cpu_cores)); Line 310: vds.setcpu_sockets(AssignIntValue(xmlRpcStruct, VdsProperties.cpu_sockets)); Line 311: vds.setcpu_model(AssignStringValue(xmlRpcStruct, VdsProperties.cpu_model)); -- To view, visit http://gerrit.ovirt.org/9337 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I142198d2059cf109be3859f255621e6ceca8582b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
