[ovs-dev] [PATCH] Reset the backoff timer for bootstrapped SSL connections

ned-off-by: Arun Sharma --- lib/rconn.c |8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/rconn.c b/lib/rconn.c index e24c356..2b88601 100644 --- a/lib/rconn.c +++ b/lib/rconn.c @@ -1197,7 +1197,6 @@ static void disconnect(struct rconn *rc, int error) OVS_R

Re: [ovs-dev] [PATCH] ovs-dpctl: Don't print mask when all fields exact-match.

Thanks Ben, I have send v2. On 01/05/14 8:30 PM, "Ben Pfaff" wrote: >On Mon, Apr 28, 2014 at 10:25:21PM -0700, Arun Sharma wrote: >> For each field, if the mask is all zeros, don't print it at all, >> if it is all ones, print it without the mask trailer. >>

[ovs-dev] [PATCH] Man ovs-ofctl: Typo in arp_spa & arp_tpa

It seems there is a typo in definition of arp_spa & arp_tpa Signed-off-by: Arun Sharma --- utilities/ovs-ofctl.8.in |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in index 9a8fd33..1d4d89d 100644 --- a/utilities/ovs-o

Re: [ovs-dev] [PATCH] tests: Add missing banner.

Verified & looks good. Thanks Ben. On 11/02/14 12:54 AM, "Ben Pfaff" wrote: >Reported-by: Arun Sharma >Signed-off-by: Ben Pfaff >--- > tests/ovsdb-server.at |2 ++ > 1 file changed, 2 insertions(+) > >diff --git a/tests/ovsdb-server.at b/tests/ovsdb-serve

Re: [ovs-dev] [PATCH 3.2] lib: Add ipv6 support for active and passive socket connections

Thanks Ben for your quick reply. On 21/01/14 9:43 PM, "Ben Pfaff" wrote: >On Tue, Jan 21, 2014 at 11:40:29AM +0530, Arun Sharma wrote: >> Thanks for reply. Please find inline comments. > >Your responses look reasonable, except that I'm not entirely sure about >

[ovs-dev] FW: [PATCH 3.2] lib: Add ipv6 support for active and passive socket connections

Hello Ben, Thanks for reply. Please find inline comments. I will send v4 patch. Arun >On 18/01/14 4:53 AM, "Ben Pfaff" wrote: > >>On Tue, Dec 31, 2013 at 01:08:26PM +0530, Arun Sharma wrote: >>> Allows active/passive socket operations to use IPv6 network addres

Re: [ovs-dev] [PATCH v3] lib: Add ipv6 support for active and passive socket connections

Rebased and reposted v3.2. Thanks for the fix on ovsdb-server tests. On 31/12/13 2:06 AM, "Ben Pfaff" wrote: >On Mon, Dec 30, 2013 at 11:40:23AM -0800, Ben Pfaff wrote: >> On Mon, Dec 30, 2013 at 06:09:51PM +0530, Arun Sharma wrote: >> > As per MASTER, tests/ovsdb

Re: [ovs-dev] [PATCH v2] lib: Add ipv6 support for active and passive socket connections

wrote: >On Tue, Dec 24, 2013 at 01:57:38AM +0530, Arun Sharma wrote: >> Allows active/passive socket operations to use IPv6 network addresses, >> except in-band control which is not supported for IPv6. >> >> tests: Added tests for socket connections over IPv6. >>

Re: [ovs-dev] [PATCH 1/3] lib: Add ipv6 support for active and passive socket connections

Thanks Ben for review comments. Couple of queries inline. On 18/12/13 4:57 AM, "Ben Pfaff" wrote: >On Mon, Dec 09, 2013 at 04:21:21AM -0800, Arun Sharma wrote: >> Allows all socket operations to use ipv6 network addresses, >> except in-band control which is not support

[ovs-dev] [PATCH 1/3] lib: Add ipv6 support for active and passive socket connections

Allows all socket operations to use ipv6 network addresses, except in-band control which is not supported for ipv6 Signed-off-by: Nandan Nivgune Signed-off-by: Abhijit Bhopatkar Signed-off-by: Arun Sharma --- lib/packets.h| 12 +++ lib/rconn.c | 39

[ovs-dev] [PATCH 3/3] man: Update manpages for IPv6 socket connections.

Signed-off-by: Nandan Nivgune Signed-off-by: Abhijit Bhopatkar Signed-off-by: Arun Sharma --- lib/vconn-active.man |6 +++--- lib/vconn-passive.man| 10 ++ ovsdb/remote-active.man |9 + ovsdb/remote-passive.man | 13 - 4 files changed, 22

[ovs-dev] [PATCH 2/3] tests: Add tests for socket connections over IPv6.

Also added tests for NetFlow and sFlow using IPv6 sockets. Signed-off-by: Nandan Nivgune Signed-off-by: Abhijit Bhopatkar Signed-off-by: Arun Sharma --- tests/ofproto-dpif.at | 872 --- tests/ofproto-macros.at | 17 + tests/ovs-vsctl.at

[ovs-dev] [PATCH v5] ovs-bugtool: Added --ovs option to get only ovs related information

t passed and user keeps entering "y" or "n" on prompt. Plus, trailing whitespaces are fixed. White space before '=' and after in function def and call is also fixed. Signed-off-by: Arun Sharma --- utilities/bugtool/ovs-bugtool.8|3 + utiliti

Re: [ovs-dev] **SPAM** Re: [PATCH v4] ovs-bugtool: Added --ovs option to get only ovs related information

Thanks Ben. I will send v5 patch. On 7/19/12 12:00 AM, "Ben Pfaff" wrote: >On Wed, Jul 18, 2012 at 11:59:27PM +0530, Arun Sharma wrote: >> Please let me know your comments. > >See inline. > >> On 7/14/12 9:20 AM, "Arun Sharma" wrote: >> >

Re: [ovs-dev] **SPAM** Re: [PATCH v4] ovs-bugtool: Added --ovs option to get only ovs related information

Please let me know your comments. On 7/14/12 9:20 AM, "Arun Sharma" wrote: >Please see comments inline. > >On 7/13/12 10:08 PM, "Ben Pfaff" wrote: > >>On Wed, Jul 11, 2012 at 07:41:59PM -0700, Arun Sharma wrote: >>> Option --ovs is added for ovs-

Re: [ovs-dev] [PATCH v4] ovs-bugtool: Added --ovs option to get only ovs related information

Please see comments inline. On 7/13/12 10:08 PM, "Ben Pfaff" wrote: >On Wed, Jul 11, 2012 at 07:41:59PM -0700, Arun Sharma wrote: >> Option --ovs is added for ovs-bugtool command to collect >> only OpenvSwitch relevant information. To perform >> filtering in plugi

[ovs-dev] [PATCH v4] ovs-bugtool: Added --ovs option to get only ovs related information

t passed and user keeps entering "y" or "n" on prompt. Plus, trailing whitespaces are fixed. White space before '=' and after in function def and call is also fixed. Signed-off-by: Arun Sharma --- utilities/bugtool/ovs-bugtool.8|3 + utiliti

Re: [ovs-dev] [PATCH v3] ovs-bugtool: Added --ovs option to get only ovs related information

Thanks. From: Reid Price Date: Thu, 12 Jul 2012 13:51:31 -0700 To: Arun Sharma Cc: Subject: Re: [ovs-dev] [PATCH v3] ovs-bugtool: Added --ovs option to get only ovs related information On Wed, Jul 11, 2012 at 9:53 AM, Arun Sharma wrote: > +if len(filters): > +

[ovs-dev] [PATCH v3] ovs-bugtool: Added --ovs option to get only ovs related information

t passed and user keeps entering "y" or "n" on prompt. Plus, trailing whitespaces are fixed. White space before '=' and after in function def and call is also fixed. Signed-off-by: Arun Sharma --- utilities/bugtool/ovs-bugtool.8|3 + utiliti

Re: [ovs-dev] [PATCH v2] ovs-bugtool: Added --ovs option to get only ovs related information

Reworked and its present in v3. Also updated ovs-bugtool manual file. Comments inline. From: Reid Price Date: Wed, 11 Jul 2012 12:00:02 -0700 To: Arun Sharma Cc: Subject: Re: [ovs-dev] [PATCH v2] ovs-bugtool: Added --ovs option to get only ovs related information Thanks for the fixes! A

[ovs-dev] [PATCH v2] ovs-bugtool: Added --ovs option to get only ovs related information

of 'filters' attribute will be compared with filtering option in load_plugins to get all relevant operation to collect information. If no "--ovs" option is passed then it will behave as earlier. Plus, trailing whitespaces are fixed. White space before '=' and aft

Re: [ovs-dev] [PATCH] ovs-bugtool: Added --ovs option to get only ovs related information

Thanks for the review comments. Applied the comments and the comments are mentioned inline. I will share Patch V2 in next email. From: Reid Price Date: Tue, 10 Jul 2012 18:57:12 -0700 To: Arun Sharma Cc: Subject: Re: [ovs-dev] [PATCH] ovs-bugtool: Added --ovs option to get only ovs

[ovs-dev] [PATCH] ovs-bugtool: Added --ovs option to get only ovs related information

iltering option like --ovs to get all relevant operation to collect information. If no "--option" is passed then it will behave as earlier. Plus, trailing whitespaces are fixed. Signed-off-by: Arun Sharma --- utilities/bugtool/ovs-bugtool.in | 83 ++-

[ovs-dev] [PATCH] Code cleanup - In ovs-dpctl, ovs-ofctl command and dpif-netdev

Rename do_* in ovs-dpctl and ovs-ofctl command with "dpctl_" or "ofctl_" prefix. Rename add_flow with dp_netdev_flow_add in lib/dpif-netdev.c. Signed-off-by: Arun Sharma --- lib/dpif-netdev.c |8 +- utilities/ovs-dpctl.c | 52 +++--- utilitie

Re: [ovs-dev] [PATCH V2] ovs-ofctl: Ability to sort dump-flows by priority/other flow fields.

Thanks Ben. I have tested this patch. Arun On 7/4/12 1:19 AM, "Ben Pfaff" wrote: >On Fri, Jun 29, 2012 at 11:11:19AM -0700, Arun Sharma wrote: >> ovs-ofctl will have --sort[=field]|--rsort[=field] option for >>'dump-flows'. >> Sort by field if speci

[ovs-dev] [PATCH] ipsec gre: Do not reread ovs monitor ipsec pidfile in netdev vport so much

ipsec tunnel. Signed-off-by: Arun Sharma --- lib/netdev-vport.c | 12 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/lib/netdev-vport.c b/lib/netdev-vport.c index 7fe169e..d2c2672 100644 --- a/lib/netdev-vport.c +++ b/lib/netdev-vport.c @@ -681,10 +6

Re: [ovs-dev] [PATCH V2] Using UTC in all of ovs logs

he unit tests are what happen when you run "make check". What you >attached is just the output of running a couple of commands by hand. > >Anyway I've pushed this commit. > >Thanks, > >Ben. > >On Mon, Jul 02, 2012 at 02:04:09PM +0530, Arun Sharma wrote: &

[ovs-dev] [PATCH] Python daemons: Changed log date time format.

Changed the python daemons log date and time format to keep similar as C binaries logging format. The date time is in UTC. Signed-off-by: Arun Sharma --- python/ovs/vlog.py |2 +- tests/vlog.at |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/python/ovs/vlog.py

Re: [ovs-dev] [PATCH V2] Using UTC in all of ovs logs

Unit test results is attached with this email. Thanks, Arun On 6/30/12 1:20 AM, "Arun Sharma" wrote: >Fixed the remaining programs which does not logs in UTC timestamp format. >Also updated NEWS. > >Signed-off-by: Arun Sharma >--- > NEWS

Re: [ovs-dev] [PATCH] Using UTC in all of ovs logs

Fixed and shared the Patch V2. On 6/28/12 8:44 PM, "Ben Pfaff" wrote: >On Thu, Jun 28, 2012 at 07:23:48PM -0700, Arun Sharma wrote: >> Fixed the remaining programs which does not logs in UTC timestamp >>format. >> >> Signed-off-by: Arun Sharma > >In

[ovs-dev] [PATCH V2] Using UTC in all of ovs logs

Fixed the remaining programs which does not logs in UTC timestamp format. Also updated NEWS. Signed-off-by: Arun Sharma --- NEWS |5 - lib/table.c |2 +- lib/vlog.c|4 ++-- ovsdb/ovsdb-tool.c|2 +- utilities/ovs-ofctl.c |2 +- 5

[ovs-dev] [PATCH V2] ovs-ofctl: Ability to sort dump-flows by priority/other flow fields.

s found equal in some entries then those entries will be displayed after sorting by priority field in descending order. Feature# 8754 Signed-off-by: Arun Sharma --- lib/ofp-print.c | 80 +++-- lib/ofp-print.h |6 + tests/ovs-ofctl.at | 60 + utiliti

[ovs-dev] [PATCH] Debian packaging to remove system-id.conf

Debian packaging, "--purge" to remove /etc/openvswitch/system-id.conf Signed-off-by: Arun Sharma --- debian/openvswitch-switch.postrm |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/debian/openvswitch-switch.postrm b/debian/openvswitch-switch.postrm ind

[ovs-dev] [PATCH] Using UTC in all of ovs logs

Fixed the remaining programs which does not logs in UTC timestamp format. Signed-off-by: Arun Sharma --- lib/table.c |2 +- lib/vlog.c|2 +- ovsdb/ovsdb-tool.c|2 +- utilities/ovs-ofctl.c |2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff

[ovs-dev] [PATCH] ovs-ofctl: Ability to sort ovs-ofctl dump-flows by priority/other flow fields.

case of priority field, output should be reversed with "--sort" going in descending order and "--rsort" going in ascending order. Feature# 8754 Signed-off-by: Arun Sharma --- lib/ofp-print.c | 83 -- lib/ofp-print.h |7 + tests/ovs-ofctl.a

[ovs-dev] [PATCH v3] ovs-vsctl: Add "--all" option for "destroy" command in ovs-vsctl.

Adds the ability to delete all records from table. This will help users to destroy all records from Qos or Queue table using single command rather then current method. Feature #11306 Suggested-by: Kevin Mancuso Signed-off-by: Arun Sharma --- AUTHORS |1 + tests/ovs

[ovs-dev] [PATCH v3] ovs-vsctl: Add "--all" option for "destroy" command in ovs-vsctl.

Adds the ability to delete all records from table. This will help users to destroy all records from Qos or Queue table using single command rather then current method. Feature #11306 Suggested-by: Kevin Mancuso Signed-off-by: Arun Sharma --- AUTHORS |1 + tests/ovs

[ovs-dev] [PATCH v2] ovs-vsctl: Add "--all" option for "destroy" command in ovs-vsctl.

Adds the ability to delete all records from table. This will help users to destroy all records from Qos or Queue table using single command rather then current method. Feature #11306 Suggested-by: Kevin Mancuso Signed-off-by: Arun Sharma --- tests/ovs-vsctl.at | 48

[ovs-dev] [PATCH] ovs-vsctl: Add "--all" option for "destroy" command in ovs-vsctl.

Adds the ability to delete all records from table. This will help users to destroy all records from Qos or Queue table using single command rather then current method. Feature #11306 Suggested-by: Kevin Mancuso Signed-off-by: Arun Sharma --- tests/ovs-vsctl.at | 26

[ovs-dev] [PATCH] bugtool - rename label names in plugins related to ovs-appctl

It improves to have proper out file name in bugtool archive with respect to ovs-appctl commands. E.g. if command is 'ovs-appctl lacp/show' then the related out file will be 'ovs-appctl-lacp-show.out' Feature #11283 Signed-off-by: Arun Sharma --- .../bugtool/pl

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

As per Feature #11283, it was expected to have the label name containing 'appctl'. It is also planned to have 'appctl' added for other labels which uses ovs-appctl commands. On 5/9/12 7:04 PM, "Ben Pfaff" wrote: >On Tue, May 08, 2012 at 11:47:30PM -0700,

[ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

This is an enhancement in bugtool archive output to determine the bond state information. It is implemented as a plugin which internally calls "ovs-appctl bond/show" command to get bond state. Feature #11283 Signed-off-by: Arun Sharma --- utilities/bugtool/a

[ovs-dev] [PATCH] Python binaries to write comment in db show-log while updating database

This is an improvement in python script ovs-xapi-sync to call add_comment() method while updating database. This will help developer to debug binaries and database from ovsdb-tool show-log command output and understand which python binary is updating db. Feature #10543 Signed-off-by: Arun Sharma

[ovs-dev] [PATCH v2] bugtool - Collect version information for all running Open vSwitch daemons.

. Feature #10383 Signed-off-by: Arun Sharma --- AUTHORS|1 + debian/openvswitch-common.install |1 + utilities/bugtool/automake.mk |3 +- utilities/bugtool/ovs-bugtool-daemons-ver

[ovs-dev] [PATCH] bugtool - Collect version info for all running open Vswitch daemons/binaries.

directory. Feature #10383 Signed-off-by: Arun Sharma --- AUTHORS|1 + debian/openvswitch-common.install |1 + utilities/bugtool/automake.mk |3 +- utilities/bugtool/ovs-bugtool-daemons-ver