Re: [ovs-dev] [PATCH 1/2] ovn-controller: Fix memory leak and bug in setup_qos().

On Wed, Feb 1, 2017 at 10:52 PM, Ben Pfaff wrote: > The caller of netdev_get_qos() is responsible for freeing its 'details' > smap. > > Is this patch a good candidate for back porting to 2.6 branch since it's fixing memory leaks ? Thanks Numan ​ > Also, the previous qdisc

Re: [ovs-dev] [PATCH v2] ovn-controller: Provide the option to set Encap.options:csum

On Wed, Feb 1, 2017 at 4:31 AM, Ben Pfaff <b...@ovn.org> wrote: > On Sun, Jan 15, 2017 at 12:36:09PM +0530, Numan Siddique wrote: > > ovn-controller by default enables tunnel encapsulation checksums > > for geneve tunnels. With this patch user can

[ovs-dev] [PATCH] ovn-northd: Add flows in DHCP_OPTIONS pipeline to support renew requests

to its offered ip and ip4.dst set to the DHCP server ip or broadcast ip. This patch supports this missing scenario by adding the necessary flows in DHCP_OPTIONS ingress pipeline. Signed-off-by: Numan Siddique <nusid...@redhat.com> --- ovn/northd/ovn-northd.

Re: [ovs-dev] [PATCH] ovn-northd: Add flows in DHCP_OPTIONS pipeline to support renew requests

On Wed, Jan 25, 2017 at 1:40 AM, Russell Bryant <russ...@ovn.org> wrote: > > > On Tue, Jan 24, 2017 at 1:58 PM, Numan Siddique <nusid...@redhat.com> > wrote: > >> ovn-northd adds the flows to send the DHCPv4 packets to ovn-controller >> only with

[ovs-dev] [PATCH v2] ovn-northd: Add flows in DHCP_OPTIONS pipeline to support renew requests

to its offered ip and ip4.dst set to the DHCP server ip or broadcast ip. This patch supports this missing scenario by adding the necessary flows in DHCP_OPTIONS ingress pipeline. Signed-off-by: Numan Siddique <nusid...@redhat.com> --- ovn/northd/ovn-northd.c | 37 +++--- tests/

Re: [ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

​​ On Thu, Feb 23, 2017 at 11:06 PM, Mickey Spiegel <mickeys@gmail.com> wrote: > > On Thu, Feb 23, 2017 at 6:04 AM, <nusid...@redhat.com> wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> Having zone id per datapath is more than su

Re: [ovs-dev] [PATCH] ovn-controller: Provide the option to set Encap.options:csum

1:29 AM, Ben Pfaff <b...@ovn.org> wrote: >> > >> > > On Sat, Jan 14, 2017 at 07:37:53PM +0530, Numan Siddique wrote: >> > > > On Sat, Jan 14, 2017 at 3:18 AM, Ben Pfaff <b...@ovn.org> wrote: >> > > > >> > > > > On Tue, Jan 10,

[ovs-dev] [PATCH] rhel: Support building python ovs package with native json impl

/libopenvswitch.so /usr/lib64/libopenvswitch.so.1 /usr/lib64/libopenvswitch.so.1.0.0 /usr/lib64/python2.7/site-packages/ovs /usr/lib64/python2.7/site-packages/ovs-2.6.90-py2.7.egg-info ... /usr/lib64/python2.7/site-packages/ovs/_json.so ... CC: Terry Wilson <twil...@redhat.com> Signed-off-by: Numan Si

[ovs-dev] [PATCH] AUTHORS: Add Andrew Beekhof

Signed-off-by: Numan Siddique <nusid...@redhat.com> --- AUTHORS | 1 + 1 file changed, 1 insertion(+) diff --git a/AUTHORS b/AUTHORS index f21cee4..ea3a9fb 100644 --- a/AUTHORS +++ b/AUTHORS @@ -14,6 +14,7 @@ Ambika Aroraambika.ar...@tcs.com Amit Bose

Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

On Thu, Dec 1, 2016 at 9:07 PM, Lance Richardson wrote: > Add support for SSL connections to OVN northbound and/or > southbound databases. > > To improve security, the NB and SB ovsdb daemons no longer > have open ptcp connections by default. This is a change in > behavior

Re: [ovs-dev] [PATCH v2] ovn-trace: Implement DHCP option support.

On Wed, Dec 7, 2016 at 1:49 PM, Numan Siddique <nusid...@redhat.com> wrote: > > > On Tue, Dec 6, 2016 at 4:12 AM, Ben Pfaff <b...@ovn.org> wrote: > >> On Sun, Oct 23, 2016 at 02:32:11PM -0700, Ben Pfaff wrote: >> > The put_dhcp_opts(v6) logical action

Re: [ovs-dev] [PATCH v2] ovn-trace: Implement DHCP option support.

t; > > > This also makes the put_dhcp_opts(v6) logical provide useful tracing > output > > showing what's happening and the assumptions. > > > > Signed-off-by: Ben Pfaff <b...@ovn.org> > ​ Acked-by: ​Numan Siddique < ​ <b...@ovn.org>nusid...@redhat.com

Re: [ovs-dev] [PATCH V3] python: Fix The SSL connection is not reconnected when the OVSDB Server is restarted

closed, return (0, "") > > Signed-off-by: Guoshuai Li <l...@dtdream.com> > > ​Acked-by: Numan Siddique <nusid...@redhat.com> ​ ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] ovn-controller: Provide the option to set Encap.options:csum

On Sat, Jan 14, 2017 at 3:18 AM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Jan 10, 2017 at 11:34:42AM +0530, Numan Siddique wrote: > > ovn-controller by default enables UDP checksums for geneve > > tunnels. With this patch user can

Re: [ovs-dev] intermittent ovs-vswitchd segfault

On Wed, Jan 11, 2017 at 5:24 AM, Joe Stringer <j...@ovn.org> wrote: > On 10 January 2017 at 00:25, Numan Siddique <nusid...@redhat.com> wrote: > > Hi, > > > > I am seeing intermittent segfault's in ovs-vswitchd. We have like 20 > > compute nodes and noticed

Re: [ovs-dev] [PATCH 1/2] revalidator: Prevent double-delete of ukey.

On Thu, Jan 12, 2017 at 4:05 AM, Joe Stringer wrote: > On 10 January 2017 at 16:00, Jarno Rajahalme wrote: > > Acked-by: Jarno Rajahalme > > Thanks, applied to master. > > Numan, I believe that this is an issue regardless of whether it fixes > your

[ovs-dev] [PATCH] ovn-controller: Provide the option to set Encap.options:csum

ovn-controller by default enables UDP checksums for geneve tunnels. With this patch user can set the desired value in Open_vSwitch.external_ids:ovn_encap_csum. Signed-off-by: Numan Siddique <nusid...@redhat.com> --- ovn/controller/chassis.c| 12 ++-- ovn/controll

[ovs-dev] Updating Logical_Switch_Port.up with ovn-controller being a read-only client to SB DB

​Hello, With the patches proposed to make ovn-controller not updating the Port_Binding.chassis column [1], the value of Logical_Switch_Port.up in the NB DB will not be the true value. Below is a proposal which solves this problem - ovn-northd and ovn-controller will make use of locking

Re: [ovs-dev] [PATCH] ovsdb-server: Add ovsdb-server/is-backup-server command in unixctl

y to have missed this! Hi Ben, No its not relevant as "ovsdb-server/sync-status" provides the same information. Thanks Numan Ben On Fri, Aug 19, 2016 at 08:17:43PM +0530, Numan Siddique wrote: > This command will be useful to query if the ovsdb-server > instance is active or backup. &

Re: [ovs-dev] ovn ping from VM to external gateway IP failed.

On Fri, Dec 30, 2016 at 5:36 PM, Dong Jun wrote: > Start devstack in one node(master code). > > (10.0.0.7)vm --- (10.0.0.1)dr(169.254.128.2) --- > (169.254.128.1)ogr(172.24.4.10) --- (172.24.4.1)br-ex > (fip 172.24.4.7) > > $ ip addr show eth0 > 2: eth0:

Re: [ovs-dev] ovn ping from VM to external gateway IP failed.

On Mon, Jan 2, 2017 at 2:07 AM, Mickey Spiegel <mickeys@gmail.com> wrote: > > On Sun, Jan 1, 2017 at 10:31 AM, Numan Siddique <nusid...@redhat.com> > wrote: > >> >> >> On Sun, Jan 1, 2017 at 6:39 AM, Mickey Spiegel <mickeys@gmail.com> >

Re: [ovs-dev] ovn ping from VM to external gateway IP failed.

rote: >> >>> >>> On Fri, Dec 30, 2016 at 7:46 AM, Numan Siddique <nusid...@redhat.com> >>> wrote: >>> >>>> On Fri, Dec 30, 2016 at 5:36 PM, Dong Jun <do...@dtdream.com> wrote: >>>> >>> >>> >>>

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: After thawing, retrieve tunnel table from thawed xbridge

lt;mickeys....@gmail.com> > ​Tested-by: Numan Siddique <nusid...@redhat.com> Numan ​ > --- > ofproto/ofproto-dpif-xlate.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c >

Re: [ovs-dev] ovn ping from VM to external gateway IP failed.

On Tue, Jan 3, 2017 at 2:06 AM, Mickey Spiegel <mickeys@gmail.com> wrote: > > On Mon, Jan 2, 2017 at 3:46 AM, Numan Siddique <nusid...@redhat.com> > wrote: > >> >> >> On Mon, Jan 2, 2017 at 2:07 AM, Mickey Spiegel <mickeys@gmail.com> >

Re: [ovs-dev] Issues with the use of the clone action for resubmission to the pipeline

On Fri, Jan 6, 2017 at 9:52 AM, Ben Pfaff wrote: > On Thu, Jan 05, 2017 at 05:54:46PM -0800, Jarno Rajahalme wrote: > > > > > On Jan 5, 2017, at 4:28 PM, Ben Pfaff wrote: > > > > > > On Tue, Jan 03, 2017 at 02:55:19AM -0800, Mickey Spiegel wrote: > > >> One of the

Re: [ovs-dev] [PATCH v5] ovn-ctl: add support for SSL nb/sb db connections

On Fri, Dec 23, 2016 at 5:13 AM, Ben Pfaff wrote: > I see that Numan acked this. Russell, are you satisfied? > > Thanks, > > Ben. > > On Thu, Dec 22, 2016 at 01:54:44PM -0500, Lance Richardson wrote: > > Add support for SSL connections to OVN northbound and/or > > southbound

Re: [ovs-dev] [PATCH v4] ovn-ctl: add support for SSL nb/sb db connections

On Thu, Dec 22, 2016 at 11:32 PM, Ben Pfaff wrote: > Russell and Numan, are you happy with this? > > ​I am happy with this. Thanks Numan ​ > In the manpage, I see that some command-line options are shown with == > instead of =; please fix. > > Acked-by: Ben Pfaff

Re: [ovs-dev] [PATCH v3 2/2] ovn-northd: Add logical flows to support native DNS

On Mon, Mar 27, 2017 at 9:45 PM, Guru Shetty <g...@ovn.org> wrote: > > > On 27 March 2017 at 07:13, <nusid...@redhat.com> wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> OVN implements native DNS resolution which can be used to resolve

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

On Mar 22, 2017 11:56 PM, "Joe Stringer" <j...@ovn.org> wrote: On 22 March 2017 at 09:00, <nusid...@redhat.com> wrote: > From: Numan Siddique <nusid...@redhat.com> > > When ovs-vswitchd sends the NX_PACKET_IN2 message, it may not > encode ETH_TYPE of the p

Re: [ovs-dev] [PATCH v2 2/2] ovn-northd: Add logical flows to support native DNS

On Thu, Mar 23, 2017 at 8:52 PM, Guru Shetty <g...@ovn.org> wrote: > > > On 22 March 2017 at 21:25, Numan Siddique <nusid...@redhat.com> wrote: > >> >> >> On Wed, Mar 22, 2017 at 10:43 PM, Guru Shetty <g...@ovn.org> wrote: >> >>> >

Re: [ovs-dev] [PATCH] Decode the NXT_RESUME message with a loose match.

Wed, Mar 22, 2017 at 01:40:10PM -0700, Jarno Rajahalme wrote: > >>> > >>>> On Mar 22, 2017, at 1:21 PM, Ben Pfaff <b...@ovn.org> wrote: > >>>> > >>>> On Wed, Mar 22, 2017 at 09:30:36PM +0530, nusid...@redhat.com wrote: > >>>>

Re: [ovs-dev] [PATCH] meta-flow: Remove metadata prerequisite on ether type.

an error to the controller if it tries to > install a mismatched flow. > > Reported-by: Dong Jun <do...@dtdream.com> > Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/ > 330052.html > Fixes: 7befb20d0f70 ("nx-match: Fix oxm decode."

Re: [ovs-dev] [PATCH v3 1/2] ovn-controller: Add 'dns_lookup' action

On Wed, Mar 29, 2017 at 11:12 PM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Mar 27, 2017 at 07:42:58PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > This patch adds a new OVN action 'dns_lookup' to support nati

Re: [ovs-dev] [PATCH v3 2/2] ovn-northd: Add logical flows to support native DNS

On Wed, Mar 29, 2017 at 11:47 PM, Guru Shetty wrote: > >> >> Agree. It was in my mind. I somehow missed it out. >> >> >>> >>> The above would mean that for each logical port, we need to create a DNS >>> record in the DNS table. Having the schema like the above has advantages. >>>

Re: [ovs-dev] [PATCH v2 2/2] ovn-northd: Add logical flows to support native DNS

On Wed, Mar 22, 2017 at 10:43 PM, Guru Shetty <g...@ovn.org> wrote: > > > On 22 March 2017 at 09:51, Guru Shetty <g...@ovn.org> wrote: > >> >> >> On 13 March 2017 at 00:32, <nusid...@redhat.com> wrote: >> >>> From: Numan Siddique <

Re: [ovs-dev] [PATCH v4 2/6] lib: rename ovs_nd_opt to ovs_nd_lla_opt

and ovs_nd_prefix_opt is introducted, rename > ovs_nd_opt to ovs_nd_lla_opt to specify it's Source/Target Link-layer > Address Option. > > Signed-off-by: Zongkai LI <zealo...@gmail.com> > Signed-off-by: Numan Siddique <nusid...@redhat.com> > --- > lib/flow.c| 38 +

Re: [ovs-dev] [PATCH v4 1/6] packets: add compose_nd_ra

Please ignore this patch. The updated one is here - https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/330135.html Thanks Numan On Thu, Mar 23, 2017 at 11:33 AM, wrote: > From: Zong Kai LI > > This patch introduces methods to compose a Router

Re: [ovs-dev] [PATCH] Document OVN support in ovs-sandbox.

Signed-off-by: Russell Bryant <russ...@ovn.org> > ​Nice to have the basic documentation back. Acked-by: Numan Siddique <nusid...@redhat.com> --- > Documentation/automake.mk | 1 + > Documentation/index.rst | 1 + > Documentation/tuto

[ovs-dev] [RFC] [net]openvswitch: Clear the ct flow key for the recirculated packet

this issue or it should be addressed in ovs-vswitchd to set the OVS_KEY_ATTR_CT_STATE and other related attributes properly for ct_clear action. Signed-off-by: Numan Siddique <nusid...@redhat.com> --- net/openvswitch/flow.c | 5 + 1 file changed, 5 insertions(+) diff --git a/net/openvswi

Re: [ovs-dev] [RFC] [net]openvswitch: Clear the ct flow key for the recirculated packet

On Fri, Mar 17, 2017 at 5:42 AM, Joe Stringer <j...@ovn.org> wrote: > On 16 March 2017 at 05:25, Numan Siddique <nusid...@redhat.com> wrote: > > It is possible that the ct flow key information would have > > gone stale for the packets received from the userspace d

Re: [ovs-dev] [RFC] [net]openvswitch: Clear the ct flow key for the recirculated packet

On Fri, Mar 17, 2017 at 2:41 AM, Lance Richardson <lrich...@redhat.com> wrote: > > From: "Numan Siddique" <nusid...@redhat.com> > > To: net...@vger.kernel.org, "ovs dev" <d...@openvswitch.org> > > Cc: "Joe Stringer" <j...@ovn.

Re: [ovs-dev] [ovn][dhcp]networkin-ovn CI for master branch failed by DHCP

On Tue, Mar 21, 2017 at 5:50 PM, Dong Jun wrote: > All networking-ovn patches' CI failed for > gate-tempest-dsvm-networking-ovn-ovs-master, > VM can not acquire a DHCP IP. The CI of branch-2.7 is OK. > > I have found some error info, I did not replicate it in my local

Re: [ovs-dev] [PATCH] ovn-northd: Skip icmp4 packets destined for router ports from conntrack

Adding Joe and Jarno to CC. On Tue, Mar 14, 2017 at 9:01 PM, Lance Richardson <lrich...@redhat.com> wrote: > > > - Original Message ----- > > From: "Numan Siddique" <nusid...@redhat.com> > > To: "Russell Bryant" <russ...@ovn.org>

Re: [ovs-dev] [PATCH v4 3/3] ovn-northd: Add logical flows to support native DNS

On Sat, Apr 15, 2017 at 8:51 AM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Apr 03, 2017 at 04:10:27PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > OVN implements native DNS resolution which can be used to resolve the

Re: [ovs-dev] [PATCH] ovn-northd: Skip icmp4 packets destined for router ports from conntrack

Thanks for the review. Please see inline. On Fri, Mar 10, 2017 at 1:44 AM, Russell Bryant <russ...@ovn.org> wrote: > On Mon, Feb 27, 2017 at 12:59 AM, <nusid...@redhat.com> wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > Presently, the

Re: [ovs-dev] [PATCH v2 2/2] ovn-northd ipam: Support IPv6 dynamic assignment

Thanks for the review. On Thu, Mar 9, 2017 at 2:48 AM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Mar 07, 2017 at 07:19:45AM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > OVN will generate the IPv6 address for a lo

Re: [ovs-dev] [PATCH] ovn-northd: Skip icmp4 packets destined for router ports from conntrack

one of you have an opinion on whether this > is > the right approach? > > Thanks, > > Ben. > > On Mon, Feb 27, 2017 at 11:29:14AM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > Pre

[ovs-dev] clone action is broken in the master

Hi Andy and All, The clone action is broken after the commit [1] - xlate: Translate openflow clone into odp sample action. Because of which, the openstack CI job for OVN is broken. It works once I revert this commit. Thanks Numan [1] -

Re: [ovs-dev] [PATCH] ovn-northd: Skip icmp4 packets destined for router ports from conntrack

On Fri, Mar 10, 2017 at 10:22 AM, Numan Siddique <nusid...@redhat.com> wrote: > Thanks for the review. Please see inline. > > > On Fri, Mar 10, 2017 at 1:44 AM, Russell Bryant <russ...@ovn.org> wrote: > >> On Mon, Feb 27, 2017 at 12:59 AM, <nusid...@redhat.c

Re: [ovs-dev] [PATCH 1/2] ovn-controller: Add 'dns_lkup' action

Thanks for the review Ben. Please see inline for few comments. Numan On Wed, Mar 8, 2017 at 10:22 PM, Ben Pfaff <b...@ovn.org> wrote: > On Fri, Feb 10, 2017 at 08:02:15PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > >

Re: [ovs-dev] [PATCH] ovn-northd: Skip icmp4 packets destined for router ports from conntrack

On Tue, Mar 14, 2017 at 12:57 AM, Russell Bryant <russ...@ovn.org> wrote: > On Fri, Mar 10, 2017 at 4:48 PM, Russell Bryant <russ...@ovn.org> wrote: > > On Fri, Mar 10, 2017 at 2:35 PM, Russell Bryant <russ...@ovn.org> wrote: > >> On Thu, Mar 9, 201

Re: [ovs-dev] [ovs-discuss][ovn][dhcp]Could ovn remove restriction of option 3(router)

On Fri, Mar 3, 2017 at 7:04 AM, Dong Jun wrote: > > > On 2017/3/3 1:33, Ben Pfaff wrote: > >> On Thu, Mar 02, 2017 at 07:15:48PM +0800, Dong Jun wrote: >> >>> An router IP MUST be set to dhcp v4 option for option 3, but not all >>> subnet >>> do have a gateway IP. Furthermore,

Re: [ovs-dev] [PATCH v5 2/3] ovn-controller: Add 'dns_lookup' action

On Mon, Apr 24, 2017 at 10:04 PM, Guru Shetty <g...@ovn.org> wrote: > > > On 17 April 2017 at 08:13, <nusid...@redhat.com> wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> This patch adds a new OVN action 'dns_lookup' to support

Re: [ovs-dev] [PATCH v3 3/3] ovn-northd: Add logical flows to support native IPv6 RA

onfigs" is added in the Logical_Router_Port >> table, which the CMS is expected to configure IPv6 RA >> configurations - "address_mode" and "mtu" for adding these flows. >> >> Co-authored-by: Numan Siddique <nusid...@redhat.com> >> Signed-off-b

Re: [ovs-dev] [PATCH] ovn-northd: Add native active-standby HA.

ock again, if we want it to be standby? Otherwise, this > > instance won't have a chance to be active any more. > > Good question ... I was assuming this scenario was due to a lost > connection, and that the IDL would automatically try to re-acquire the > lock. > >

Re: [ovs-dev] [PATCH v2 2/3] ovn-controller: Add a new action - 'put_nd_ra_opts'

On Fri, Jul 14, 2017 at 5:31 AM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Jul 11, 2017 at 08:14:45PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > This patch adds a new OVN action 'put_nd_ra_opts' to support native >

Re: [ovs-dev] [PATCH] Revert "netdev: Fix netdev_open() to adhere to class type if given"

On Fri, Jul 14, 2017 at 12:14 PM, Han Zhou wrote: > This reverts commit 67ac844b55d4c5f6bbfa01773c82b3d6d8b62131. > > The commit introduced a problem that "File exists" will be reported > when trying to open br0. > > The operation that adds eth0 to br0 while moving IP address

Re: [ovs-dev] [PATCH v2 0/3] Support IPv6 link-local address scopes on Linux

On Sun, Jul 16, 2017 at 10:23 AM, Numan Siddique <nusid...@redhat.com> wrote: > > > On Sat, Jul 15, 2017 at 3:03 AM, Ben Pfaff <b...@ovn.org> wrote: > >> v1->v2: >> * Patches 1 and 2 are new. >> * Patch 3 now formats addresses with scope, in

Re: [ovs-dev] [PATCH] Revert "netdev: Fix netdev_open() to adhere to class type if given"

On Mon, Jul 17, 2017 at 1:10 PM, Eelco Chaudron wrote: > On 14/07/17 18:59, Han Zhou wrote: > >> >> >> On Fri, Jul 14, 2017 at 1:35 AM, Eelco Chaudron > > wrote: >> > >> > On 14/07/17 08:44, Han Zhou wrote: >> >> >> >> This

Re: [ovs-dev] [PATCH v1 3/6] ovn: extend expr symbols for ND

d.target", > > "nd.sll" and "nd.tll" to describe more clear about "icmp6.type" > > predicate. > > > > It adds new symbols: > > - "nd_rs" - to match Router Solicitation messages > > - "nd_ra" - to match R

Re: [ovs-dev] [PATCH] Revert "netdev: Fix netdev_open() to adhere to class type if given"

>> > >> > On Jul 17, 2017, at 7:49 AM, Eelco Chaudron <echau...@redhat.com> > wrote: > >> > > >> > On 17/07/17 11:10, Numan Siddique wrote: > >> >> > >> >> > >> >> On Mon, Jul 17, 2017 at 1:1

Re: [ovs-dev] [PATCH] rhel: Fix creation of symlink for ocf script

gt; > openvswitch-ovn-common-2.7.90-1.fc26.x86_64 > > > > Fixes: a4245b7869c8 ("ovn: Add ovn db servers ocf script in fedora > packager") > > > > Signed-off-by: Timothy Redaelli <tredae...@redhat.com> > > --- > > LGTM > LGTM. Acked-by: N

Re: [ovs-dev] [PATCH 1/2] Set release date for 2.7.2.

On Wed, Jul 19, 2017 at 5:37 AM, Justin Pettit wrote: > > > On Jul 18, 2017, at 4:44 PM, Joe Stringer wrote: > > > > On 18 July 2017 at 16:29, Justin Pettit wrote: > >> Signed-off-by: Justin Pettit > > > > Acked-by: Joe Stringer

Re: [ovs-dev] [PATCH v2 0/3] Support IPv6 link-local address scopes on Linux

On Sat, Jul 15, 2017 at 3:03 AM, Ben Pfaff wrote: > v1->v2: > * Patches 1 and 2 are new. > * Patch 3 now formats addresses with scope, instead of just parsing them. > Patch 3 accepts ifindexes instead of just interface names. > > Ben Pfaff (3): > stream: Make

Re: [ovs-dev] [RFC] Basic active-active HA for ovn-northd

On Tue, Aug 1, 2017 at 4:25 PM, Numan Siddique <nusid...@redhat.com> wrote: > > > On Tue, Aug 1, 2017 at 5:00 AM, Han Zhou <zhou...@gmail.com> wrote: > >> On Mon, Jul 31, 2017 at 1:53 PM, Russell Bryant <russ...@ovn.org> wrote: >> > >> &

Re: [ovs-dev] [PATCH] ovn: Fix the failing "2335: ovn -- ACL logging" test case

> AT_CHECK, is still not processed by ovn-controller. To fix this issue, > > OVS_WAIT_UNTIL function is used. > > > > Fixes: d383eed59589 ("ovn: Add support for ACL logging.") > > CC: Justin Pettit <jpet...@ovn.org> > > Signed-off-by: Numan Siddique <n

Re: [ovs-dev] [RFC] Basic active-active HA for ovn-northd

On Tue, Aug 1, 2017 at 5:00 AM, Han Zhou wrote: > On Mon, Jul 31, 2017 at 1:53 PM, Russell Bryant wrote: > > > > I wanted to share the idea before I code it to see if it makes sense. > > I imagine the patch would be small, though. > > > > We currently provide

Re: [ovs-dev] [PATCH v5 2/3] ovn-controller: Add a new action - 'put_nd_ra_opts'

On Fri, Aug 4, 2017 at 4:09 PM, Numan Siddique <nusid...@redhat.com> wrote: > > > On Fri, Aug 4, 2017 at 3:02 AM, Ben Pfaff <b...@ovn.org> wrote: > >> On Mon, Jul 31, 2017 at 06:11:35PM +0530, nusid...@redhat.com wrote: >> > From: Numan Siddique <nusi

Re: [ovs-dev] [PATCH] ovn: increase size of ingress and egress pipelines

On Fri, May 12, 2017 at 12:06 AM, Mickey Spiegel wrote: > The OVN ingress pipeline for a logical switch is maxed out at 16 stages. > > This patch takes the simple approach of starting the ingress pipeline at > table 8 rather than table 16, and starting the egress pipeline

Re: [ovs-dev] [PATCH v5 5/6] ovn-controller: add 'put_nd_ra_addr_mode' and 'put_nd_opt_*' actions support

Option for RA packet with inner > > integer value(32-bit unsigned integer). > > e.g. put_nd_ra_opt_mtu(1450); > > > > Action put_nd_ra_opt_prefix will append Prefix Information Option with > > following inner parameters for RA packet: > > - prefix length(8-bit

Re: [ovs-dev] [PATCH] python ovs: Fix SSL exceptions with pyOpenSSL v0.13

on -c "import OpenSSL.SSL"'. > > > > - The SSL.Context class do not the function "set_session_cache_mode" > >defined. So this patch uses hasattr() to detect this function > >before accessing it. > > > > I have not tested with older ve

Re: [ovs-dev] [PATCH v1 4/6] ovn util: Refactor dhcp_opts_map to make it generic

On Fri, Jun 16, 2017 at 7:42 PM, Brian Haley <haleyb@gmail.com> wrote: > On 06/15/2017 04:39 AM, nusid...@redhat.com wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> Renamed 'struct dhcp_opts_map' to 'struct gen_opts_map' and >> renamed

Re: [ovs-dev] [PATCH] Supporting ovn-northd service HA depend on OVNDB-HA

On Wed, May 17, 2017 at 9:15 PM, Ben Pfaff wrote: > From: Zhengwei Gao > > As ovn-northd servcie parse network element between ovnnb_db and > ovnsb_db, it ensures ovn-northd service connecting to ovnnb_db and > ovnsb_db. OVNDB-HA was supported with pacemaker,

Re: [ovs-dev] [PATCH 1/2] ovn pacemaker: Fix return code errors in start/stop action

On Tue, May 23, 2017 at 5:21 AM, Andy Zhou <az...@ovn.org> wrote: > On Sun, May 21, 2017 at 6:35 PM, <nusid...@redhat.com> wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > start action returns OCF_RUNNING_MASTER in certain scenarios. > &g

Re: [ovs-dev] [PATCH] Supporting ovn-northd service HA depend on OVNDB-HA

On Tue, May 23, 2017 at 9:44 AM, multi_t...@163.com <multi_t...@163.com> wrote: > Hi Numan, > > Thank you very much, I had pushed the patch new version > https://github.com/openvswitch/ovs/pull/178 > Thanks. I tested it. Acked-by: Numan Siddique <nusid...@redhat.com>

Re: [ovs-dev] [PATCH v2] python ovs: Fix SSL exceptions with pyOpenSSL v0.13

On May 23, 2017 11:04 PM, "Russell Bryant" <russ...@ovn.org> wrote: On Mon, May 15, 2017 at 11:39 AM, <nusid...@redhat.com> wrote: > From: Numan Siddique <nusid...@redhat.com> > > Centos provides pyOpenSSL version pyOpenSSL-0.13.1-3.el7.x86_64. > Ther

Re: [ovs-dev] [PATCH] ovn-ctl: Add commands to manage OVN DB ovsdb-servers individually

On Fri, May 26, 2017 at 3:30 AM, Andy Zhou <az...@ovn.org> wrote: > On Thu, May 25, 2017 at 1:55 AM, <nusid...@redhat.com> wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > This patch adds the following functions > > - start_nb_ovsdb, s

Re: [ovs-dev] [PATCH] Supporting ovn-northd service HA depend on OVNDB-HA

ed. For other cases, it would be a no-op. +# Set --ovn-manage-ovsdb=no so that ovn-ctl doesn't stop ovsdb-servers. +${OVN_CTL} --ovn-manage-ovsdb=no stop_northd +fi + ovsdb_server_check_status case $? in $OCF_NOT_RUNNING)return ${OCF_SUCCESS};;

Re: [ovs-dev] [PATCH] Fix flake8 check

On Fri, May 19, 2017 at 9:39 AM, Terry Wilson <twil...@redhat.com> wrote: > Stop occluding the previous definition of filename. > > Signed-off-by: Terry Wilson <twil...@redhat.com> > Acked-by: Numan Siddique <nusid...@redhat.com> > --- > Documentation/con

[ovs-dev] Request to tag 2.7 branch with v2.7.1

Hello Ben, Justin, Is it possible to tag the 2.7 branch with v2.7.1. Recently there were many back ports to 2.7 branch. OVS 2.7 is required for RDO [1] and the RDO community is expecting a new tag to build the latest OVS 2.7 branch. [1] -

Re: [ovs-dev] [PATCH v6 3/3] ovn-northd: Add logical flows to support native DNS

On Wed, May 3, 2017 at 4:38 AM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Apr 25, 2017 at 07:59:54PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > OVN implements native DNS resolution which can be used to resolve the

Re: [ovs-dev] [PATCH v5 3/6] ovn: extend expr symbols for ND

d", > > "nd.target", "nd.sll", "nd.tll", to describe more clear about > "icmp6.type" > > predicate. > > It adds new symbols: > > - "nd_rs" stands for Router Solicitation message, > > - "nd_ra" stands for Rou

Re: [ovs-dev] [PATCH v5 0/6] ovn: Add IPv6 Router Solicitation responder support

On Fri, May 5, 2017 at 9:31 AM, Ben Pfaff <b...@ovn.org> wrote: > On Fri, May 05, 2017 at 09:20:37AM +0530, Numan Siddique wrote: > > On Thu, May 4, 2017 at 8:48 PM, Aaron Conole <acon...@redhat.com> wrote: > > > > > Hi Numan, > > > > > >

Re: [ovs-dev] [PATCH v5 0/6] ovn: Add IPv6 Router Solicitation responder support

On Thu, May 4, 2017 at 8:48 PM, Aaron Conole <acon...@redhat.com> wrote: > Hi Numan, > > nusid...@redhat.com writes: > > > From: Numan Siddique <nusid...@redhat.com> > > > > I have picked up the patch series authored by Zong Kai LI to support IPv6 &

Re: [ovs-dev] 答复: [RFC] Question about ovn-controller performance

On Thu, Sep 14, 2017 at 4:16 PM, wrote: > Hi everyone, > > Whether we could add a port property to logical flow? port property have > value of "normal" or logical port name. If the port property is "normal" > means this logical flow should be allocated to all HVs, if port

Re: [ovs-dev] [PATCH 0/4] ovn: Add support for IPv6 load balancers

On Thu, Sep 21, 2017 at 7:01 PM, Jakub Sitnicki wrote: > Hi Mark, > > On Wed, 6 Sep 2017 16:08:35 -0500 > Mark Michelson wrote: > > > This patchset adds the necessary items in order to support IPv6 load > > balancers in OVN. No syntax has changed in

Re: [ovs-dev] [PATCH] ofproto: Include patch ports in mtu overriden check

On Thu, Sep 21, 2017 at 11:13 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Sep 12, 2017 at 02:22:03PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > When a patch port is deleted from the bridge (with no other ports >

Re: [ovs-dev] [PATCH] ovn pacemaker: Provide the option to configure inactivity probe value

On Sat, Oct 14, 2017 at 2:56 AM, Ben Pfaff <b...@ovn.org> wrote: > On Fri, Oct 13, 2017 at 12:06:56PM -0400, Russell Bryant wrote: > > On Fri, Oct 13, 2017 at 8:30 AM, Numan Siddique <nusid...@redhat.com> > wrote: > > > On Fri, Oct 13, 2017 at 6:05 AM

Re: [ovs-dev] [PATCH] ovn pacemaker: Provide the option to configure inactivity probe value

On Fri, Oct 13, 2017 at 9:36 PM, Russell Bryant <russ...@ovn.org> wrote: > On Fri, Oct 13, 2017 at 8:30 AM, Numan Siddique <nusid...@redhat.com> > wrote: > > On Fri, Oct 13, 2017 at 6:05 AM, Andy Zhou <az...@ovn.org> wrote: > > > >> Hi, Numan,

Re: [ovs-dev] [PATCH] ovn pacemaker: Provide the option to configure inactivity probe value

On Sat, Oct 14, 2017 at 2:56 AM, Ben Pfaff <b...@ovn.org> wrote: > On Fri, Oct 13, 2017 at 12:06:56PM -0400, Russell Bryant wrote: > > On Fri, Oct 13, 2017 at 8:30 AM, Numan Siddique <nusid...@redhat.com> > wrote: > > > On Fri, Oct 13, 2017 at 6:05 AM

Re: [ovs-dev] [PATCH] ovn pacemaker: Provide the option to configure inactivity probe value

t; > >> Thanks, > >> > >> Ben. > >> > >> On Wed, Oct 11, 2017 at 02:22:33PM +0530, nusid...@redhat.com wrote: > >>> From: Numan Siddique <nusid...@redhat.com> > >>> > >>> In the case of OVN HA deployments wi

[ovs-dev] ovs-vswitchd is resetting the MTU of a bridge when a patch port is deleted.

Hello, Daniel (CC'd) and I were debugging an issue in openstack tripleo CI for OVN job. We are noticing an issue when ovn-controller deletes the patch ports in the external bridge specified in "ovn-bridge-mappings". The external bridge (br-ex) is configured with MTU 1450 as it has an vxlan port.

Re: [ovs-dev] [PATCH v8 2/4] ovn-controller: Add a new action - 'put_nd_ra_opts'

Thanks Mark for the review. I will respin another patch set with your comments addressed. Numan On Fri, Sep 29, 2017 at 7:17 PM, Mark Michelson <mmich...@redhat.com> wrote: > > > On Thu, Sep 21, 2017 at 11:10 AM <nusid...@redhat.com> wrote: > >> From: Numan

Re: [ovs-dev] [PATCH v8 4/4] ovn: Generate Neighbor Solicitation packet for unknown MAC IPv6 packets

On Thu, Sep 28, 2017 at 2:12 AM, Mark Michelson <mmich...@redhat.com> wrote: > > > On Thu, Sep 21, 2017 at 11:12 AM <nusid...@redhat.com> wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> In the router ingress pipeline, if the destina

Re: [ovs-dev] [PATCH v2] OVN pacemaker: Add the monitor action for Master role

sid...@redhat.com> wrote: > From: Numan Siddique <nusid...@redhat.com> > > Pacemaker Resource agent periodically calls the OVN OCF's "monitor" action > periodically to check the status. But the OVN OCF script doesn't add the > action "monitor" for the role &qu

Re: [ovs-dev] [PATCH v2] ovn pacemaker: Provide the option to configure inactivity probe value

On Thu, Oct 26, 2017 at 1:41 AM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Oct 16, 2017 at 03:12:07PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > In the case of OVN HA deployments with openstack, it has been noticed

Re: [ovs-dev] [PATCH v2] ovn-ctl: Add new commands 'run_nb_server' and 'run_sb_server'

On Thu, Nov 30, 2017 at 11:27 PM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Nov 30, 2017 at 04:15:09PM +0530, Numan Siddique wrote: > > On Wed, Nov 29, 2017 at 11:19 PM, Ben Pfaff <b...@ovn.org> wrote: > > > > > On Tue, Nov 21, 2017 at 12:35:56PM +0530, nus

Re: [ovs-dev] [PATCH v2] ovn pacemaker: Provide the option to configure inactivity probe value

On Thu, Nov 30, 2017 at 11:21 PM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Nov 30, 2017 at 11:02:23PM +0530, Numan Siddique wrote: > > On Thu, Oct 26, 2017 at 1:41 AM, Ben Pfaff <b...@ovn.org> wrote: > > > > > On Mon, Oct 16, 2017 at 03:12:07PM +0530, nus

Re: [ovs-dev] [PATCH v2 0/2] OVN DB log fixes

ranch-2.7 I had to backport all of the following commits to make > these patches apply cleanly: > > commit 2abbe32153b7e4719b39f477b35e7cc40231338a > Author: Numan Siddique <nusid...@redhat.com> > Date: Wed Nov 8 14:28:49 2017 +0530 > > ovn-ctl: Add -

Re: [ovs-dev] [PATCH v2] ovn-ctl: Add new commands 'run_nb_server' and 'run_sb_server'

On Wed, Nov 29, 2017 at 11:19 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Nov 21, 2017 at 12:35:56PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique <nusid...@redhat.com> > > > > Presently if the user wants to start OVN db servers as separate &

  1   2   3   4   5   6   7   8   9   10   >