Liron Aravot has uploaded a new change for review.

Change subject: core: clear log message on failure during disk removal.
......................................................................

core: clear log message on failure during disk removal.

1. changed message.
2. reduced audit log severity.

Change-Id: I591c9f4cbb0aeb5fd6c646211e815bdd88fcadd8
Bug-Url: https://bugzilla.redhat.com/??????
Signed-off-by: Liron Aravot <[email protected]>
---
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
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/14/10614/1

diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
index afaeb6d..1ea5f00 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
@@ -361,7 +361,7 @@
         mSeverities.put(AuditLogType.USER_REMOVE_DISK, 
AuditLogSeverity.NORMAL);
         mSeverities.put(AuditLogType.USER_FAILED_REMOVE_DISK, 
AuditLogSeverity.ERROR);
         mSeverities.put(AuditLogType.USER_FINISHED_REMOVE_DISK, 
AuditLogSeverity.NORMAL);
-        mSeverities.put(AuditLogType.USER_FINISHED_FAILED_REMOVE_DISK, 
AuditLogSeverity.ERROR);
+        mSeverities.put(AuditLogType.USER_FINISHED_FAILED_REMOVE_DISK, 
AuditLogSeverity.WARNING);
         mSeverities.put(AuditLogType.IRS_FAILURE, AuditLogSeverity.ERROR);
         mSeverities.put(AuditLogType.IRS_DISK_SPACE_LOW_ERROR, 
AuditLogSeverity.ERROR);
         mSeverities.put(AuditLogType.IRS_DISK_SPACE_LOW, 
AuditLogSeverity.WARNING);
diff --git 
a/backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
 
b/backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
index 6508e21..67298d3 100644
--- 
a/backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
+++ 
b/backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
@@ -95,7 +95,7 @@
 USER_REMOVE_DISK=User ${UserName} initiate removing of disk ${DiskAlias} from 
domain ${StorageDomainName}.
 USER_FAILED_REMOVE_DISK=User ${UserName} failed to initiate removing of disk 
${DiskAlias} from domain ${StorageDomainName}.
 USER_FINISHED_REMOVE_DISK=User ${UserName} finished removing disk ${DiskAlias} 
from domain ${StorageDomainName}.
-USER_FINISHED_FAILED_REMOVE_DISK=User ${UserName} finished removing disk 
${DiskAlias} with failure from domain ${StorageDomainName}.
+USER_FINISHED_FAILED_REMOVE_DISK=User ${UserName} finished removing disk 
${DiskAlias} with storage failure in domain ${StorageDomainName}.
 USER_ATTACH_DISK_TO_VM=Disk ${DiskAlias} was successfully attached to VM 
${VmName} by ${UserName}.
 USER_FAILED_ATTACH_DISK_TO_VM=Failed to attach Disk ${DiskAlias} to VM 
${VmName} (User: ${UserName}).
 USER_DETACH_DISK_FROM_VM=Disk ${DiskAlias} was successfully detached from VM 
${VmName} by ${UserName}.


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I591c9f4cbb0aeb5fd6c646211e815bdd88fcadd8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to