ofri masad has posted comments on this change.
Change subject: core: Add QoS to to DB and entities
......................................................................
Patch Set 8: (11 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/AddNetworkQoSCommand.java
Line 29:
Line 30: @Override
Line 31: protected void setActionMessageParameters() {
Line 32: addCanDoActionMessage(VdcBllMessages.VAR__ACTION__ADD);
Line 33: addCanDoActionMessage(VdcBllMessages.VAR__TYPE__NETWORK_QOS);
Done
Line 34: }
Line 35:
Line 36: @Override
Line 37: public AuditLogType getAuditLogTypeValue() {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/NetworkQoSCommandBase.java
Line 18: private NetworkQoS networkQoS;
Line 19:
Line 20: public NetworkQoSCommandBase(NetworkQoSParametersBase parameters) {
Line 21: super(parameters);
Line 22:
setStoragePoolId(parameters.getNetworkQoS().getStoragePoolId());
Done
Line 23: addCustomValue("NetworkQoSName",
parameters.getNetworkQoS().getName());
Line 24: getParameters().setShouldBeLogged(true);
Line 25: }
Line 26:
Line 26:
Line 27: public NetworkQoS getNetworkQoS() {
Line 28: if (networkQoS == null) {
Line 29: if (getParameters().getNetworkQoS() == null) {
Line 30: if (getParameters().getNetworkQoSGuid() != null) {
this was done to supprt API command in which the user only sends an ID. for
example: remove(qos_id)
Line 31:
getNetworkQoSDao().get(getParameters().getNetworkQoSGuid());
Line 32: }
Line 33: } else {
Line 34: networkQoS = getParameters().getNetworkQoS();
Line 37: return networkQoS;
Line 38: }
Line 39:
Line 40: protected boolean validateNameAndStoragePoolNotNull() {
Line 41: if (getNetworkQoS().getName() == null ||
getNetworkQoS().getStoragePoolId() == null) {
Done.
Line 42: return
failCanDoAction(VdcBllMessages.ACTION_TYPE_FAILED_NETWORK_QOS_MISSING_DATA);
Line 43: }
Line 44: return true;
Line 45: }
Line 94: ||
averageOutOfRange(getNetworkQoS().getOutboundAverage())
Line 95: || peakOutOfRange(getNetworkQoS().getOutboundPeak())
Line 96: || burstOutOfRange(getNetworkQoS().getOutboundBurst());
Line 97: }
Line 98:
Done
Line 99: private boolean averageOutOfRange(Integer average) {
Line 100: return average != null && average >
ConfigValues.MaxAverageNetworkQoSValue.getValue();
Line 101: }
Line 102:
Line 107: private boolean burstOutOfRange(Integer burst) {
Line 108: return burst != null && burst >
ConfigValues.MaxBurstNetworkQoSValue.getValue();
Line 109: }
Line 110:
Line 111: protected boolean negativeValues() {
Done
Line 112: return negativeValue(getNetworkQoS().getInboundAverage())
Line 113: || negativeValue(getNetworkQoS().getInboundPeak())
Line 114: || negativeValue(getNetworkQoS().getInboundBurst())
Line 115: || negativeValue(getNetworkQoS().getOutboundAverage())
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/RemoveNetworkQoSCommand.java
Line 13: }
Line 14:
Line 15: @Override
Line 16: protected boolean canDoAction() {
Line 17: if (validateNameAndStoragePoolNotNull()) {
Done.
Will check that ether parameters.networkQoSGuid or parameters.networkQoS.id
can be found before remove.
Line 18: NetworkQoS oldNetworkQoS =
getNetworkQoSDao().get(getNetworkQoS().getId());
Line 19: if (oldNetworkQoS == null) {
Line 20: return
failCanDoAction(VdcBllMessages.ACTION_TYPE_FAILED_NETWORK_QOS_NOT_FOUND);
Line 21: } else if
(!oldNetworkQoS.getStoragePoolId().equals(getNetworkQoS().getStoragePoolId())) {
Line 34:
Line 35: @Override
Line 36: protected void setActionMessageParameters() {
Line 37: addCanDoActionMessage(VdcBllMessages.VAR__ACTION__REMOVE);
Line 38: addCanDoActionMessage(VdcBllMessages.VAR__TYPE__NETWORK_QOS);
Done
Line 39: }
Line 40:
Line 41: @Override
Line 42: public AuditLogType getAuditLogTypeValue() {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/UpdateNetworkQoSCommand.java
Line 38:
Line 39: @Override
Line 40: protected void setActionMessageParameters() {
Line 41: addCanDoActionMessage(VdcBllMessages.VAR__ACTION__UPDATE);
Line 42: addCanDoActionMessage(VdcBllMessages.VAR__TYPE__NETWORK_QOS);
Done
Line 43: }
Line 44:
Line 45: @Override
Line 46: public AuditLogType getAuditLogTypeValue() {
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/NetworkQoSParametersBase.java
Line 7: public class NetworkQoSParametersBase extends VdcActionParametersBase {
Line 8:
Line 9: private static final long serialVersionUID = -42449085707523901L;
Line 10: private NetworkQoS networkQoS;
Line 11: private Guid networkQoSGuid;
this was done to support API command in which the user only sends an ID. for
example: remove(qos_id)
Line 12:
Line 13: public NetworkQoS getNetworkQoS() {
Line 14: return networkQoS;
Line 15: }
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
Line 750: WATCHDOG_EVENT(9901),
Line 751:
Line 752: //trusted service
Line 753: VDS_UNTRUSTED(10000, AuditLogTimeInterval.MINUTE.getValue()),
Line 754:
Done
Line 755: //network Qos
Line 756: USER_ADDED_NETWORK_QOS(9920),
Line 757: USER_FAILED_TO_ADD_NETWORK_QOS(9921),
Line 758: USER_REMOVED_NETWORK_QOS(9922),
--
To view, visit http://gerrit.ovirt.org/16294
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If39d20b77d06165e4adcc27e6b6dc5458cac93d3
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-Reviewer: ofri masad <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
