Alona Kaplan has posted comments on this change. Change subject: engine: Persist total RX/TX in compatible hosts/VMs ......................................................................
Patch Set 8: (2 comments) http://gerrit.ovirt.org/#/c/36470/8/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java File backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java: Line 67: public static final String rx_total = "rx"; // in vm also Line 68: public static final String tx_dropped = "txDropped"; // in vm also Line 69: public static final String tx_rate = "txRate"; // in vm also Line 70: public static final String tx_total = "tx"; // in vm also Line 71: public static final String iface_status = "state"; // in vm also Didn't find any use case of this field for vms. Line 72: public static final String sample_time = "sampleTime"; // in vm also Line 73: public static final String vm_active = "vmActive"; Line 74: public static final String vm_count = "vmCount"; Line 75: public static final String vm_migrating = "vmMigrating"; http://gerrit.ovirt.org/#/c/36470/8/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java File frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java: Line 531: obj.setReceivedBytesOffset(instance.getReceivedBytesOffset()); Line 532: obj.setTransmitDropRate(instance.getTransmitDropRate()); Line 533: obj.setTransmitRate(instance.getTransmitRate()); Line 534: obj.setTransmittedBytes(instance.getTransmittedBytes()); Line 535: obj.setTransmittedBytesOffset(instance.getTransmittedBytesOffset()); what about the sample time? Line 536: obj.setStatus(instance.getStatus()); Line 537: } Line 538: Line 539: private static VmNetworkStatistics cloneVmNetworkStatistics(VmNetworkStatistics instance) -- To view, visit http://gerrit.ovirt.org/36470 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3063bb36fb7e4d5a545d42c648091688ce1e6964 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lver...@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com> Gerrit-Reviewer: Martin Mucha <mmu...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches