Martin Sivák has posted comments on this change. Change subject: engine: Use proper units in Storage QoS when talking to VDSM ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/33906/1/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java File backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java: Line 389: // build map Line 390: Map<String, Integer> ioTuneMap = new HashMap<>(); Line 391: if (storageQos.getMaxThroughput() != null) { Line 392: // Convert MiB/s to B/s vdsm is expecting Line 393: ioTuneMap.put(VdsProperties.TotalBytesSec, storageQos.getMaxThroughput() * 1024 * 1024); > RPC should use strings indeed and we shouldn't limit this calculation. we d I did a quick research and there are indeed hard drives with more than 700MB/s read speed.. and the read speed records are much faster than that.. I will use Long :) And I will do the conversion somewhere in the XML-RPC specific code. Line 394: } Line 395: if (storageQos.getMaxReadThroughput() != null) { Line 396: // Convert MiB/s to B/s vdsm is expecting Line 397: -- To view, visit http://gerrit.ovirt.org/33906 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2644e9e029c8b6ebe4b5b7626f7b1ed6b6b5aea4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Tomer Saban <[email protected]> Gerrit-Reviewer: Vojtech Szocs <[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
