Gilad Chaplik has uploaded a new change for review. Change subject: core: log CRUD Cluster supports virt/gluster ......................................................................
core: log CRUD Cluster supports virt/gluster Add info level log for creating and updating a cluster for cluster supporting virt and gluster. Change-Id: I01a2b3ed5348145084f244a374d309cbd30e7c97 Signed-off-by: Gilad Chaplik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java 2 files changed, 8 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/56/11256/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java index 51dc3af..41e73b5 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java @@ -32,7 +32,9 @@ @Override protected void executeCommand() { - + log.infoFormat("Adding Cluster, supports virt={0}, supports gluster={1}", + getVdsGroup().supportsVirtService(), + getVdsGroup().supportsGlusterService()); // If either the low or the high utilization value is the configuration one, then we set both to the // configuration value, // to prevent cases where low utilization > high utilization due to specific configuration defaults diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java index bba3ac5..adb009d 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java @@ -52,6 +52,11 @@ @Override protected void executeCommand() { + log.infoFormat("Updating Cluster, supports virt={0} to {1}, supports gluster={2} to {3}", + oldGroup.supportsVirtService(), + getVdsGroup().supportsVirtService(), + oldGroup.supportsGlusterService(), + getVdsGroup().supportsGlusterService()); // TODO: This code should be revisited and proper compensation logic should be introduced here CheckMaxMemoryOverCommitValue(); getVdsGroupDAO().update(getParameters().getVdsGroup()); -- To view, visit http://gerrit.ovirt.org/11256 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I01a2b3ed5348145084f244a374d309cbd30e7c97 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
