Hi,
I have some initial questions in this design.
> On Sep 30, 2015, at 1:56 PM, Ben Pfaff wrote:
>
> This is a proposed plan for logical L3 in OVN. It is not entirely
> complete but it includes many important details and I believe that it moves
> planning forward.
>
> +* L3 support
> +
> +**
On Thu, 1 Oct 2015 22:19:10 -0700, Pravin Shelar wrote:
> On Thu, Oct 1, 2015 at 9:08 AM, Jiri Benc wrote:
> > This is for IPv4 only at this point. Only vxlan is recognized for now.
> >
> > The usage is as follows:
> >
> > ip l a type vxlan metadata dstport 4789
> > ovs-vsctl add-port ovsbridge vx
On Thu, 1 Oct 2015 17:11:56 -0700, Pravin Shelar wrote:
> I dont see point of adding this code when IPv6 sampling not support by
> the patch series.
It was requested by Jesse:
http://article.gmane.org/gmane.linux.network/380348
I don't mind leaving this and the previous patch out, it's actually
w
The original message was received at Fri, 2 Oct 2015 12:59:36 +0700 from
openvswitch.org [43.197.74.108]
- The following addresses had permanent fatal errors -
dev@openvswitch.org
___
dev mailing list
dev@openvswitch.org
http://openvswitch.or
On Thu, 1 Oct 2015 17:11:43 -0700, Pravin Shelar wrote:
> On Thu, Oct 1, 2015 at 7:44 AM, Jiri Benc wrote:
> > +static inline void ip_tunnel_key_init(struct ip_tunnel_key *key,
> > + __be32 saddr, __be32 daddr,
> > + u8 tos, u
On Thu, Oct 1, 2015 at 9:08 AM, Jiri Benc wrote:
> This is for IPv4 only at this point. Only vxlan is recognized for now.
>
> The usage is as follows:
>
> ip l a type vxlan metadata dstport 4789
> ovs-vsctl add-port ovsbridge vxlan0
>
> Now, the encapsulation can be specified using ovs-ofctl, e.g.
L
6Ãü<ô²ò!´«ìïifGÎ"ïâ¯4\!ùB§×õÔç2`æëýÁjI
CÈñU-|Ë
¿¼^ÝØv¥³~²HÆÖbÝpf¬Î®CcΪҥ5¯É¿ÂâÕIùÖu˨çç ìSwÑÀæ.¶´
òiä¸É!
¶°´dÞG©¢¼mj zñvó?h¥)ÙÕ§
/aY3µ*½Ö´ïyòlól!§ÑP¦Ð/²ÔIí&°¦Á¹åPYrèÖ©ê#çÔ6´ØÁRr>Ø¿Óhɾ\åtwÞÚl
ea5/×ÂGìåìë;¬î·ã"%ä[ɦದ$¸h®±ñ±rý0Qèg4ÇFD¸áÏâ
âdͬÌ-([
On Thu, Oct 1, 2015 at 6:06 PM, Russell Bryant wrote:
> On 10/01/2015 06:47 PM, Andy Zhou wrote:
>>> Our default systemd unit files don't make use of the --user option that
>>> requires this library, but conceivably someone may want to customize
>>> them and use this option.
>>
>> Any down side if
On 10/01/2015 06:47 PM, Andy Zhou wrote:
>> Our default systemd unit files don't make use of the --user option that
>> requires this library, but conceivably someone may want to customize
>> them and use this option.
>
> Any down side if we change the unit files to make use of --user option
> by d
On 10/01/2015 06:31 PM, Flavio Leitner wrote:
> Commit e91b927d8 (lib/daemon: support --user option for all OVS daemon)
> added optional usage of the libcap-ng library. It's packaged in Fedora,
> so go ahead and added it by default to the Fedora spec file.
>
> Our default systemd unit files don't
> On Oct 1, 2015, at 12:07 PM, Bruce Davie wrote:
>
> v1 -> v2: split typos into separate patch; document VLAN zero meaning
> for VLAN bindings and statistics; update VTEP ACL documentation in response
> to Justin's review
Thanks. I pushed the series along with the two ACKs from Russell.
--Ju
On Thu, Oct 1, 2015 at 7:44 AM, Jiri Benc wrote:
> For compat tunnel interfaces, reject IPv6 keys.
>
> Also fixes a related thinko in vport-vxlan: upcall->egress_tun_info is not
> yet set at the point where it is used, thus the obtained family is
> incorrect. As this is IPv4 anyway, just use AF_IN
On Thu, Oct 1, 2015 at 7:44 AM, Jiri Benc wrote:
> We already have ip_tunnel_key_init, this does the same for IPv6. Will be
> used also by openvswitch in the next patch.
>
> Signed-off-by: Jiri Benc
> ---
> New patch in v2 of the set.
> ---
> include/net/dst_metadata.h | 18 ++
>
On Thu, Oct 1, 2015 at 7:44 AM, Jiri Benc wrote:
> Add netlink attributes for IPv6 tunnel addresses. This enables IPv6 support
> for tunnels.
>
> Signed-off-by: Jiri Benc
> ---
> v2: Changed description. Rebased tun_proto assignment in ip_tun_from_nlattr
> to accomodate changes in the previou
On Thu, Oct 1, 2015 at 7:44 AM, Jiri Benc wrote:
> Store tunnel protocol (AF_INET or AF_INET6) in sw_flow_key. This field now
> also acts as an indicator whether the flow contains tunnel data (this was
> previously indicated by tun_key.u.ipv4.dst being set but with IPv6 addresses
> in an union wit
> Our default systemd unit files don't make use of the --user option that
> requires this library, but conceivably someone may want to customize
> them and use this option.
Any down side if we change the unit files to make use of --user option
by default, like how Apache runs?
On Thu, Oct 01, 2015 at 03:35:05PM -0700, Andy Zhou wrote:
> On Thu, Oct 1, 2015 at 1:04 PM, Russell Bryant wrote:
> > Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
> > the libcap-ng library. It's packaged in Fedora, so go ahead and added
> > it to the Fedora spec file.
On Thu, Oct 1, 2015 at 1:04 PM, Russell Bryant wrote:
> Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
> the libcap-ng library. It's packaged in Fedora, so go ahead and added
> it to the Fedora spec file.
>
> Our default systemd unit files don't make use of the --user opt
On Thu, Oct 01, 2015 at 05:22:39PM -0300, Flavio Leitner wrote:
> On Thu, Oct 01, 2015 at 04:00:09PM -0400, Russell Bryant wrote:
> > On 10/01/2015 03:54 PM, Ben Pfaff wrote:
> > > On Thu, Oct 01, 2015 at 03:50:35PM -0400, Russell Bryant wrote:
> > >> Commit e91b927d8966bfcb9768225392324dde4fd7d7f6
Commit e91b927d8 (lib/daemon: support --user option for all OVS daemon)
added optional usage of the libcap-ng library. It's packaged in Fedora,
so go ahead and added it by default to the Fedora spec file.
Our default systemd unit files don't make use of the --user option that
requires this librar
Signed-off-by: Gurucharan Shetty
---
ovn/utilities/ovn-ctl | 40
utilities/ovs-ctl.in | 41 +
utilities/ovs-lib.in | 42 ++
3 files changed, 63 insertions(+), 60 deletions(-)
Signed-off-by: Gurucharan Shetty
---
ovn/utilities/ovn-ctl | 3 ---
1 file changed, 3 deletions(-)
diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl
index a86f515..89c774e 100755
--- a/ovn/utilities/ovn-ctl
+++ b/ovn/utilities/ovn-ctl
@@ -234,9 +234,6 @@ case $command in
restart_co
Signed-off-by: Gurucharan Shetty
---
ovn/utilities/ovn-ctl | 6 --
utilities/ovs-ctl.in | 12 ++--
utilities/ovs-lib.in | 10 ++
3 files changed, 12 insertions(+), 16 deletions(-)
diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl
index 89c774e..9f16021 100755
---
Signed-off-by: Gurucharan Shetty
---
ovn/.gitignore | 1 +
ovn/automake.mk | 28
ovn/ovn-nb.ovsschema | 2 ++
ovn/ovn-sb.ovsschema | 2 ++
4 files changed, 33 insertions(+)
diff --git a/ovn/.gitignore b/ovn/.gitignore
index 5b3bc55..d971938 100644
--- a
A tag value of 0 is not used by containers running inside
VMs.
Signed-off-by: Gurucharan Shetty
---
ovn/controller/physical.c | 2 +-
ovn/ovn-nb.ovsschema | 2 +-
ovn/ovn-sb.ovsschema | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/ovn/controller/physical.c b/ovn
hi Alin,
The change looks good. However I have a few minor comments. Pls. take a look.
> On Sep 22, 2015, at 5:00 PM, Alin Serdean
> wrote:
>
> This patch adds OVS_KEY_ATTR_TCP_FLAGS to our flow mechanism.
>
> Also clean unecesarry parts of code.
>
> Signed-off-by: Alin Gabriel Serdean
> ---
On Thu, Oct 01, 2015 at 04:00:09PM -0400, Russell Bryant wrote:
> On 10/01/2015 03:54 PM, Ben Pfaff wrote:
> > On Thu, Oct 01, 2015 at 03:50:35PM -0400, Russell Bryant wrote:
> >> Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
> >> the libcap-ng library. It's packaged in F
On 10/01/2015 04:07 PM, Andy Zhou wrote:
>> I'm going to look into updating the RPM spec file for the new
>> dependency, as well.
> What do you plan to change there? I also plan to update the packaging scripts
> so that 1) it will create 'ovs' user and group on the box, 2) OVS daemons
> will run as
> I'm going to look into updating the RPM spec file for the new
> dependency, as well.
What do you plan to change there? I also plan to update the packaging scripts
so that 1) it will create 'ovs' user and group on the box, 2) OVS daemons
will run as the ovs user, using the facilities just committe
Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
the libcap-ng library. It's packaged in Fedora, so go ahead and added
it to the Fedora spec file.
Our default systemd unit files don't make use of the --user option that
requires this library, but conceivably someone may want
On 10/01/2015 03:54 PM, Ben Pfaff wrote:
> On Thu, Oct 01, 2015 at 03:50:35PM -0400, Russell Bryant wrote:
>> Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
>> the libcap-ng library. It's packaged in Fedora, so go ahead and added
>> it to the Fedora spec file.
>>
>> Our de
On 10/01/2015 03:07 PM, Bruce Davie wrote:
> The meaning of a value of zero in the VLAN field when mapping
> pairs to
> logical switches had not previously been specified in the VTEP schema. It is
> now
> clarified that a value of zero refers to untagged traffic.
>
> Signed-off-by: Bruce Davie
On Thu, Oct 01, 2015 at 03:50:35PM -0400, Russell Bryant wrote:
> Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
> the libcap-ng library. It's packaged in Fedora, so go ahead and added
> it to the Fedora spec file.
>
> Our default systemd unit files don't make use of the
On 10/01/2015 03:07 PM, Bruce Davie wrote:
> Correct a number of typos in vtep.xml
>
> Signed-off-by: Bruce Davie
Acked-by: Russell Bryant
--
Russell Bryant
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Commit e91b927d8966bfcb9768225392324dde4fd7d7f6 added optional usage of
the libcap-ng library. It's packaged in Fedora, so go ahead and added
it to the Fedora spec file.
Our default systemd unit files don't make use of the --user option that
requires this library, but conceivably someone may want
Andy, no problem! It was an easy fix. That's the life of working
against master. :-)
I'm going to look into updating the RPM spec file for the new
dependency, as well.
--
Russell Bryant
On 10/01/2015 02:48 PM, Andy Zhou wrote:
> Russell,
>
> Thanks for the fix. I have libcap-ng installed on
The meaning of a value of zero in the VLAN field when mapping
pairs to
logical switches had not previously been specified in the VTEP schema. It is now
clarified that a value of zero refers to untagged traffic.
Signed-off-by: Bruce Davie
---
vtep/vtep.xml | 7 +--
1 file changed, 5 inserti
Correct a number of typos in vtep.xml
Signed-off-by: Bruce Davie
---
vtep/vtep.xml | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/vtep/vtep.xml b/vtep/vtep.xml
index ff8d0fe..0450999 100644
--- a/vtep/vtep.xml
+++ b/vtep/vtep.xml
@@ -367,7 +367,7 @@
Two new tables are added to the VTEP schema, for ACL entries and
ACLs (which are groups of entries). The physical port table is modified
to allow ACLs to be associated with ports, and the logical router table
is modified to allow ACLs to be attached to logical router ports.
Signed-off-by: Bruce Da
v1 -> v2: split typos into separate patch; document VLAN zero meaning
for VLAN bindings and statistics; update VTEP ACL documentation in response
to Justin's review
Bruce Davie (3):
vtep: fix typos
vtep: Document the meaning of VLAN zero for vlan_bindings
vtep: add ACLs to VTEP schema
vtep
Russell,
Thanks for the fix. I have libcap-ng installed on my system and forgot
to test compiling without it. Sorry.
andy
On Thu, Oct 1, 2015 at 8:52 AM, Ben Pfaff wrote:
> On Thu, Oct 01, 2015 at 11:29:16AM -0400, Russell Bryant wrote:
>> The function daemon_become_new_user_linux was condition
This patch makes it possible use a localnet port for connecting to a
specific VLAN on a locally accessible network. The only logical
modeling change is that it is now valid to set the "tag" field on
logical ports with a type of "localnet". Previously, the "tag" field
was only use for child ports.
While working on OVN and OVN integration, I've collected a set of
scripts for quickly setting up simple test environments using
ovs-sandbox with OVN enabled. It seemed like they could be useful to
others for learning about OVN or doing quick testing.
This patch introduces an ovs-sandbox based tut
On Thu, Oct 01, 2015 at 09:14:18AM -0700, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Signed-off-by: Gurucharan Shetty
---
ovn/utilities/ovn-ctl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl
index f4fb7ed..a86f515 100755
--- a/ovn/utilities/ovn-ctl
+++ b/ovn/utilities/ovn-ctl
@@ -229,7 +229,7 @@ case $command in
Hi,
On 10/1/15 2:38 PM, niti Rohilla wrote:
> Hi All,
>
> I tried to create a lisp tunnel on ovs stable version 2.3.2 and i am
> getting the following error:
>
> ovs-vsctl add-port br0 lisp0 -- set interface lisp0 type=lisp
> options:remote_ip=192.168.1.45 options:key=5001
> ovs-vsctl: Error detec
On Thu, 1 Oct 2015 18:08:51 +0200, Jiri Benc wrote:
> This is for IPv4 only at this point. Only vxlan is recognized for now.
>
> The usage is as follows:
>
> ip l a type vxlan metadata dstport 4789
> ovs-vsctl add-port ovsbridge vxlan0
^ should be "ovs0"
> Now, the encapsul
On Thu, Oct 01, 2015 at 08:59:49AM -0700, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
This is for IPv4 only at this point. Only vxlan is recognized for now.
The usage is as follows:
ip l a type vxlan metadata dstport 4789
ovs-vsctl add-port ovsbridge vxlan0
Now, the encapsulation can be specified using ovs-ofctl, e.g.:
ovs-ofctl add-flow ovs0 'in_port=2
actions=set_field:2->tun
Signed-off-by: Gurucharan Shetty
---
lib/daemon.h |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/lib/daemon.h b/lib/daemon.h
index cb663ca..4990415 100644
--- a/lib/daemon.h
+++ b/lib/daemon.h
@@ -90,8 +90,8 @@ pid_t read_pidfile(const char *name);
OPT_PIDFILE
On Thu, Oct 01, 2015 at 06:09:28PM +0530, RishiRaj Maulick wrote:
> We are trying to cross-compile and run the ovs-vswitchd on a machine that
> runs on powerPC architecture. We have cross-compiled the code successfully
> however we are hitting some issues :-
>
> 1) When we try to create the conf.d
I sent the patch too soon. I will send a v2.
On Thu, Oct 1, 2015 at 8:54 AM, Ben Pfaff wrote:
> On Thu, Oct 01, 2015 at 08:33:30AM -0700, Gurucharan Shetty wrote:
>> Signed-off-by: Gurucharan Shetty
>
> What's the error? I don't see it. Anyway, it's better without the
> extra backslash, so:
>
On Thu, Oct 01, 2015 at 08:33:30AM -0700, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty
What's the error? I don't see it. Anyway, it's better without the
extra backslash, so:
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
On Thu, Oct 01, 2015 at 11:13:06AM -0400, Russell Bryant wrote:
> Running "make dist-docs" to generate docs creates a "dist-docs"
> directory that should be ignored by git.
>
> Signed-off-by: Russell Bryant
Applied, thanks!
___
dev mailing list
dev@ope
On Thu, Oct 01, 2015 at 11:29:16AM -0400, Russell Bryant wrote:
> The function daemon_become_new_user_linux was conditionally defined but
> then used in code unconditionally. If HAVE_LIBCAPNG is not defined, the
> function would never be called, but it still must exist.
>
> Adjust the #if guard a
Signed-off-by: Gurucharan Shetty
---
lib/daemon.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/daemon.h b/lib/daemon.h
index cb663ca..a1d8300 100644
--- a/lib/daemon.h
+++ b/lib/daemon.h
@@ -91,7 +91,7 @@ pid_t read_pidfile(const char *name);
OPT_PIPE_HANDLE,
The function daemon_become_new_user_linux was conditionally defined but
then used in code unconditionally. If HAVE_LIBCAPNG is not defined, the
function would never be called, but it still must exist.
Adjust the #if guard around the function to be around the body of the
function instead of outsid
On 09/22/2015 08:51 PM, Andy Zhou wrote:
> OVS daemons can now support --user option to run as a non-root
> user with less privileges.
>
> See the manpage patch for more descriptions.
>
> Signed-off-by: Andy Zhou
FYI, this patch series seems to break the build for me.
With clang:
> lib/daemon
Running "make dist-docs" to generate docs creates a "dist-docs"
directory that should be ignored by git.
Signed-off-by: Russell Bryant
---
.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitignore b/.gitignore
index c96ceed..71bb21b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -49
Store tunnel protocol (AF_INET or AF_INET6) in sw_flow_key. This field now
also acts as an indicator whether the flow contains tunnel data (this was
previously indicated by tun_key.u.ipv4.dst being set but with IPv6 addresses
in an union with IPv4 ones this won't work anymore).
The new field was a
For compat tunnel interfaces, reject IPv6 keys.
Also fixes a related thinko in vport-vxlan: upcall->egress_tun_info is not
yet set at the point where it is used, thus the obtained family is
incorrect. As this is IPv4 anyway, just use AF_INET.
Signed-off-by: Jiri Benc
---
New patch in v2 of the s
Add netlink attributes for IPv6 tunnel addresses. This enables IPv6 support
for tunnels.
Signed-off-by: Jiri Benc
---
v2: Changed description. Rebased tun_proto assignment in ip_tun_from_nlattr
to accomodate changes in the previous patch.
---
include/uapi/linux/openvswitch.h | 2 +
net/ope
This builds on the previous work that added IPv6 support to lwtunnels and
adds IPv6 tunneling support to ovs.
To use IPv6 tunneling, there needs to be a metadata based tunnel net_device
created and added to the ovs bridge. Currently, only vxlan is supported by
the kernel, with geneve to follow sho
We already have ip_tunnel_key_init, this does the same for IPv6. Will be
used also by openvswitch in the next patch.
Signed-off-by: Jiri Benc
---
New patch in v2 of the set.
---
include/net/dst_metadata.h | 18 ++
include/net/ip_tunnels.h | 40 +++---
Hi ,
We are trying to cross-compile and run the ovs-vswitchd on a machine that
runs on powerPC architecture. We have cross-compiled the code successfully
however we are hitting some issues :-
1) When we try to create the conf.db file from the schema - it fails to
decode the JSON objects.
2) Then
Hi All,
I tried to create a lisp tunnel on ovs stable version 2.3.2 and i am
getting the following error:
ovs-vsctl add-port br0 lisp0 -- set interface lisp0 type=lisp
options:remote_ip=192.168.1.45 options:key=5001
ovs-vsctl: Error detected while setting up 'lisp0'. See ovs-vswitchd log
for det
66 matches
Mail list logo
