Change in ovirt-engine[master]: core: Add external event error messages need to be improved

emesika at redhat.com emesika at redhat.com
Mon Aug 12 07:54:30 UTC 2013


Eli Mesika has submitted this change and it was merged.

Change subject: core: Add external event error messages need to be improved
......................................................................


core: Add external event error messages need to be improved

Add external event error messages need to be improved

Adding EventValidator class to validate severity enum values.
The flood_rate can be omitted and the default is set to 30
(this was working prior to this patch)

Change-Id: I743ff20bd3d4317bc054ba314ada6bd918a7502f
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=949551
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendEventsResource.java
A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/EventValidator.java
2 files changed, 18 insertions(+), 0 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I743ff20bd3d4317bc054ba314ada6bd918a7502f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list