Alexander Wels has posted comments on this change.

Change subject: core,restapi,webadmin: encrypt fence agent options
......................................................................


Patch Set 3:

> Shoudn't the encryptOptions attribute be used only for apc_snmp
 > agent? This patch adds it and enables it for all agent types,
 > right? Or am I missing something?

No you are right, this patch will enable it for all agent types. If it should 
only be for apc_snmp let me know and I will change it.

-- 
To view, visit https://gerrit.ovirt.org/38392
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60ff2c8663d68a5fcb8771831d36e15932b24ceb
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to