On Tue, Mar 3, 2015 at 10:07 PM, Ben Pfaff <b...@nicira.com> wrote: > On Sat, Feb 28, 2015 at 10:24:19AM -0800, Alex Wang wrote: > > Commit 9ba4f3c5(XenServer: PEP8 Cleanup for openvswitch-cfg-update). > > introduced a bug which causes the setting of 'ssl::6632' at xenserver > > boot. > > It appears to me that it was the following commit 032c09ddd10 > (XenServer: Don't reset on xe-toolstack-restart) that introduced the > bug. I think the one that you cite only made cosmetic, not behavioral, > changes. >
Yes, you are right. I'll update the commit log. > > Subsequently, at each bridge configuration, ovs will report > > '|socket_util|ERR|:6632: bad IPv4 address ""' due to the empty > > address. > > > > The exact calltrace is: > > > > 0 parse_sockaddr_components (ss=0xbf8dd87c, host_s=0x94c99f0 "", > > port_s=0x94c99f1 "6632", default_port=6632, s=0x948b9dc ":6632") > > at lib/socket-util.c:350 > > 1 0x081163b5 in inet_parse_active (target_=0x948b9dc ":6632", > default_port= > > 6632, ss=0xbf8dd87c) at lib/socket-util.c:412 > > 2 0x08119dcf in stream_parse_target_with_default_port (target=0x948b9d8 > > "ssl::6632", default_port=6632, ss=0xbf8dd87c) at lib/stream.c:734 > > 3 0x0804c946 in collect_in_band_managers (ovs_cfg=0x94433b0, > > managersp=0xbf8dd9a8, n_managersp=0xbf8dd9a0) at > vswitchd/bridge.c:516 > > 4 0x08054b87 in bridge_reconfigure (ovs_cfg=0x94433b0) at > vswitchd/bridge.c: > > 614 > > 5 0x080557cf in bridge_run () at vswitchd/bridge.c:2897 > > 6 0x080564aa in main (argc=155674720, argv=0x0) at > vswitchd/ovs-vswitchd.c: > > 116 > > > > This commit fixes the issue. > > > > Signed-off-by: Alex Wang <al...@nicira.com> > > I think I agree with your diagnosis but I'd like to hear from Jason if > he's around. Jason? > I'll wait for his reply, Thanks, Alex Wang, _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
