Well, that's annoying. I pushed a fix.
On Wed, Nov 08, 2017 at 05:30:15PM +0800, 通天晓0280 wrote: > It seems this patch broke OVS compiling，there is an recent error while > compiling OVS in networking-ovn CI. [1] > 0xC2A0C2A0 appeared between words "Optional." and "The" [2]. > [1]http://logs.openstack.org/03/518203/3/check/networking-ovn-tempest-dsvm-ovs-master/d021028/logs/devstacklog.txt.gz#_2017-11-08_07_08_57_141[2]https://github.com/openvswitch/ovs/blob/master/vswitchd/vswitch.xml#L2370------------------------------------------------------------------From:Ben > Pfaff <[email protected]>Time:2017 Nov 8 (Wed) 12:23To:duanjiong > <[email protected]>Cc:dev <[email protected]>Subject:Re: [ovs-dev] > [PATCH] vswitchd: doc the option dst_port > On Mon, Nov 06, 2017 at 01:43:23PM +0800, duanjiong wrote: > > Otherwise the user does not know how to change it > > > > Signed-off-by: Duan Jiong <[email protected]> > > --- > > vswitchd/vswitch.xml | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml > > index d7f6839..c4e3d62 100644 > > --- a/vswitchd/vswitch.xml > > +++ b/vswitchd/vswitch.xml > > @@ -2367,6 +2367,10 @@ > > </ul> > > </column> > > > > + <column name="options" key="dst_port"> > > + Optional. The tunnel transport layer destination port. > > + </column> > > + > > Thanks for the patch. It did not apply properly and it took me a long > time to figure out why. It is because it is corrupted by using > nonbreaking spaces (U+00A0) where spaces (U+0020) must be used. I fixed > that and applied this to master, first adding a little more information: > > --8<--------------------------cut here-------------------------->8-- > > From: duanjiong <[email protected]> > Date: Mon, 6 Nov 2017 13:43:23 +0800 > Subject: [PATCH] vswitchd: Document option dst_port. > > Otherwise the user does not know how to change it > > Signed-off-by: Duan Jiong <[email protected]> > Signed-off-by: Ben Pfaff <[email protected]> > --- > vswitchd/vswitch.xml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml > index d7f68393b25e..20e516cfe8d3 100644 > --- a/vswitchd/vswitch.xml > +++ b/vswitchd/vswitch.xml > @@ -2367,6 +2367,11 @@ > </ul> > </column> > > + <column name="options" key="dst_port"> > + Optional. The tunnel transport layer destination port, for UDP and > TCP > + based tunnel protocols (Geneve, VXLAN, LISP, and STT). > + </column> > + > <column name="options" key="key"> > Optional. Shorthand to set <code>in_key</code> and > <code>out_key</code> at the same time. > -- > 2.10.2 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
