mooli tayer has posted comments on this change.

Change subject: tools: support snmp trap as a notification method.
......................................................................


Patch Set 14:

Doron, 

>Last thing- the notifications are design to be selected from the UI. I suggest 
>to close this gap for snmp as well. So admins should be able to choose which 
>traps go to which destination using the UI.
Notice The infra for this is already present in these patches i.e if a record 
is inserted into the DB with notification_method = SNMP_TRAP and address = 
SNMP_MANAGER[:port] It will be sent (the difference is the logic is not 
white/black list but only white) See previous patch.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0cd22d022ae535f45e046b09a2cbfadd837b465c
Gerrit-PatchSet: 14
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Martin PeÅ™ina <[email protected]>
Gerrit-Reviewer: mooli tayer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to