Change in ovirt-engine[ovirt-engine-3.3]: backend: Fixing log print when vdsm version is not supported...

[oschreib at redhat.com]

Ofer Schreiber has submitted this change and it was merged.

Change subject: backend: Fixing log print when vdsm version is not supported in cluster
......................................................................


backend: Fixing log print when vdsm version is not supported in cluster

When adding new host we verify its vdsm version and cluster
compatibility as part of HandleVdsVersionCommand. During that we
printed the same message for both compatibilities issue (vdsm version
and cluster level support). This patch separates the validation part with
two different prints.

Change-Id: Iacfb3875a0617f978b4453769a0b1c5704282fa5
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=974101
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HandleVdsVersionCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetNonOperationalVdsCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/NonOperationalReason.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
6 files changed, 29 insertions(+), 23 deletions(-)

Approvals:
  Ofer Schreiber: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/18633
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iacfb3875a0617f978b4453769a0b1c5704282fa5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org