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