On Tue, Jul 5, 2016 at 7:39 PM, Ryan Moats wrote:
> "dev" wrote on 06/26/2016 08:34:01 PM:
>
> > From: Darrell Ball
> > To: dlu...@gmail.com, d...@openvswitch.com
> > Date: 06/26/2016 08:34 PM
> > Subject: [ovs-dev] [patch_v3
Some design micro-details (e.g.) register assignments) that may change
over time were moved from the ovn-architecture.7.xml document to the
OVN-DESIGN.md document. A table is added to summarize and quantify
MFF logical metadata and register usage. The OVN-DESIGN.md file was
tested using the
OVN only supports source_node replication and previously vtep interaction,
which used service node replication by default for
multicast/broadcast/unknown unicast traffic worked by happenstance.
Because of limited vxlan encapsulation metadata, received packets were
resubmitted to find the egress
OVN only supports source_node replication and previously vtep interaction,
which used service node replication by default for multicast/broadcast/unknown
unicast traffic worked by happenstance. Because of limited vxlan
encapsulation metadata, received packets were resubmitted to find the egress
The message was undeliverable due to the following reason(s):
Your message was not delivered because the destination server was
unreachable within the allowed queue period. The amount of time
a message is queued before it is returned depends on local configura-
tion parameters.
Most likely there
Daniele Di Proietto writes:
> On 06/07/2016 11:26, "Aaron Conole" wrote:
>
>>Ansis Atteka writes:
>>
>>> On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote:
Aaron Conole writes:
On Wed, Jul 6, 2016 at 4:34 PM, Jesse Gross wrote:
> On Wed, Jul 6, 2016 at 3:44 PM, pravin shelar wrote:
>> On Tue, Jul 5, 2016 at 6:54 PM, Jesse Gross wrote:
>>> On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote:
On 06/07/2016 11:26, "Aaron Conole" wrote:
>Ansis Atteka writes:
>
>> On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote:
>>> Aaron Conole writes:
>>>
Daniele Di Proietto writes:
Patched datapaths that are no longer referenced should be removed from
the patched_datapaths map; otherwise incorrect state references for a
patched datapath may be used and also datapaths that are absent will be
interpreted as present.
Signed-off-by: Darrell Ball
---
On 06/07/2016 17:28, "Aaron Conole" wrote:
>Daniele Di Proietto writes:
>
>> Fixes: c9b9d6dfc06c("INSTALL.DPDK: Refactor DPDK install guide, add
>> ADVANCED doc")
>>
>> Signed-off-by: Daniele Di Proietto
>> ---
>>
I took your version of the code (much clearer!) and pushed this to master.
Thanks,
Daniele
On 24/06/2016 21:38, "Ben Pfaff" wrote:
>On Fri, Jun 10, 2016 at 03:52:16PM -0700, Daniele Di Proietto wrote:
>> In the userspace datapath we use tap devices as internal netdev. The
>>
Flavio Leitner writes:
> On Wed, Jul 06, 2016 at 04:28:26PM -0400, Aaron Conole wrote:
>> Flavio Leitner writes:
>>
>> > On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote:
>> >> +ConditionPathIsReadWrite=/var/run/openvswitch/db.sock
>> >
>> >
Daniele Di Proietto writes:
> Fixes: c9b9d6dfc06c("INSTALL.DPDK: Refactor DPDK install guide, add
> ADVANCED doc")
>
> Signed-off-by: Daniele Di Proietto
> ---
> Makefile.am | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Makefile.am
Hi Ilya,
apologies for the delay
On 20/06/2016 07:22, "Ilya Maximets" wrote:
>On 11.06.2016 02:53, Daniele Di Proietto wrote:
>> On 02/06/2016 06:55, "Ilya Maximets" wrote:
>>
>>> Hi, Daniele.
>>> Thanks for review.
>>>
>>> On 02.06.2016
On 06/07/2016 16:49, "Guru Shetty" wrote:
>
>
>On 6 July 2016 at 16:41, Daniele Di Proietto
> wrote:
>
>Also update a comment.
>
>Signed-off-by: Daniele Di Proietto
>
>
>Acked-by: Gurucharan
> Shetty
Pushed to
The original message was received at Thu, 7 Jul 2016 06:57:05 +0700
from [161.112.192.13]
- The following addresses had permanent fatal errors -
___
dev mailing list
dev@openvswitch.org
On 6 July 2016 at 16:41, Daniele Di Proietto wrote:
> Also update a comment.
>
> Signed-off-by: Daniele Di Proietto
>
Acked-by: Gurucharan Shetty
> ---
> python/ovs/vlog.py | 5 ++---
> 1 file changed, 2 insertions(+), 3
2016-07-06 15:32 GMT-07:00 Guru Shetty :
> On 6 July 2016 at 05:38, Paul Boca wrote:
>
> > Disable the syslog logger in case on Windows, '/dev/log' doesn't exist.
> > Seems like on Python34 a default handler is added to the logger and it
> > prints
> >
Also update a comment.
Signed-off-by: Daniele Di Proietto
---
python/ovs/vlog.py | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/python/ovs/vlog.py b/python/ovs/vlog.py
index 49210b8..48d52ad 100644
--- a/python/ovs/vlog.py
+++
Investigation found that Some of the occasional failures in the
"ovn -- vtep: 3 HVs, 1 VIFs/HV, 1 GW, 1 LS" test case are caused
by ovs-vswitchd crashing with SIGSEGV. It turns out that the
crash occurrs when the number of netdev-dummy passive connections
transitions from 1 to 2. When xrealloc()
On Wed, Jul 6, 2016 at 3:44 PM, pravin shelar wrote:
> On Tue, Jul 5, 2016 at 6:54 PM, Jesse Gross wrote:
>> On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote:
>>> diff --git a/acinclude.m4 b/acinclude.m4
>>> index 263c31d..05b5f48 100644
On Wed, Jul 06, 2016 at 04:28:26PM -0400, Aaron Conole wrote:
> Flavio Leitner writes:
>
> > On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote:
> >> +ConditionPathIsReadWrite=/var/run/openvswitch/db.sock
> >
> > Interesting, so we start ovs-vswitchd only when
Fixes: c9b9d6dfc06c("INSTALL.DPDK: Refactor DPDK install guide, add
ADVANCED doc")
Signed-off-by: Daniele Di Proietto
---
Makefile.am | 1 +
1 file changed, 1 insertion(+)
diff --git a/Makefile.am b/Makefile.am
index b2c1126..be42921 100644
--- a/Makefile.am
+++
I applied this to master, thanks
On 27/06/2016 06:28, "Ilya Maximets" wrote:
>Current implementarion of TX packet's queueing is broken in several ways:
>
> * TX queue flushing implemented on receive assumes that all
> core_id-s are sequential and starts
On Tue, Jul 5, 2016 at 7:57 PM, Jesse Gross wrote:
> On Fri, Jul 1, 2016 at 5:59 PM, Pravin B Shelar wrote:
>> diff --git a/datapath/linux/compat/geneve.c b/datapath/linux/compat/geneve.c
>> index 2ed9ad1..dbb220f 100644
>> --- a/datapath/linux/compat/geneve.c
I applied this to branch-2.5, thanks
2016-07-04 1:10 GMT-07:00 Bodireddy, Bhanuprakash <
bhanuprakash.bodire...@intel.com>:
> >-Original Message-
> >From: Stokes, Ian
> >Sent: Wednesday, June 29, 2016 2:32 PM
> >To: dev@openvswitch.org
> >Cc: Bodireddy, Bhanuprakash
On Tue, Jul 5, 2016 at 6:54 PM, Jesse Gross wrote:
> On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote:
>> diff --git a/acinclude.m4 b/acinclude.m4
>> index 263c31d..05b5f48 100644
>> --- a/acinclude.m4
>> +++ b/acinclude.m4
>> @@ -556,6 +556,7 @@
On Tue, Jul 5, 2016 at 6:07 PM, Jesse Gross wrote:
> On Fri, Jul 1, 2016 at 5:59 PM, Pravin B Shelar wrote:
>> diff --git a/datapath/linux/compat/include/net/ip6_tunnel.h
>> b/datapath/linux/compat/include/net/ip6_tunnel.h
>> index ce65087..565b69b 100644
>>
On 6 July 2016 at 05:38, Paul Boca wrote:
> Disable the syslog logger in case on Windows, '/dev/log' doesn't exist.
> Seems like on Python34 a default handler is added to the logger and it
> prints
> even if no handler is set by us.
>
> Signed-off-by: Paul-Daniel
I applied the series to master, thanks
2016-07-06 6:06 GMT-07:00 Bhanuprakash Bodireddy <
bhanuprakash.bodire...@intel.com>:
> This patchset refactors the present INSTALL.DPDK.md guide.
>
> The INSTALL guide is split in to two documents named INSTALL.DPDK and
> INSTALL.DPDK-ADVANCED. The former
Compliments
I am the personnel department manager and I am appealing to you in the name of
the large-scale and first-rate partnership.
Our company is engaged in different areas of activity, such as:
- real estate
- companies setting-up and winding-up
- supporting business in Europe and
I'm not a windows expert, but this also solves a problem I was having on a
linux system chroot where /dev/log is absent.
I pushed this patch to master, thanks,
Daniele
2016-07-06 5:38 GMT-07:00 Paul Boca :
> Disable the syslog logger in case on Windows, '/dev/log'
On Tue, Jul 5, 2016 at 4:54 PM, Jesse Gross wrote:
> On Fri, Jul 1, 2016 at 5:59 PM, Pravin B Shelar wrote:
>> diff --git a/datapath/linux/compat/include/linux/udp.h
>> b/datapath/linux/compat/include/linux/udp.h
>> new file mode 100644
>> index
On Tue, Jul 5, 2016 at 4:40 PM, Jesse Gross wrote:
> On Fri, Jul 1, 2016 at 5:58 PM, Pravin B Shelar wrote:
>> diff --git a/datapath/linux/compat/geneve.c b/datapath/linux/compat/geneve.c
>> index db1c713..a7229c8 100644
>> --- a/datapath/linux/compat/geneve.c
On Wed, Jul 06, 2016 at 02:40:01PM -0400, Aaron Conole wrote:
> Flavio Leitner writes:
>
> > On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote:
> >> diff --git a/rhel/etc_sysconfig_network-scripts_ifup-ovs
> >> b/rhel/etc_sysconfig_network-scripts_ifup-ovs
> >>
Just add a comment in the code so people know.
Acked-by: Alin Gabriel Serdean
> -Mesaj original-
> De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca
> Trimis: Wednesday, July 6, 2016 3:38 PM
> Către: dev@openvswitch.org
> Subiect:
Flavio Leitner writes:
> On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote:
>> This commit builds upon some of the recent ovs-ctl changes to build a
>> more integrated systemd setup. A new service (openvswitch-network) is
>
> I think you renamed to 'ovs-vswitchd'
On Fri, Jul 01, 2016 at 05:26:23PM -0400, Aaron Conole wrote:
> This commit builds upon some of the recent ovs-ctl changes to build a
> more integrated systemd setup. A new service (openvswitch-network) is
I think you renamed to 'ovs-vswitchd' after we talked offline :-)
> added to track the
It's ok with me to skip the tests for the moment, but please add a comment why
are you skipping them i.e. "echo $! gives shell pid instead of parent process"
Otherwise:
Acked-by: Alin Gabriel Serdean
> -Mesaj original-
> De la: dev
Ben Pfaff writes:
> On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote:
>> This commit allows the ovs-ctl command to spawn daemons without the
>> internal process monitor. This is useful when integrating with,
>> ex. systemd, which provides its own monitoring facilities.
Flavio Leitner writes:
> On Fri, Jul 01, 2016 at 09:32:24PM -0700, Ben Pfaff wrote:
>> On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote:
>> > This commit allows the ovs-ctl command to spawn daemons without the
>> > internal process monitor. This is useful when
Flavio Leitner writes:
> On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote:
>> Currently, openvswitch.service calls out to start
>> openvswitch-nonetwork.service. However, openvswitch-nonetwork.service
>> is better called ovsdb-server, since that is truly
+ AT_CHECK([$3 $srcdir/test-unixctl.py --log-file=`pwd`/log --pidfile \
+ --detach 2>/dev/null], [0], [], [ignore])
Either ignore stderr or 2>/dev/null. In case [ignore] doesn't work please add a
comment so others will be aware of the problem
Alin.
> -Mesaj original-
> De la: dev
We discussed this issue offline root cause seems to be:
https://bugs.python.org/issue6926
https://github.com/openvswitch/ovs/blob/master/python/ovs/socket_util.py#L309
Needs a commit message update.
Otherwise:
Acked-by: Alin Gabriel Serdean
> -Mesaj
Ansis Atteka writes:
> On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote:
>> Aaron Conole writes:
>>
>>> Daniele Di Proietto writes:
>>>
On 10/06/2016 10:51, "Aaron Conole" wrote:
From: Lorand Jakab
Implementation of the pop_eth and push_eth actions in the kernel, and
layer 3 flow support.
This doesn't actually do anything yet as no layer 2 tunnel ports are
supported yet. The original patch by Lorand was against the Open vSwitch
tree which has L2 LISP
* Set skb protocol based on contents of packet. I have observed this is
necessary to get actual protocol of a packet when it is injected into an
internal device e.g. by libnet in which case skb protocol will be set to
ETH_ALL.
* Set the mac_len which has been observed to not be set up
Allow push and pop mpls actions to act on layer 3 packets by teaching
them not to access non-existent L2 headers of such packets.
Signed-off-by: Simon Horman
---
v11
* group l2 code in pop_mpls()
v10
* Limit scope of hdr in {push,pop}_mpls()
v9
* New Patch
---
This helper resets the mac_header of an skb to a state where
skb_transport_header_was_set() will return false.
This is intended to be used with packets received on
ARPHRD_NONE devices without an Ethernet header in the inner packet.
It allows skb_transport_header_was_set to be subsequently used to
At the core of this patch set is removing the assumption in Open vSwitch
datapath that all packets have Ethernet header. Support for layer 3 GRE
tunnels is also added by this patchset.
The implementation relies on the presence of pop_eth and push_eth actions
in datapath flows to facilitate adding
unset rather than reset mach header for non-TEB packets received by an
ipgre device. This allows skb_transport_header_was_set to be subsequently
used to differentiate TEB and non-TEB packets recieved on an ipgre device.
Signed-off-by: Simon Horman
---
v11
* New patch
From: Lorand Jakab
These actions will allow L2->L3 and L3->L2 switching, and are supposed
to be added to flows installed in the datapath transparently by
ovs-vswitchd.
Signed-off-by: Lorand Jakab
Signed-off-by: Simon Horman
Add support for layer 3 GRE vports (non-tap aka non-VTEP).
This makes use of a vport mode configuration for the existing (tap/VTEP)
GRE vports.
In order to differentiate packets for two different types of GRE vports a
new flow key attribute, OVS_KEY_ATTR_NEXT_BASE_LAYER, is used. It is
intended
From: Lorand Jakab
This commit relaxes the assumption that all packets have an Ethernet
header, and adds support for layer 3 flows. For each packet received on
the Linux kernel datapath the l2 and l3 members of struct ofpbuf are
intialized appropriately, and some functions
This series implements support for layer 3 encapsulated packets. At the
core of this change is removing the assumption that all packets/flows have
Ethernet header. Support for layer 3 packets in GRE tunnels is also added
by this patchset.
The implementation automatically adds appropriate pop_eth
On Fri, Jul 01, 2016 at 05:26:22PM -0400, Aaron Conole wrote:
> Currently, openvswitch.service calls out to start
> openvswitch-nonetwork.service. However, openvswitch-nonetwork.service
> is better called ovsdb-server, since that is truly nonetwork. This
> commit does make the file a bit of a
On Wed, Jul 6, 2016 at 7:24 AM, Aaron Conole wrote:
> Aaron Conole writes:
>
>> Daniele Di Proietto writes:
>>
>>> On 10/06/2016 10:51, "Aaron Conole" wrote:
>>>
Aaron Conole writes:
> -Original Message-
> From: Alin Serdean
> Sent: Wednesday, July 6, 2016 8:42 PM
> To: Paul Boca; dev@openvswitch.org
> Subject: RE: [ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for
> Windows
>
> > -Mesaj original-
> > De la: dev
> -Mesaj original-
> De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca
> Trimis: Wednesday, July 6, 2016 3:38 PM
> Către: dev@openvswitch.org
> Subiect: [ovs-dev] [PATCH V6 05/17] python tests: Added fcntl module for
> Windows
>
> This is needed for lockf function used
On Fri, Jul 01, 2016 at 09:32:24PM -0700, Ben Pfaff wrote:
> On Fri, Jul 01, 2016 at 05:26:21PM -0400, Aaron Conole wrote:
> > This commit allows the ovs-ctl command to spawn daemons without the
> > internal process monitor. This is useful when integrating with,
> > ex. systemd, which provides
Hi Alin!
Comments inlined.
Thanks,
Paul
> -Original Message-
> From: Alin Serdean
> Sent: Wednesday, July 6, 2016 8:18 PM
> To: Paul Boca; dev@openvswitch.org
> Subject: RE: [ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for
> Windows
>
> > -Mesaj original-
> > De
Acked-by: Alin Gabriel Serdean
> -Mesaj original-
> De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca
> Trimis: Wednesday, July 6, 2016 3:38 PM
> Către: dev@openvswitch.org
> Subiect: [ovs-dev] [PATCH V6 04/17] python tests: Fixed
> -Mesaj original-
> De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca
> Trimis: Wednesday, July 6, 2016 3:38 PM
> Către: dev@openvswitch.org
> Subiect: [ovs-dev] [PATCH V6 03/17] python tests: Fixed ctl file name for
> Windows
>
> On Windows the CTL file doesn't contain
The code is sound but looks a bit messy. What do you think about?
@@ -15,6 +15,7 @@
import atexit
import os
import signal
+import sys
import ovs.vlog
@@ -129,8 +130,12 @@ def _init():
if not _inited:
_inited = True
-for signr in (signal.SIGTERM, signal.SIGINT,
-
On Mon, Jul 4, 2016 at 11:19 AM, Thadeu Lima de Souza Cascardo
wrote:
> ofproto_port_open_type should be used for netdev_open, but not for other
> tests.
> For example, STP/RSTP check for interfaces of internal type, but that check
> will
> fail when the netdev datapath is
Hi Alin!
Please see my comment inline.
Thanks,
Paul
> -Original Message-
> From: Alin Serdean
> Sent: Wednesday, July 6, 2016 6:08 PM
> To: Paul Boca; dev@openvswitch.org
> Subject: RE: [PATCH V6 15/17] python tests: Skip ovn-controller-vtep tests on
> Windows
>
> Just out of curiosity
Just out of curiosity why is this test is skipped on Windows?
Since this patch was skipped thus far I will acknowledge the patch and we can
fix the test if we can in another.
Acked-by: Alin Gabriel Serdean
Thanks,
Alin.
> -Mesaj original-
> De la: dev
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Hi Paul,
Thanks for your information. I will fix it.
Bests,
Daniel
On Jul 6, 2016, at 8:37 PM, Paul Boca
> wrote:
Hi Daniel,
Thanks for the fast reply! I thought that it is a problem in Windows
implementation
or some sort of
Hi Mark,
I observed the same issue using Ubuntu 16.04 server with 4.4.0-28-generic
kernel.
Thanks for looking into this.
Charlie
On Tue, Jul 5, 2016 at 5:04 PM, Kavanagh, Mark B
wrote:
> Hi Charlie,
>
>
>
> Thanks for providing the setup scripts and steps to
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Speech of welcome
I am the personnel department manager and I am appealing to you in the name of
the large-scale and first-rate partnership.
Our company is met in many departments, such as:
- real estate
- companies setting-up and winding-up
- supporting business in Europe and other
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Aaron Conole writes:
> Daniele Di Proietto writes:
>
>> On 10/06/2016 10:51, "Aaron Conole" wrote:
>>
>>>Aaron Conole writes:
>>>
Christian Ehrhardt writes:
>
The original message was received at Wed, 6 Jul 2016 22:16:07 +0800
from lists.berlios.de [72.135.101.196]
- The following addresses had permanent fatal errors -
___
dev mailing list
dev@openvswitch.org
"Wojciechowicz, RobertX" writes:
> Hi Ben,
>
>
>> -Original Message-
>> From: Ben Pfaff [mailto:b...@ovn.org]
>> Sent: Tuesday, July 5, 2016 5:07 PM
>> To: Wojciechowicz, RobertX
>> Cc: dev@openvswitch.org
>> Subject: Re:
As a personal preference I think the comment:
> + # This test won't work as-is on Windows because Windows doesn't allow
> + # files that are open to be renamed.
is sufficient.
Acked-by: Alin Gabriel Serdean
> -Mesaj original-
> De la: dev
>- With SMT enabled, one physical core appears as two logical cores
>- which can improve performance.
>+ - OVS current development can be clone using 'git' tool
>
>- SMT can be utilized to add additional pmd threads without consuming
>- additional physical cores. Additional pmd threads
>-Original Message-
>From: Daniele Di Proietto [mailto:diproiet...@ovn.org]
>Sent: Wednesday, July 6, 2016 7:05 AM
>To: Bodireddy, Bhanuprakash
>Cc: Lance Richardson ; dev@openvswitch.org
>Subject: Re: [ovs-dev] [PATCH v8 2/2]
Add INSTALL.DPDK-ADVANCED document that is forked off from original
INSTALL.DPDK guide. This document is targeted at users looking for
optimum performance on OVS using dpdk datapath.
Signed-off-by: Bhanuprakash Bodireddy
Acked-by: Flavio Leitner
Refactor the INSTALL.DPDK in to two documents named INSTALL.DPDK and
INSTALL.DPDK-ADVANCED. While INSTALL.DPDK document shall facilitate the
novice user in setting up the OVS DPDK and running it out of box, the
ADVANCED document is targeted at expert users looking for the optimum
performance
This patchset refactors the present INSTALL.DPDK.md guide.
The INSTALL guide is split in to two documents named INSTALL.DPDK and
INSTALL.DPDK-ADVANCED. The former document is simplified with emphasis
on installation, basic testcases and targets novice users. Sections on
system configuration,
This patch needs to be applied last.
Acked-by: Alin Gabriel Serdean
> -Mesaj original-
> De la: Paul Boca
> Trimis: Wednesday, July 6, 2016 3:39 PM
> Către: dev@openvswitch.org
> Cc: Alin Serdean
> Subiect: [PATCH V6
> On Jul 1, 2016, at 8:30 PM, Ben Pfaff wrote:
>
> On Tue, Jun 21, 2016 at 09:57:15AM -0700, Jarno Rajahalme wrote:
>> Add '--noproxy' parameter to 'wget' so that the test cases do not try
>> to use a proxy that may be configured in the test environment.
>>
>> Signed-off-by:
Disable the syslog logger in case on Windows, '/dev/log' doesn't exist.
Seems like on Python34 a default handler is added to the logger and it prints
even if no handler is set by us.
Signed-off-by: Paul-Daniel Boca
---
V3: Initial commit
V4: No changes
V5: No
The tests are not intended to run on Windows.
Until now they were skipped based on $HAVE_PYTHON only.
Signed-off-by: Paul-Daniel Boca
---
V3: Initial commit
V4: No changes
V5: No changes
V6: No changes
---
tests/ovn-controller-vtep.at | 1 +
1 file changed, 1
On Windows if a file is opened by an application for writing, we cannot move
it until all handles to that file are closed.
Signed-off-by: Paul-Daniel Boca
---
V3: Initial commit
V4: No changes
V5: No changes
V6: Removed code that disables 'vlog/close' tests.
---
This patch removes the code which disables Python tests to be run on
Windows.
Signed-off-by: Alin Gabriel Serdean
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
test-unixctl.py and test-vlog.py outputs on stderr and on Windows
stderr is not overriden by every AT_CHECK call, the logs are only
apended to the file and subsequent AT_CHECKs get errors from previous
call.
Signed-off-by: Paul-Daniel Boca
---
V3: Initial commit
V4:
AF_UNIX sockets are not supported on Windows.
Instead of an AF_UNIX socket use localhost tcp connections to communicate
between components. This makes the python sockets compatible with
the ones used in Windows applications.
In case the socket returns WSAEWOULDBLOCK, it is replaced by EAGAIN
Used subprocess.Popen instead os.fork (not implemented on windows)
and repaced of os.pipe with Windows pipes.
To be able to identify the child process I added an extra parameter
to daemon process '--pipe-handle', this parameter also contains
the parent Windows pipe handle, used by the child to
On windows a path containint ':' is considered an absolute path.
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
python/ovs/util.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
TCP6 sockets are not supported in python on Windows.
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
tests/ovsdb-idl.at | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/ovsdb-idl.at
There is a difference between POSIX pid and Windows pid, not all the time are
equal.
On Windows when a python script is started, a sh command is triggered as the
parent
for script. So when we try to get the daemon pid with 'echo $!', this will get
the pid of sh
not of python.exe as expected.
On Windows if this exception is triggered then it will raise an exception while
in the
exception handler.
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
python/ovs/poller.py | 5 +
1 file
IPsec is not used on Windows yet, and the services used by this test
are non-existent on Windows.
Also the paths in the test do not exist on Windows.
Signed-off-by: Paul-Daniel Boca
Acked-by: Alin Gabriel Serdean
---
V2: No changes
This is needed for lockf function used to lock the PID file on Windows.
ioctl and fcntl functions are not implemented at this time because they are
not used by any script.
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No
SIGHUP and SIGALRM are not available on Windows.
Signed-off-by: Paul-Daniel Boca
---
V2: Fixed Python function inet_open_active, treat WSAEWOULDBLOCK error as
EINPROGRESS on Windows
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
On Windows the CTL file doesn't contain the pid of the process.
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
python/ovs/unixctl/__init__.py | 10 --
1 file changed, 8 insertions(+), 2
For bogus pid file path, use a windows-like file path.
Signed-off-by: Paul-Daniel Boca
---
V2: No changes
V3: No changes
V4: No changes
V5: No changes
V6: No changes
---
tests/unixctl-py.at | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff
1 - 100 of 121 matches
Mail list logo