Omer Frenkel has posted comments on this change.

Change subject: core: change the retry to run failed HA VM frequency
......................................................................


Patch Set 16: Code-Review+1

(1 comment)

....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java
Line 40:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.VM_MIGRATION_FAILED);
Line 41:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.VM_NOT_RESPONDING);
Line 42:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.VM_STATUS_RESTORED);
Line 43:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.HA_VM_RESTART_FAILED);
Line 44:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.EXCEEDED_MAXIMUM_NUM_OF_RESTART_HA_VM_ATTEMPTS);
not needed
Line 45:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.HA_VM_FAILED);
Line 46:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.VM_CONSOLE_CONNECTED);
Line 47:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.VM_CONSOLE_DISCONNECTED);
Line 48:         AddEventNotificationEntry(EventNotificationEntity.Vm, 
AuditLogType.VM_SET_TICKET);


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9f8914efc73ac1ea4f4a2ec4e588d778e9002451
Gerrit-PatchSet: 16
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to