Change in ovirt-engine[master]: host-deploy: apply user visible messages logic

masayag at redhat.com masayag at redhat.com
Wed Jul 8 12:51:59 UTC 2015


Moti Asayag has submitted this change and it was merged.

Change subject: host-deploy: apply user visible messages logic
......................................................................


host-deploy: apply user visible messages logic

remove the use of InstallerMessages, this class is now being used only
for node upgrade and should be removed when that class is migrated to
the ovirt-node-upgrade interface.

VdsMgmtPacakges issues user visible messages only if it is not in check
only mode.

VdsDeploy is interacting with user for all messages as in current
implementation, the double log/audit was removed in favour of audit
only, as it already writes to log.

Change-Id: I6480d5d2d43caccc6093300e03a840afca5e899d
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeploy.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeployBase.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsMgmtPackages.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
5 files changed, 124 insertions(+), 50 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Moti Asayag: Verified; Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/43255
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6480d5d2d43caccc6093300e03a840afca5e899d
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list