Shahar Havivi has posted comments on this change.

Change subject: rest: Add Port Mirroring to VM interface
......................................................................


Patch Set 19: (1 inline comment)

....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
Line 2157:           <xs:element ref="port_mirroring" minOccurs="0" 
maxOccurs="1"/>
ok - now I get it,
I am sorry Michael but it is my bad I didn't told you the latest decision that 
we had. (me Livnat and Ori).
it is true that we will have other properties for traffic monitoring such as 
inline, but instead of adding a new attribute we will call it by other name 
such as <traffic_forward>.
so we do stick with the original design.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic03d841d6d0f6cb7250a25e43bdda3d3e9b7488b
Gerrit-PatchSet: 19
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: Livnat Peer <lp...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to