Sure. I will add ssl usage example with some brief in the ovn-ctl.8.xml and
send v3 for this patch . Does that sound good?
On Mon, Oct 8, 2018 at 5:32 PM Han Zhou wrote:
>
>
> On Mon, Oct 8, 2018 at 4:55 PM aginwala aginwala
> wrote:
> >
> >
> >
> > On Mon, Oct 8, 2018 at 10:47 AM Han Zhou wro
Hallo mein Geliebter.
Ich grüße dich mit dem Namen unseres Herrn Jesus Christus. Es ist wahr, dass
dieser Brief als Überraschung zu dir kommen kann. Dennoch bitte ich Sie
demütig, mir Ihre Aufmerksamkeit zu schenken und mich gut zu hören. Mein Name
ist Frau Gertrude aus den Vereinigten Staate
>>
>> Giving a second thought, it seems there is still a problem.
>>
>> There should be two sets of SSL related parameters we should consider in
the active-standby scenario.
>> - One set of parameters is for the server side. For ipaddr2 use case,
both active and standby nodes will need them. For LB
On Mon, Oct 8, 2018 at 4:55 PM aginwala aginwala wrote:
>
>
>
> On Mon, Oct 8, 2018 at 10:47 AM Han Zhou wrote:
>>
>>
>>
>> On Fri, Oct 5, 2018 at 6:48 PM aginwala aginwala
wrote:
>> >
>> > Thanks for the review Han. Please find the comments inline below:
>> > On Thu, Oct 4, 2018 at 9:58 AM Han
On Mon, Oct 08, 2018 at 04:23:53PM -0400, Mark Michelson wrote:
> This adds the following commands:
>
> pg-add: Add a new port group, optionally adding switch ports at
> creation.
> pg-set-ports: Sets the logical switch ports on a port group
> pg-del: Remove a port group.
>
> The main motivation
Bleep bloop. Greetings aginwala aginwala, I am a robot and I have tried out
your patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
ERROR: Author amgin...@gmail.com needs to sign off.
WARNING: Unexpected sign-offs from de
Bleep bloop. Greetings aginwala aginwala, I am a robot and I have tried out
your patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
ERROR: Author amgin...@gmail.com needs to sign off.
WARNING: Unexpected sign-offs from de
On Mon, Oct 8, 2018 at 10:47 AM Han Zhou wrote:
>
>
> On Fri, Oct 5, 2018 at 6:48 PM aginwala aginwala
> wrote:
> >
> > Thanks for the review Han. Please find the comments inline below:
> > On Thu, Oct 4, 2018 at 9:58 AM Han Zhou wrote:
> >>
> >> Thanks Ali, please see my comments below
> >>
>
On Mon, Oct 8, 2018 at 2:17 PM Han Zhou wrote:
>
>
> On Mon, Oct 8, 2018 at 11:55 AM aginwala aginwala
> wrote:
> >
> > Yes, that's right. I will send out v2 in a bit with Han's ack.
> >
> >
> > Regards,
> > Aliasgar
> >
> > On Mon, Oct 8, 2018 at 11:04 AM Ben Pfaff wrote:
> >>
> >> On Mon, Oc
When starting OVN DBs in HA using pacemaker with ssl, we need to pass ssl
certs for starting standby DBs. Hence, we need this change.
Signed-off-by: aginwala
Acked-by: Han Zhou
---
ovn/utilities/ovndb-servers.ocf | 72 -
1 file changed, 71 insertions(+),
For OVN DBs to work with SSL in HA, we need to have capability to pass ssl
certs when starting OVN DBs. Say when starting OVN DBs in active passive mode,
in order for the standby DBs to sync from master node, it cannot sync
because the required ssl certs are not passed when standby DBs are initiali
On Windows 2016 LTSC(RTM) the Container feature and Hyper-V feature both had
DCOM API to add and delete internal (management) ports on the Hyper-V switch.
Starting from the 1703 release and above, enabling only the Container feature
does not fulfil this requirement anymore.
We need new ways to in
This patch adds the new powershell module HNSHelper.psm1 to the OVS windows
installer.
Signed-off-by: Alin Gabriel Serdean
---
windows/automake.mk | 1 +
windows/ovs-windows-installer/Product.wxs | 6 ++
2 files changed, 7 insertions(+)
diff --git a/windows/automake.m
We introduce a new powershell module for interacting with HNS (host network
service):
https://github.com/Microsoft/SDN/blob/master/Kubernetes/windows/hns.psm1
In our repository this shall be named HNSHelper.psm1.
We also add five new commandlets in our OVS powershell module:
Disable-OVSOnHNSNetw
On Mon, Oct 8, 2018 at 1:24 PM Mark Michelson wrote:
>
> This adds the following commands:
>
> pg-add: Add a new port group, optionally adding switch ports at
> creation.
> pg-set-ports: Sets the logical switch ports on a port group
> pg-del: Remove a port group.
>
> The main motivation for these
On Mon, Oct 8, 2018 at 11:55 AM aginwala aginwala
wrote:
>
> Yes, that's right. I will send out v2 in a bit with Han's ack.
>
>
> Regards,
> Aliasgar
>
> On Mon, Oct 8, 2018 at 11:04 AM Ben Pfaff wrote:
>>
>> On Mon, Oct 08, 2018 at 10:58:49AM -0700, Han Zhou wrote:
>> > On Fri, Oct 5, 2018 at 6
Thanks Ben. Let me try it out this week and report back.
On 10/8/18, 1:15 PM, "Ben Pfaff" wrote:
>I think I see a problem in the implementation of bonding when
>recirculation is available. Are you able to try out a patch? If so,
>try the following. It is not a good way to solve the issue, but
Bleep bloop. Greetings Mark Michelson, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
git-am:
:194: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
Failed to merge in the cha
---
Este correo electrónico ha sido comprobado en busca de virus por AVG.
http://www.avg.com
___
dev mailing list
d...@openvswitch.org
https://mail.
This adds the following commands:
pg-add: Add a new port group, optionally adding switch ports at
creation.
pg-set-ports: Sets the logical switch ports on a port group
pg-del: Remove a port group.
The main motivation for these commands is that it allows for adding
logical switch ports by name rat
I think I see a problem in the implementation of bonding when
recirculation is available. Are you able to try out a patch? If so,
try the following. It is not a good way to solve the issue, but it
should illustrate whether recirculation is the problem.
diff --git a/ofproto/bond.c b/ofproto/bond
On Mon, Oct 08, 2018 at 02:49:06PM -0400, Mark Michelson wrote:
> ovn-nbctl when run in daemon mode has two issues:
> 1) An extra newline is prepended to table output
> 2) Table formatting issues are ignored.
>
> This patch series fixes both issues.
> ---
> v1 -> v2: Added tests on each patch
Tha
On Thu, Oct 04, 2018 at 02:23:39PM -0700, Yifeng Sun wrote:
> In the default case when nsh's md_type is not recognized by nsh parser,
> uninitialized data in key->context can sneak into miniflow. This
> patch fixes it.
>
> Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10519
>
Yes, that's right. I will send out v2 in a bit with Han's ack.
Regards,
Aliasgar
On Mon, Oct 8, 2018 at 11:04 AM Ben Pfaff wrote:
> On Mon, Oct 08, 2018 at 10:58:49AM -0700, Han Zhou wrote:
> > On Fri, Oct 5, 2018 at 6:34 PM aginwala aginwala
> wrote:
> > >
> > > Thanks for the review Han. P
On 10/08/2018 11:40 AM, Ben Pfaff wrote:
On Fri, Oct 05, 2018 at 04:05:22PM -0400, Mark Michelson wrote:
Thanks for the feedback Han. Interestingly, when I move the tests from
ovn.at to ovn-nbctl.at, the tests no longer pass. When running in non-daemon
mode, for whatever reason, trying to get th
When ovn-nbctl is running in daemon mode, nbctl_client attempts to parse
table formatting options. The problem is that this then removes the table
formatting options from the array of options passed to the server loop. The
server loop resets the table formatting options to the defaults and then
att
Table formatting has a local static integer that is intended to insert
line breaks between tables. This works exactly as intended, as long as
each call to table_format() is done as a single unit within the run of a
process.
When ovn-nbctl is run in daemon mode, it is a long-running process that
ma
ovn-nbctl when run in daemon mode has two issues:
1) An extra newline is prepended to table output
2) Table formatting issues are ignored.
This patch series fixes both issues.
---
v1 -> v2: Added tests on each patch
---
Mark Michelson (2):
table: Create method for resetting table formatting.
o
On Mon, 8 Oct 2018 at 11:12, Ben Pfaff wrote:
> On Sun, Oct 07, 2018 at 09:53:15PM -0700, Gurucharan Shetty wrote:
> > We now have an extra space in the o/p of `ovs-ofctl dump-tlv-map`.
> >
> > Fixes: 5a0e4aec1af (treewide: Convert leading tabs to spaces.)
> > Signed-off-by: Gurucharan Shetty
>
On Sat, Oct 06, 2018 at 08:04:09PM +0530, nusid...@redhat.com wrote:
> From: Numan Siddique
>
> With this commit the users can override the default values of
> the BFD params - min_rx, min_tx, decay_min_rx and mult if desired.
> This can be useful to debug any issues related to BFD (like
> freque
On Sun, Oct 07, 2018 at 09:53:15PM -0700, Gurucharan Shetty wrote:
> We now have an extra space in the o/p of `ovs-ofctl dump-tlv-map`.
>
> Fixes: 5a0e4aec1af (treewide: Convert leading tabs to spaces.)
> Signed-off-by: Gurucharan Shetty
> ---
> utilities/ovs-save | 2 +-
> 1 file changed, 1 ins
On Mon, Oct 08, 2018 at 05:39:04PM +, Stokes, Ian wrote:
> > For what it's worth, Ian, I'm hoping that you'll take a look at this and
> > decide whether to merge it.
>
> Sure, thanks for flagging Ben, apologies for the delay, I've been on
> vacation the past 2 weeks but will have time to look
Thanks for the review! I will come up with a new version.
Yifeng
On Mon, Oct 8, 2018 at 11:02 AM Ben Pfaff wrote:
> On Thu, Oct 04, 2018 at 04:30:10PM -0700, Yifeng Sun wrote:
> > Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10714
> > Signed-off-by: Yifeng Sun
> > Suggest
On Mon, Oct 08, 2018 at 10:58:49AM -0700, Han Zhou wrote:
> On Fri, Oct 5, 2018 at 6:34 PM aginwala aginwala wrote:
> >
> > Thanks for the review Han. Please find the comments inline below:
> >
> > On Thu, Oct 4, 2018 at 10:16 AM Han Zhou wrote:
> >>
> >> Thanks Ali, please see my comm
> >>
> >>
On Thu, Oct 04, 2018 at 04:30:10PM -0700, Yifeng Sun wrote:
> Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10714
> Signed-off-by: Yifeng Sun
> Suggested-by: Ben Pfaff
Thanks for fixing this bug.
In parse_chassis_resident(), in two error cases, paren_depth is
incremented bu
On Fri, Oct 5, 2018 at 6:34 PM aginwala aginwala wrote:
>
> Thanks for the review Han. Please find the comments inline below:
>
> On Thu, Oct 4, 2018 at 10:16 AM Han Zhou wrote:
>>
>> Thanks Ali, please see my comm
>>
>> On Fri, Sep 21, 2018 at 5:38 PM wrote:
>> >
>> > When starting OVN DBs in
On Fri, Oct 5, 2018 at 6:48 PM aginwala aginwala wrote:
>
> Thanks for the review Han. Please find the comments inline below:
> On Thu, Oct 4, 2018 at 9:58 AM Han Zhou wrote:
>>
>> Thanks Ali, please see my comments below
>>
>> On Fri, Sep 21, 2018 at 5:34 PM wrote:
>> >
>> > For OVN DBs to wor
Hi Ian (or others), do you have any thoughts about this?
On Fri, Sep 28, 2018 at 04:24:33PM -0700, Ben Pfaff wrote:
> I have been playing with the OFTest framework off and on over the last
> few months and I believe it's caught an actual bug with its
> DirectVlanPackets test. I'm appending a scri
On Thu, Oct 04, 2018 at 01:01:09PM -0700, Han Zhou wrote:
> From: Han Zhou
>
> Current ovs-ctl forces to set full hostname in external-ids. In
> some situation users may want to set short hostname. For example,
> in OpenStack - OVN integration, Neutron uses the host-id provided
> by Nova, which i
On Mon, Oct 08, 2018 at 10:23:55AM -0700, Han Zhou wrote:
> On Mon, Oct 8, 2018 at 8:42 AM Ben Pfaff wrote:
> >
> > On Fri, Oct 05, 2018 at 12:14:23PM -0700, Han Zhou wrote:
> > > From: Han Zhou
> > >
> > > This is an enhancement for commit:
> > > 102781c ovsdb-idl: Track changes for table refere
> For what it's worth, Ian, I'm hoping that you'll take a look at this and
> decide whether to merge it.
Sure, thanks for flagging Ben, apologies for the delay, I've been on vacation
the past 2 weeks but will have time to look at this now.
Thanks
Ian
>
> On Tue, Oct 02, 2018 at 10:32:40AM +0100
On Fri, Oct 05, 2018 at 03:16:50PM -0700, Yifeng Sun wrote:
> This seems to be a copy and paste bug that iterates and frees
> the wrong table. This commit fixes that.
>
> Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10730
> Co-authored-by: Ben Pfaff
> Signed-off-by: Yifeng S
On Mon, Oct 8, 2018 at 8:42 AM Ben Pfaff wrote:
>
> On Fri, Oct 05, 2018 at 12:14:23PM -0700, Han Zhou wrote:
> > From: Han Zhou
> >
> > This is an enhancement for commit:
> > 102781c ovsdb-idl: Track changes for table references.
> >
> > The seqno change is needed so that the change-tracking hel
Thanks, applied to master, backported as far as 2.7.
On Fri, Oct 05, 2018 at 03:43:03PM -0700, Yifeng Sun wrote:
> This patch should also fix the bug reported at
> https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10802
>
> On Fri, Oct 5, 2018 at 2:50 PM Yifeng Sun wrote:
>
> > After ofpbug
On Fri, Oct 05, 2018 at 09:19:54AM -0700, Yi-Hung Wei wrote:
> The patch address vswitchd crash when it receives NXT_RESUME with geneve
> tunnel metadata. The crash is due to segmentation fault with the
> following stack trace, and it is observed only in kernel datapath.
> A test is added to preve
On Fri, Oct 05, 2018 at 06:57:24PM +0200, Lorenzo Bianconi wrote:
> Add buffering support for IPv4/IPv6 packets that will be processed
> by arp{}/nd_ns{} action when L2 address is not discovered yet since
> otherwise the packet will be substituted with an ARP/Neighbor
> Solicitation frame and this
On Fri, Oct 05, 2018 at 10:58:35AM +0100, Markos Chandras wrote:
> On 05/10/2018 09:55, Matteo Croce wrote:
> > On Fri, Oct 5, 2018 at 8:32 AM Markos Chandras wrote:
> >>
> >> On 25/09/2018 22:14, Ben Pfaff wrote:
> >>>
> >>> Applied to master thanks!
> >>>
> >>> I sent a patch to remove support f
Current patch-set breaks a lot of STP unit tests:
## --- ##
## openvswitch 2.10.90 test suite. ##
## --- ##
ofproto-dpif
1149: ofproto-dpif - MPLS handlingFAILED
(ofproto-dpif.at:2068)
1164: ofproto-dpif - VLAN+MPLS han
We now have an extra space in the o/p of `ovs-ofctl dump-tlv-map`.
Fixes: 5a0e4aec1af (treewide: Convert leading tabs to spaces.)
Signed-off-by: Gurucharan Shetty
---
utilities/ovs-save | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/utilities/ovs-save b/utilities/ovs-save
in
On Fri, Oct 05, 2018 at 12:14:23PM -0700, Han Zhou wrote:
> From: Han Zhou
>
> This is an enhancement for commit:
> 102781c ovsdb-idl: Track changes for table references.
>
> The seqno change is needed so that the change-tracking helper function
> ..._is_new() can work properly.
>
> Signed-off-
On Fri, Oct 05, 2018 at 04:05:22PM -0400, Mark Michelson wrote:
> Thanks for the feedback Han. Interestingly, when I move the tests from
> ovn.at to ovn-nbctl.at, the tests no longer pass. When running in non-daemon
> mode, for whatever reason, trying to get the port group's ports from the
> databa
On Fri, Oct 05, 2018 at 06:40:33PM -0400, Mark Michelson wrote:
> Table formatting has a local static integer that is intended to insert
> line breaks between tables. This works exactly as intended, as long as
> each call to table_format() is done as a single unit within the run of a
> process.
>
On Fri, Oct 05, 2018 at 06:40:34PM -0400, Mark Michelson wrote:
> When ovn-nbctl is running in daemon mode, nbctl_client attempts to parse
> table formatting options. The problem is that this then removes the table
> formatting options from the array of options passed to the server loop. The
> serv
On Mon, Oct 08, 2018 at 03:15:44PM +, Matteo Croce wrote:
> On Mon, Oct 8, 2018 at 3:11 PM Guru Shetty wrote:
> >
> >
> >
> > On Sat, 6 Oct 2018 at 09:20, Matteo Croce wrote:
> >>
> >> In dpif_netlink_port_add__(), socksp could be NULL, because
> >> vport_socksp_to_pids() would allocate a new
In dpif_netlink_port_add__(), socksp could be NULL, because
vport_socksp_to_pids() would allocate a new array and return a single
zero element.
Following vport_socksp_to_pids() removal, a NULL pointer can happen when
dpif_netlink_port_add__() is called and dpif->handlers is 0.
Restore the old beha
On Mon, Oct 8, 2018 at 3:11 PM Guru Shetty wrote:
>
>
>
> On Sat, 6 Oct 2018 at 09:20, Matteo Croce wrote:
>>
>> In dpif_netlink_port_add__(), socksp could be NULL, because
>> vport_socksp_to_pids() would allocate a new array and return a single
>> zero element.
>> Following vport_socksp_to_pids(
On Sat, 6 Oct 2018 at 09:20, Matteo Croce wrote:
> In dpif_netlink_port_add__(), socksp could be NULL, because
> vport_socksp_to_pids() would allocate a new array and return a single
> zero element.
> Following vport_socksp_to_pids() removal, a NULL pointer can happen when
> dpif_netlink_port_add
On Sat, Oct 6, 2018 at 4:11 AM Mark Michelson wrote:
> When ovn-nbctl is running in daemon mode, nbctl_client attempts to parse
> table formatting options. The problem is that this then removes the table
> formatting options from the array of options passed to the server loop. The
> server loop r
On Sat, Oct 6, 2018 at 4:11 AM Mark Michelson wrote:
> Table formatting has a local static integer that is intended to insert
> line breaks between tables. This works exactly as intended, as long as
> each call to table_format() is done as a single unit within the run of a
> process.
>
> When ovn
From: Bhargava Shastry
Accepts fixes suggested by 0-day robot.
Refactor miniflow tests out of flow_extract_target.c into
a new target called miniflow_target.c
The biggest motivation for this massive (7-10x) increase in fuzzing
speed. Prior to the refactoring, we were doing roughly 900 executi
60 matches
Mail list logo