Change in ovirt-engine[ovirt-engine-3.4]: tools: notifier: Notification properties changes.

iheim at redhat.com iheim at redhat.com
Thu Feb 20 15:22:42 UTC 2014


Itamar Heim has submitted this change and it was merged.

Change subject: tools: notifier: Notification properties changes.
......................................................................


tools: notifier: Notification properties changes.

Before this change validation was done both in
NotificationService.initConfigurationProperties and in NotificationProperties
itself. all moved to  NotificationProperties.

The properties themself are not read from the notification service
to be instance variables inside the notifier but are read each time
they are needed.

Change-Id: I5753ba2cad2de90a7f2d14bd4675edc1dd35045e
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
---
M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/NotificationService.java
M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java
2 files changed, 92 insertions(+), 118 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, approved
  mooli tayer: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5753ba2cad2de90a7f2d14bd4675edc1dd35045e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list