On Tue, Jan 5, 2016 at 5:13 AM, Liran Schour <lir...@il.ibm.com> wrote:
> This patch series implements conditional monitoring by introducing an OVSDB > RFC extension with 2 new JSON-RPC methods: "monitor_cond" and > "monitor_cond_change". Specification of this extension is defined in the > ovsdb-server (1) man page. > Monitor2 is now merged into monitor_cond. A monitor_cond session with an > empty > condition, will behave exactly like monitor2 and will get update2 > notifications. > > Thanks for posting. I will take a closer look. When applying patches, git-am complained about some white space issues: Description: [ovs-dev,monitor_cond,02/12] ovsdb: add conditions utilities to support monitor_cond Applying: ovsdb: add conditions utilities to support monitor_cond /home/azhou/projs/ovs-review/ovs/.git/rebase-apply/patch:141: trailing whitespace. field = !columns_index_map ? warning: 1 line adds whitespace errors. Description: [ovs-dev,monitor_cond,04/12] ovsdb-client: support monitor-cond Applying: ovsdb-client: support monitor-cond /home/azhou/projs/ovs-review/ovs/.git/rebase-apply/patch:53: trailing whitespace. all rows will be monitored. If at least one \fIcolumn\fR is specified, only those warning: 1 line adds whitespace errors. Description: [ovs-dev,monitor_cond,07/12] ovsdb: enable jsonrpc-server to service "monitor_cond_change" request Applying: ovsdb: enable jsonrpc-server to service "monitor_cond_change" request /home/azhou/projs/ovs-review/ovs/.git/rebase-apply/patch:78: trailing whitespace. /home/azhou/projs/ovs-review/ovs/.git/rebase-apply/patch:180: trailing whitespace. warning: 2 lines add whitespace errors. > Note: With this patch the json cache will be used only for monitor sessions > with an empty condition. We can think on implementing a per row json cache > that > will be matched against condition clauses in the future. > > > Can we add conditions as part of cache selector? > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev