Re: [ovs-dev] [PATCH] ovn-controller: fix vlan_trunk bug

2017-07-13 Thread Ben Pfaff 
On Tue, Jul 04, 2017 at 06:14:09PM +0800, wang.qia...@zte.com.cn wrote:
> The bug description is as follow:
> 
> Neutron configure a trunk-sub port. The parent-port and sub-port located 
> in different network. there is a vm attached to parent port. And no vm 
> attached to the network of sub-port in the same chassis. In this 
> situation, the ovn-controller can not get the configuration info of 
> sub-port.
> 
> The reason is that ovn-controller does not monitor the port-binding with 
> parent.
> 
> This patch fix this bug.
> 
> Change-Id: I0ce3f5b1b5a9e81608ecc587a796998520a8da47
> Signed-off-by: 10110202 

Thank you for the bug fix.

Would you mind using a sign-off that uses your name instead of a random
number?

Thanks,

Ben.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH] ovn-controller: fix vlan_trunk bug

2017-07-04 Thread wang . qianyu 
The bug description is as follow:

Neutron configure a trunk-sub port. The parent-port and sub-port located 
in different network. there is a vm attached to parent port. And no vm 
attached to the network of sub-port in the same chassis. In this 
situation, the ovn-controller can not get the configuration info of 
sub-port.

The reason is that ovn-controller does not monitor the port-binding with 
parent.

This patch fix this bug.

Change-Id: I0ce3f5b1b5a9e81608ecc587a796998520a8da47
Signed-off-by: 10110202 
---
 ovn/controller/ovn-controller.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ovn/controller/ovn-controller.c 
b/ovn/controller/ovn-controller.c
index 00b4cda..4406f00 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -174,6 +174,7 @@ update_sb_monitors(struct ovsdb_idl *ovnsb_idl,
 const char *name;
 SSET_FOR_EACH (name, local_ifaces) {
 sbrec_port_binding_add_clause_logical_port(, OVSDB_F_EQ, 
name);
+sbrec_port_binding_add_clause_parent_port(, OVSDB_F_EQ, 
name);
 }
 }
 if (local_datapaths) {
-- 
1.8.3.1
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev