[ovs-dev] [PATCH ovn] ovn-nbctl: Update usage output for show command.

2015-04-22 Thread Russell Bryant
The show command takes an optional LSWITCH argument. In that case, it only prints the info for a single switch instead of all of them. Update the usage text to reflect that option. Signed-off-by: Russell Bryant --- ovn/ovn-nbctl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ovn/ovn

Re: [ovs-dev] [PATCH 2/2] ovn-nbctl: Add lport-set-port-security command.

2015-04-22 Thread Russell Bryant
Port security limits the addresses from which a logical port may send > packets and to which it may receive packets. See the > ovn-nb(5) documentation for the column="port_security" table="Logical_Port"/> colum

Re: [ovs-dev] [PATCH] ovn: Remove complete to-do items.

2015-04-22 Thread Russell Bryant
is in a minute. > _______ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 0/2] Update Fedora packaging for OVN.

2015-04-24 Thread Russell Bryant
/usr_share_openvswitch_scripts_systemd_ovnnorthd_sysconfig.template | 32 + 13 files changed, 495 insertions(+), 6 deletions(-) -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 1/2] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-04-24 Thread Russell Bryant
single host or small set of hosts. In general, that would look like: # ovn-ctl create_ovn_dbs # ovs-ctl start --extra-dbs="ovnnb.db ovnsb.db" # ovn-ctl start_northd --no-ovsdb-server # ovn-ctl start_controller Signed-off-by: Russell Bryant --- ovn/automake.mk

[ovs-dev] [PATCH ovn 2/2] fedora.spec: Create openvswitch-ovn package.

2015-04-24 Thread Russell Bryant
x27;" EOF # cat << EOF > /etc/sysconfig/ovnnorthd OPTIONS="--no-ovsdb-server" EOF # ovn-ctl create_ovn_dbs # systemctl start openvswitch # systemctl start ovnnorthd # systemctl start ovncontroller Signed-off-by: Russell Bryant --- rhel/au

Re: [ovs-dev] [PATCH ovn v3 01/10] ovn-sb.xml: Revise description of "actions" to match my current thoughts.

2015-04-28 Thread Russell Bryant
e it did before. It matches the doc update you made though. Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn v3 02/10] ovn-northd: Port security allows receiving packets to multicast/broadcast.

2015-04-28 Thread Russell Bryant
-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn v3 03/10] ovn: Back off on port_security specification and implementation.

2015-04-28 Thread Russell Bryant
" The old match in table 3 for output was: "outport == \"sw0-port1\" and is now: "outport == \"sw0-port1\" && eth.dst == {00:00:00:00:00:01}" so it appears to be doing what I would expect. Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn v3 04/10] ovn-nb: Mark some features as not yet implemented.

2015-04-28 Thread Russell Bryant
On 04/24/2015 06:34 PM, Ben Pfaff wrote: > Reported-by: Justin Pettit > Signed-off-by: Ben Pfaff > --- > ovn/ovn-nb.xml | 18 +++--- > 1 file changed, 15 insertions(+), 3 deletions(-) Acked-by: Russell Bryant --

Re: [ovs-dev] [PATCH ovn v3 05/10] lex: New functions lex_lookahead() and lexer_match_id().

2015-04-28 Thread Russell Bryant
); > +enum lex_type lexer_lookahead(const struct lexer *); > bool lexer_match(struct lexer *, enum lex_type); > +bool lexer_match_id(struct lexer *, const char *id); I think you meant to remove "id" here. Acked-by: Russell Bryant -- Russell Bryant __

Re: [ovs-dev] [PATCH ovn v3 07/10] ovn: Add tunnel_key concept to Bindings table, assign in ovn-northd.

2015-04-28 Thread Russell Bryant
insertions(+), 22 deletions(-) The code here looks correct and I also tested it. I was just wondering if you could comment on the choice of a 16 bit integer here instead of a UUID. My guess is that it has to do with where this ID will be used in a tunnel protocol, but it might be nice to capture

Re: [ovs-dev] [ovn 1/6] ovn-controller: Make integration bridge config part of general context.

2015-04-28 Thread Russell Bryant
> ovn/controller/ovn-controller.c | 44 -- > ovn/controller/ovn-controller.h |5 +++- > 3 files changed, 48 insertions(+), 34 deletions(-) Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing li

Re: [ovs-dev] [ovn 2/6] ovn-controller: Clarify that some config changes require a restart.

2015-04-28 Thread Russell Bryant
On 04/28/2015 01:14 AM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > ovn/controller/ovn-controller.8.xml |5 + > ovn/controller/ovn-controller.c |6 -- > 2 files changed, 9 insertions(+), 2 deletions(-) Acked-by: Russell Bryant --

Re: [ovs-dev] [ovn 3/6] ovn-controller: Don't log every interface missing "iface-id".

2015-04-28 Thread Russell Bryant
1 - > 1 files changed, 0 insertions(+), 1 deletions(-) Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn v3 06/10] ovn-northd: Set logical_datapath in each Bindings record.

2015-04-28 Thread Russell Bryant
The code looks good and I verified that the logical_datapath column is set on Bindings records now. Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn v3 05/10] lex: New functions lex_lookahead() and lexer_match_id().

2015-04-29 Thread Russell Bryant
On 04/28/2015 08:05 PM, Ben Pfaff wrote: > On Tue, Apr 28, 2015 at 03:34:31PM -0400, Russell Bryant wrote: >> On 04/24/2015 06:34 PM, Ben Pfaff wrote: >>> These functions will have their first users in upcoming commits. >>> >>> Signed-off-by: Ben Pfaf

Re: [ovs-dev] [PATCH ovn v3 07/10] ovn: Add tunnel_key concept to Bindings table, assign in ovn-northd.

2015-04-29 Thread Russell Bryant
On 04/28/2015 09:11 PM, Ben Pfaff wrote: > On Tue, Apr 28, 2015 at 05:36:50PM -0700, Justin Pettit wrote: >> >>> On Apr 28, 2015, at 5:21 PM, Ben Pfaff wrote: >>> >>> On Tue, Apr 28, 2015 at 03:53:03PM -0400, Russell Bryant wrote: >>>> >>>&

Re: [ovs-dev] [PATCH ovn v3 07/10] ovn: Add tunnel_key concept to Bindings table, assign in ovn-northd.

2015-04-29 Thread Russell Bryant
On 04/29/2015 12:22 PM, Justin Pettit wrote: > >> On Apr 29, 2015, at 6:34 AM, Russell Bryant wrote: >> >> I wonder how realistic it would be to have 65k ports and hit this as a >> limit. If your deployment has many containers in each VM, it seems like >> we

Re: [ovs-dev] [PATCH] test-ovsdb: Fix conditional statement.

2015-04-29 Thread Russell Bryant
by: Alex Wang The code looks fine, so: Acked-by: Russell Bryant ... but what system are you still supporting that doesn't have this? It looks like it was added in Python 2.5. So, RHEL 5 or some derivative? or something else? Just curious ... > --- > tests/test-ovsdb.py |5 ++

Re: [ovs-dev] [PATCH ovn v5 00/15] complete end-to-end OVN system

2015-04-30 Thread Russell Bryant
ernal_ids: {attached-mac="fa:16:3e:46:35:55", iface-id="04cb530f-5406-4f5d-b10e-82a5a07581a9", iface-status=active, vm-id="aabd7289-c27f-4bbb-81f6-decceec2d2a6"} ... $ ssh -i id_rsa_demo cirros@10.0.0.3 $ ping 10.0.0.4 PING 10.0.0.4 (10.0.0.4

[ovs-dev] [PATCH ovn 1/2] ovn-controller: Fix invalid br_int_name handling.

2015-05-01 Thread Russell Bryant
you set "ovn_bridge". If you relied on the default, the bug would not occur. Signed-off-by: Russell Bryant --- ovn/controller/ovn-controller.c | 11 +++ ovn/controller/ovn-controller.h | 2 +- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/ovn/controller/ovn-con

[ovs-dev] [PATCH ovn 2/2] ovn-controller: Free some more memory before exit.

2015-05-01 Thread Russell Bryant
Free a few pointers to malloc'd memory before exit. Signed-off-by: Russell Bryant --- ovn/controller/ovn-controller.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-controller.c index 778dd43..9d61a48 100644 ---

Re: [ovs-dev] [PATCH ovn v5 02/15] ovn-controller: Implement translation of OVN flows into OpenFlow.

2015-05-01 Thread Russell Bryant
continue; > +} It looks like I'm getting this message when deleting a logical datapath. It's pretty easy to replicate. $ ovn-nbctl lswitch-add foo $ ovn-nbctl lswitch-del foo 2015-05-01T19:25:23.282Z|8|pipeline|INFO|l

Re: [ovs-dev] [PATCH ovn v6 3/9] rconn: Only increment seqno when connection actually drops or reconnects.

2015-05-04 Thread Russell Bryant
This fixes the problem. > > rconn_get_connection_seqno() didn't have any existing users, but an > upcoming commit adds one. > > Signed-off-by: Ben Pfaff Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@open

Re: [ovs-dev] [PATCH ovn v6 4/9] ovs-sandbox: Use "secure" mode for br-int, in OVN mode.

2015-05-04 Thread Russell Bryant
On 05/01/2015 07:17 PM, Ben Pfaff wrote: > This is documented in ovn-architecture(7). > > Signed-off-by: Ben Pfaff Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn v6 5/9] match: Add MATCH_CATCHALL_INITIALIZER.

2015-05-04 Thread Russell Bryant
On 05/01/2015 07:17 PM, Ben Pfaff wrote: > An upcoming patch will add the first user. > > Signed-off-by: Ben Pfaff Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Testing OVN with OpenStack

2015-05-04 Thread Russell Bryant
7;re working on expanding the environment automation and documentation to support multi-host testing. That page will be updated as we make progress. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] Testing OVN with OpenStack

2015-05-04 Thread Russell Bryant
loser at OVN and OVS to verify and understand what's happening. In the meantime, I'll put some links to additional resources. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 0/2 v2] Update Fedora packaging for OVN.

2015-05-04 Thread Russell Bryant
hel/usr_lib_systemd_system_ovnnorthd.service | 13 + 9 files changed, 409 insertions(+), 7 deletions(-) -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 1/2 v2] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-05-04 Thread Russell Bryant
emd, but is still nice to provide as an option to enable running on other systems. Signed-off-by: Russell Bryant --- ovn/automake.mk | 1 + ovn/utilities/.gitignore| 1 + ovn/utilities/automake.mk | 9 ++ ovn/utilities/ovn-ctl | 277 ++

[ovs-dev] [PATCH ovn 2/2 v2] fedora.spec: Create openvswitch-ovn package.

2015-05-04 Thread Russell Bryant
running the following commands before starting or restarting openvswitch: # ovn-ctl create_ovn_dbs # cat << EOF > /etc/sysconfig/openvswitch OPTIONS="'--extra-dbs=ovnnb.db ovnsb.db'" EOF Signed-off-by: Russell Bryant --- rhel/automake.mk

[ovs-dev] [PATCH ovn] controller: Set chassis for container port.

2015-05-05 Thread Russell Bryant
being on that chassis, as well. Signed-off-by: Russell Bryant --- ovn/controller/bindings.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/ovn/controller/bindings.c b/ovn/controller/bindings.c index ef233c0..65054c9 100644 --- a/ovn/controller/bindings.c +++ b

Re: [ovs-dev] [PATCH ovn] controller: Set chassis for container port.

2015-05-05 Thread Russell Bryant
On 05/05/2015 02:26 PM, Justin Pettit wrote: > >> On May 5, 2015, at 6:59 AM, Russell Bryant wrote: >> >> ovn-controller updates the chassis column of the Bindings table in >> OVN_Southbound when a logical port appears on the local switch. A >> logical port th

[ovs-dev] [PATCH ovn v2] controller: Set chassis for container port.

2015-05-05 Thread Russell Bryant
being on that chassis, as well. Signed-off-by: Russell Bryant --- ovn/controller/bindings.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) v1->v2 - Applied suggestions from Justin. diff --git a/ovn/controller/bindings.c b/ovn/controller/bindings.c index ef233c0..e5b4

Re: [ovs-dev] [ovn] match: Fix compile of match initializer on GCC 4.6.1.

2015-05-06 Thread Russell Bryant
th the change, so: Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] OVN OpenStack integration status

2015-05-13 Thread Russell Bryant
l have it sorted out this week, though. That page will include the multi-node test instructions once it's all merged. I'm also aiming to write another update blog post this week if I can that will cover a detailed overview of a multi-node OpenStack+OVN environment with DevStack. -- Russe

Re: [ovs-dev] [PATCH] Start fixing some Python 3 compatibility issues

2015-05-13 Thread Russell Bryant
orrect to me. Thanks! Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] OVN OpenStack integration status

2015-05-13 Thread Russell Bryant
On 05/13/2015 11:57 AM, Justin Pettit wrote: > >> On May 13, 2015, at 6:48 AM, Russell Bryant wrote: >> >> There's some documentation for testing it with DevStack here: >> >> http://networking-ovn.readthedocs.org/en/latest/testing.html >> >> I

Re: [ovs-dev] OVN OpenStack integration status

2015-05-13 Thread Russell Bryant
On 05/13/2015 01:28 PM, Gurucharan Shetty wrote: > On Wed, May 13, 2015 at 10:19 AM, Russell Bryant wrote: >> On 05/13/2015 11:57 AM, Justin Pettit wrote: >>> >>>> On May 13, 2015, at 6:48 AM, Russell Bryant wrote: >>>> >>>> There&

Re: [ovs-dev] OVN OpenStack integration status

2015-05-14 Thread Russell Bryant
ng a single VM on each hypervisor which seemed to be working: http://blog.russellbryant.net/2015/05/14/an-ez-bake-ovn-for-openstack/ I'll apply your patch and add some more VMs to the environment and see what happens. -- Russell Bryant ___ dev

[ovs-dev] [PATCH ovn] ovs-sandbox: Fix ovn support.

2015-05-15 Thread Russell Bryant
The last merge from master broke ovs-sandbox OVN support. The rungdb function now takes an additional argument for whether or not the daemon should be automatically started under gdb. Signed-off-by: Russell Bryant --- tutorial/ovs-sandbox | 14 -- 1 file changed, 12 insertions

Re: [ovs-dev] [PATCH] ovn-northd: Add appctl support.

2015-05-23 Thread Russell Bryant
ed it in ovs-sandbox. Nice work. Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH ovn 1/2 v2] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-05-23 Thread Russell Bryant
On 05/22/2015 05:11 AM, Flavio Leitner wrote: > It seems reasonable to have the same scheme used by OVS in OVN. > Looks good to me. > Acked-by: Flavio Leitner Thanks for the review! I've added the ACK to this patch. -- Russell Bryant _

Re: [ovs-dev] [PATCH ovn 2/2 v2] fedora.spec: Create openvswitch-ovn package.

2015-05-23 Thread Russell Bryant
On 05/22/2015 05:31 AM, Flavio Leitner wrote: > On Mon, May 04, 2015 at 05:40:46PM -0400, Russell Bryant wrote: >> This patch creates a new subpackage for OVN, openvswitch-ovn. It also >> installs systemd unit files for ovn-controller and ovn-northd. >> >> If you

Re: [ovs-dev] OVN multiple MAC addresses on a single port

2015-05-26 Thread Russell Bryant
rity=50,metadata=0x1,dl_dst=00:00:00:00:00:02 > actions=set_field:0x2->reg7,resubmit(,18) > cookie=0x0, duration=343.574s, table=17, n_packets=0, n_bytes=0, > priority=0,metadata=0x1 actions=set_field:0x3->reg7,resubmit(,18) If you add more ports with "unknown", the logical Pipeline entry and corresponding flow just output to all ports with "unknown" set. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovs-vswitchd: Update documentation

2015-05-26 Thread Russell Bryant
of bridges and ports is decided > by > the availability of file descriptors. With the Linux kernel datapath, > creation > of a single bridge consumes 3 file descriptors and adding a port consumes > Looks good to me. I appreciate the inclusion of the hash for the co

Re: [ovs-dev] [PATCH 1/2] Increase prerequisite from Python 2.4 to Python 2.7.

2015-05-26 Thread Russell Bryant
st have to fork this library for OpenStack for now so we can carry a patch set to support Python 3. It's a last resort, but we can't really wait if we want to use this in OpenStack. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 2/2 v3] fedora.spec: Create openvswitch-ovn package.

2015-05-26 Thread Russell Bryant
running the following commands before starting or restarting openvswitch: # ovn-ctl create_ovn_dbs # cat << EOF > /etc/sysconfig/openvswitch OPTIONS="'--extra-dbs=ovnnb.db ovnsb.db'" EOF Signed-off-by: Russell Bryant --- rhel/automake.mk

[ovs-dev] [PATCH ovn 0/2 v3] Update Fedora packaging for OVN.

2015-05-26 Thread Russell Bryant
em_ovn-northd.service | 17 + 9 files changed, 413 insertions(+), 8 deletions(-) -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 1/2 v3] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-05-26 Thread Russell Bryant
emd, but is still nice to provide as an option to enable running on other systems. Signed-off-by: Russell Bryant Acked-by: Flavio Leitner --- ovn/automake.mk | 1 + ovn/utilities/.gitignore| 1 + ovn/utilities/automake.mk | 9 ++ ovn/util

Re: [ovs-dev] [PATCH 1/2] Increase prerequisite from Python 2.4 to Python 2.7.

2015-05-26 Thread Russell Bryant
On 05/26/2015 10:31 AM, Russell Bryant wrote: > On 05/21/2015 10:23 PM, Andy Hill wrote: >>> As a consequence, this requires dropping support for old versions of >>> XenServer. I don't expect that to be much of a problem. >> >> Unfortunately, the most recent

Re: [ovs-dev] [PATCH 1/2] Increase prerequisite from Python 2.4 to Python 2.7.

2015-05-26 Thread Russell Bryant
On 05/26/2015 10:14 PM, Ben Pfaff wrote: > On Tue, May 26, 2015 at 10:31:57AM -0400, Russell Bryant wrote: >> On 05/21/2015 10:23 PM, Andy Hill wrote: >>>> As a consequence, this requires dropping support for old versions of >>>> XenServer. I don'

Re: [ovs-dev] [PATCH ovn 2/2 v3] fedora.spec: Create openvswitch-ovn package.

2015-05-26 Thread Russell Bryant
On 05/26/2015 10:53 PM, Flavio Leitner wrote: > On Tue, May 26, 2015 at 01:06:59PM -0400, Russell Bryant wrote: >> This patch creates a new subpackage for OVN, openvswitch-ovn. It also >> installs systemd unit files for ovn-controller and ovn-northd. >> >> If you

Re: [ovs-dev] OVN OpenStack integration status

2015-05-26 Thread Russell Bryant
On 05/26/2015 10:25 PM, Ben Pfaff wrote: > On Thu, May 14, 2015 at 04:39:15PM -0400, Russell Bryant wrote: >> On 05/14/2015 03:57 PM, Gurucharan Shetty wrote: >>> For the problem in hand, I sent out I think is a fix yesterday. >>> https://www.mail-archive.com/dev@op

[ovs-dev] [PATCH ovn 1/2 v4] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-05-26 Thread Russell Bryant
emd, but is still nice to provide as an option to enable running on other systems. Signed-off-by: Russell Bryant Acked-by: Flavio Leitner --- ovn/automake.mk | 1 + ovn/utilities/.gitignore| 1 + ovn/utilities/automake.mk | 9 ++ ovn/util

[ovs-dev] [PATCH ovn 0/2 v4] Update Fedora packaging for OVN.

2015-05-26 Thread Russell Bryant
42 ++- rhel/usr_lib_systemd_system_ovn-controller.service | 16 + rhel/usr_lib_systemd_system_ovn-northd.service | 17 + 9 files changed, 413 insertions(+), 8 deletions(-) -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswit

[ovs-dev] [PATCH ovn 2/2 v4] fedora.spec: Create openvswitch-ovn package.

2015-05-26 Thread Russell Bryant
running the following commands before starting or restarting openvswitch: # ovn-ctl create_ovn_dbs # cat << EOF > /etc/sysconfig/openvswitch OPTIONS="'--extra-dbs=ovnnb.db ovnsb.db'" EOF Signed-off-by: Russell Bryant Acked-by: Flavio L

Re: [ovs-dev] [PATCH 1/2] Increase prerequisite from Python 2.4 to Python 2.7.

2015-05-26 Thread Russell Bryant
On 05/27/2015 12:11 AM, Russell Bryant wrote: > On 05/26/2015 10:14 PM, Ben Pfaff wrote: >> On Tue, May 26, 2015 at 10:31:57AM -0400, Russell Bryant wrote: >>> On 05/21/2015 10:23 PM, Andy Hill wrote: >>>>> As a consequence, this requires dropping support for ol

[ovs-dev] [PATCH ovn] controller: Fix crash on binding with no chassis.

2015-05-27 Thread Russell Bryant
The current code crashes when it encounters a binding that does not yet have a chassis set. This patch restores the intended behavior of ignoring it in that case. Signed-off-by: Russell Bryant --- ovn/controller/physical.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ovn/controller

[ovs-dev] [PATCH ovn] controller: Drop unknown datapath log message.

2015-05-28 Thread Russell Bryant
's nothing useful to be done with it. Signed-off-by: Russell Bryant --- ovn/controller/pipeline.c | 4 1 file changed, 4 deletions(-) diff --git a/ovn/controller/pipeline.c b/ovn/controller/pipeline.c index 58412e6..8ee92dc 100644 --- a/ovn/controller/pipeline.c +++ b/ovn/contro

Re: [ovs-dev] [PATCH ovn 1/2] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-05-29 Thread Russell Bryant
On 05/29/2015 04:47 PM, Ben Pfaff wrote: > On Fri, Apr 24, 2015 at 01:06:05PM -0400, Russell Bryant wrote: >> This patch introduces ovn-ctl, which is similar to ovs-ctl. I opted >> for a new script as everything in OVN so far is nicely isolated, so a >> new script seemed t

Re: [ovs-dev] [PATCH ovn 2/2] fedora.spec: Create openvswitch-ovn package.

2015-05-29 Thread Russell Bryant
On 05/29/2015 04:50 PM, Ben Pfaff wrote: > On Fri, Apr 24, 2015 at 01:06:06PM -0400, Russell Bryant wrote: >> This patch creates a new subpackage for OVN, openvswitch-ovn. It also >> installs systemd unit files for ovncontroller and ovnnorthd services. >> Finally, it i

Re: [ovs-dev] [PATCH ovn 2/2] fedora.spec: Create openvswitch-ovn package.

2015-06-02 Thread Russell Bryant
On 06/02/2015 09:24 AM, Flavio Leitner wrote: > On Fri, May 29, 2015 at 02:51:26PM -0700, Ben Pfaff wrote: >> On Fri, May 29, 2015 at 05:07:19PM -0400, Russell Bryant wrote: >>> On 05/29/2015 04:50 PM, Ben Pfaff wrote: >>>> On Fri, Apr 24, 2015 at 01:06:06PM -0400, Ru

[ovs-dev] OVN and OpenStack Provider Networks

2015-06-10 Thread Russell Bryant
base, as well. No schema changes have been identified. Entries in the Bindings table would be the same, except that the UUID in the logical_datapath column would refer to a Physical_Switch instead of a Logical_Switch. The contents of the Pipeline and the flows set up

Re: [ovs-dev] first OVN scale testing thoughts

2015-06-11 Thread Russell Bryant
tests together so that we have an idea > where we stand. +1! Thanks a bunch! I look forward to trying this out. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovsdb-idl: Document that the IDL always presents a consistent view.

2015-06-11 Thread Russell Bryant
ut presenting all of its. > + * The addition seems like a good one, but the last sentence ends weird, like a word was left off. Maybe "its changes." ? -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn 1/2 v5] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-06-12 Thread Russell Bryant
ng everything on the same host. This would be common in a test environment with a single host or small set of hosts. That would simply be: # ovs-ctl start --system-id=random # ovn-ctl start_northd # ovn-ctl start_controller Signed-off-by: Russell Bryant CC: Flavio Leitner

[ovs-dev] (no subject)

2015-06-12 Thread Russell Bryant
From Russell Bryant # This line is ignored. From: Russell Bryant Subject: [PATCH ovn 0/2 v5] Update Fedora packaging for OVN. In-Reply-To: <1432700278-10112-1-git-send-email-rbry...@redhat.com> v1->v2: - Change systemd units to run ovn-northd and ovn-controller directly instead of

[ovs-dev] [PATCH ovn 2/2 v5] fedora.spec: Create openvswitch-ovn package.

2015-06-12 Thread Russell Bryant
that is hosting the OVN databases. That seems like a reasonable assumption in the current architecture and can be evolved later when needed. Signed-off-by: Russell Bryant CC: Flavio Leitner CC: Ben Pfaff --- rhel/automake.mk | 4 +- rhel/openvswitch

[ovs-dev] [PATCH ovn 1/2 v5] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-06-12 Thread Russell Bryant
ng everything on the same host. This would be common in a test environment with a single host or small set of hosts. That would simply be: # ovs-ctl start --system-id=random # ovn-ctl start_northd # ovn-ctl start_controller Signed-off-by: Russell Bryant CC: Flavio Leitner

[ovs-dev] [PATCH ovn 0/2 v5] Update Fedora packaging for OVN.

2015-06-12 Thread Russell Bryant
|4 rhel/openvswitch-fedora.spec.in| 77 +- rhel/usr_lib_systemd_system_ovn-controller.service | 16 + rhel/usr_lib_systemd_system_ovn-northd.service | 15 + 9 files changed, 420 insertions(+), 8 deletions(-) -- Russell Bryant _

[ovs-dev] [PATCH ovn 2/2 v5] fedora.spec: Create openvswitch-ovn package.

2015-06-12 Thread Russell Bryant
that is hosting the OVN databases. That seems like a reasonable assumption in the current architecture and can be evolved later when needed. Signed-off-by: Russell Bryant CC: Flavio Leitner CC: Ben Pfaff --- rhel/automake.mk | 4 +- rhel/openvswitch

Re: [ovs-dev] first OVN scale testing thoughts

2015-06-12 Thread Russell Bryant
down 29 : down 30 : down 38 : down 41 : down 48 : down $ ovs-vsctl --db=unix:/home/rbryant/src/ovs/sandbox/hv1/db.sock show | grep vif Port "vif1" Interface "vif1" $ ovs-vsctl --db=unix:/home/rbryant/src/ovs/sandbox/hv29/db.sock show | grep vif (nothing) I haven't quite figured out what causes it yet, though. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH ovn] Update location for Neutron plugin.

2015-06-12 Thread Russell Bryant
(launchpad bugs). Signed-off-by: Russell Bryant --- ovn/TODO | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ovn/TODO b/ovn/TODO index ec05a83..fe296b4 100644 --- a/ovn/TODO +++ b/ovn/TODO @@ -175,8 +175,6 @@ This is being developed on OpenStack's development

Re: [ovs-dev] first OVN scale testing thoughts

2015-06-14 Thread Russell Bryant
On 06/13/2015 09:47 PM, Ben Pfaff wrote: > On Fri, Jun 12, 2015 at 02:29:38PM -0400, Russell Bryant wrote: >> On 06/11/2015 03:14 AM, Ben Pfaff wrote: >>> On Wed, Jun 10, 2015 at 11:53:47PM -0700, Ben Pfaff wrote: >>>> I want to report progress I've made tonight

Re: [ovs-dev] OVN and OpenStack Provider Networks

2015-06-22 Thread Russell Bryant
(Apologies for the slow follow-up to the responses on this thread. I've been on vacation.) On 06/15/2015 08:00 PM, Ben Pfaff wrote: > On Wed, Jun 10, 2015 at 03:13:54PM -0400, Russell Bryant wrote: >> Provider Networks >> = >> >> OpenStack Neutron

Re: [ovs-dev] OVN and OpenStack Provider Networks

2015-06-22 Thread Russell Bryant
for the bridges implementing the > mapping with physical networks, or do we consider them to be outside of > the OVN realm? I was proposing that info to be in the Open_vSwitch database, as that's where the other OVN configuration entries live that are local to the node. I don't think the bridge mappings are useful anywhere else. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] Add a few OVN files as part of DISTCLEANFILES.

2015-06-23 Thread Russell Bryant
> 4 files changed, 7 insertions(+) Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 0/2] ovn: Add administrative state to logical ports.

2015-06-23 Thread Russell Bryant
hanged, 91 insertions(+), 3 deletions(-) -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 2/2] ovn: Add get/set-enabled to ovn-nbctl.

2015-06-23 Thread Russell Bryant
This patch adds support for getting and setting the 'enabled' column for logical ports using ovn-nbctl. Signed-off-by: Russell Bryant --- ovn/ovn-nbctl.8.xml | 13 ovn/ovn-nbctl.c | 59 + 2 files changed, 72

[ovs-dev] [PATCH 1/2] ovn: Add logical port 'enabled' state.

2015-06-23 Thread Russell Bryant
h means no traffic is allowed in or out of the port. The implementation is fairly non-invasive as it only required minor changes to the logical pipeline. Signed-off-by: Russell Bryant --- ovn/northd/ovn-northd.c | 14 +++--- ovn/ovn-nb.ovsschema| 1 + ovn/ovn-nb.xml | 7 ++

Re: [ovs-dev] [PATCH 0/2] ovn: Add administrative state to logical ports.

2015-06-23 Thread Russell Bryant
On 06/23/2015 02:22 PM, Russell Bryant wrote: > While working on OpenStack Neutron integration for OVN, I came across a small > feature gap. The Neutron API supports setting a port as administratively > down. > One way to implement that would be to delete ports from OVN while > a

Re: [ovs-dev] OVN and OpenStack Provider Networks

2015-06-23 Thread Russell Bryant
On 06/23/2015 04:17 PM, Ben Pfaff wrote: > On Mon, Jun 22, 2015 at 02:34:07PM -0400, Russell Bryant wrote: >> On 06/15/2015 08:00 PM, Ben Pfaff wrote: >>> On Wed, Jun 10, 2015 at 03:13:54PM -0400, Russell Bryant wrote: >>>> Provider Networks >>>> ===

Re: [ovs-dev] OVN and OpenStack Provider Networks

2015-06-23 Thread Russell Bryant
On 06/23/2015 05:10 PM, Ben Pfaff wrote: > On Tue, Jun 23, 2015 at 04:54:20PM -0400, Russell Bryant wrote: >> On 06/23/2015 04:17 PM, Ben Pfaff wrote: >>> On Mon, Jun 22, 2015 at 02:34:07PM -0400, Russell Bryant wrote: >>>> On 06/15/2015 08:00 PM, Ben Pfaff wrote: &g

Re: [ovs-dev] OVN and OpenStack Provider Networks

2015-06-24 Thread Russell Bryant
tenant > network, but a Neutron plugin only *participates* in a provider network. > Is that fair? > > (I apologize if all this should be obvious to Neutron veterans. I'm new > to this!) Based on this thread, and some conversations I've seen between other Neutron devs, this is definitely *not* obvious. The recap you have here sounds right to me, though. However, as discussed above, what's expected from the Neutron backend for a transport type other than 'vlan' or 'flat' with a physical network specified isn't clear to me. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 0/2] ovn: Add administrative state to logical ports.

2015-06-25 Thread Russell Bryant
On 06/24/2015 08:30 PM, Ben Pfaff wrote: > On Tue, Jun 23, 2015 at 03:18:57PM -0400, Russell Bryant wrote: >> On 06/23/2015 02:22 PM, Russell Bryant wrote: >>> While working on OpenStack Neutron integration for OVN, I came across a >>> small >>> feature gap.

Re: [ovs-dev] OVN - Pluggable Distributed DB Infrastructure for OVSDB

2015-06-25 Thread Russell Bryant
ributed db, I suspect there would be a difficult perception and trust battle. "Ugh, they implemented their own distributed db?! NIH, etc .." "Ugh, a distributed db that nobody else uses? I don't trust that." Those are the reactions I'm already hearing talking to people about this stuff. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] OVN - Pluggable Distributed DB Infrastructure for OVSDB

2015-06-30 Thread Russell Bryant
ight be better time spent to think about OVN testing and how we can do good, reproducible scale tests. Ben has done some awesome work here, but it seems we may need to a multi-node environment since at some point it sounds like the number of processes on a s

Re: [ovs-dev] Allowed Address Pairs - OVN

2015-07-01 Thread Russell Bryant
because security wise this means that a certain IP > must be assigned to a certain MAC address (please correct me if i am > wrong here) > > Just wanted to put this here, so when L3 design is finalized these > connections > are also taken care of in OVN for port

Re: [ovs-dev] [PATCH] ovn: Take advantage of OVSDB garbage collection in OVN_Northbound schema.

2015-07-01 Thread Russell Bryant
ixes the race we were seeing before. Awesome, thanks everyone! The only thing I noticed in the OVN patch was that tabs were used as indentation in ovn-nb.ovsschema where it was all spaces before. Otherwise: Acked-by: Russell Bryant -- Russell Bryant ___

Re: [ovs-dev] [PATCH v2]: ovsdb: add capability to dump table content in ovsdb-client

2015-07-02 Thread Russell Bryant
logical flows (sort of similar to OpenFlow but for logical networks). I look at the contents of this table a lot. It would be nice to be able to tell it to sort by table_id, and even a secondary sort on priority. This may be overkill for ovsdb-client and we should just have an OVN specific utility for what I want, but I figured I'd share the idea out loud at least. :-) -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2]: ovsdb: add capability to dump table content in ovsdb-client

2015-07-02 Thread Russell Bryant
On 07/02/2015 09:50 AM, Russell Bryant wrote: > On 06/30/2015 11:17 PM, Andy Zhou wrote: >> On Tue, Jun 30, 2015 at 7:47 PM, Sabyasachi Sengupta >> wrote: >>> >>>> It seems we just need to extract the text between the table name and the >>>> first &

Re: [ovs-dev] [PATCH] ovn: Take advantage of OVSDB garbage collection in OVN_Northbound schema.

2015-07-02 Thread Russell Bryant
On 07/01/2015 03:46 PM, Russell Bryant wrote: > On 07/01/2015 03:40 PM, Aaron Rosen wrote: >> Fwiw the neutron side is ready to merge as well: >> >> https://review.openstack.org/#/c/196132/ >> >> we also were able to test your patch in the gate >> https:/

Re: [ovs-dev] [PATCH] ovn: Take advantage of OVSDB garbage collection in OVN_Northbound schema.

2015-07-02 Thread Russell Bryant
On 07/02/2015 01:47 PM, Ben Pfaff wrote: > On Thu, Jul 02, 2015 at 11:48:11AM -0400, Russell Bryant wrote: >> On 07/01/2015 03:46 PM, Russell Bryant wrote: >>> On 07/01/2015 03:40 PM, Aaron Rosen wrote: >>>> Fwiw the neutron side is ready to merge as well: >>

[ovs-dev] [RFC] ovn: physical network integraiton (WIP)

2015-07-06 Thread Russell Bryant
CC: Ben Pfaff --- ovn/northd/ovn-northd.c | 42 -- ovn/ovn-nb.ovsschema| 6 ++ ovn/ovn-nb.xml | 18 ++ ovn/ovn-nbctl.c | 29 + ovn/ovn-sb.xml | 2 ++ 5 files changed, 91 insert

Re: [ovs-dev] [ovn-controller-vtep V2 2/6] ovn-sbctl: Add ovn-sbctl.

2015-07-06 Thread Russell Bryant
} > +break; > + > +VLOG_OPTION_HANDLERS > +TABLE_OPTION_HANDLERS(&table_style) > + > +case '?': > +exit(EXIT_FAILURE); > + > +default: > +abort(); > +} > +} > + free(s

Re: [ovs-dev] [PATCH] Add 'name' field to Logical Router

2015-07-08 Thread Russell Bryant
s reasonable to me. It seems odd to have two s-o-b headers, though. Maybe pick one? Acked-by: Russell Bryant > --- > ovn/ovn-nb.ovsschema | 1 + > ovn/ovn-nb.xml | 9 + > 2 files changed, 10 insertions(+) > > diff --git a/ovn/ovn-nb.ovsschema b/ovn/ovn-nb.ovssc

Re: [ovs-dev] [RFC] ovn: physical network integraiton (WIP)

2015-07-08 Thread Russell Bryant
is is used for a given logical switch, no tunnels should be created and we need the different output behavior. So, I'm not sure if modeling it that way makes things easier. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [RFC] ovn: physical network integraiton (WIP)

2015-07-09 Thread Russell Bryant
(re-adding ovs dev list) On 07/09/2015 10:52 AM, Salvatore Orlando wrote: > > > On 8 July 2015 at 16:12, Russell Bryant <mailto:rbry...@redhat.com>> wrote: > > On 07/07/2015 06:06 PM, Salvatore Orlando wrote: > > Reading your post and the associated patch

<    1   2   3   4   5   6   7   8   9   10   >