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(a)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(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org