Message ID | OFD92F4AB7.B62DA390-ON48258153.0037B8DE-48258153.0038370D@zte.com.cn |
---|---|
State | Accepted |
Headers | show |
On Tue, Jul 04, 2017 at 06:14:09PM +0800, wang.qianyu@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 <wang.qianyu@zte.com.cn> Thank you for the bug fix. Would you mind using a sign-off that uses your name instead of a random number? Thanks, Ben.
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(&pb, OVSDB_F_EQ, name); + sbrec_port_binding_add_clause_parent_port(&pb, OVSDB_F_EQ, name); }
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 <wang.qianyu@zte.com.cn> --- ovn/controller/ovn-controller.c | 1 + 1 file changed, 1 insertion(+) } if (local_datapaths) {