[ovs-dev] Set release date for 2.8.0.

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
This will be applied to both master and branch-2.8, to effectively release
Open vSwitch 2.8.0.

On Thu, 31 Aug 2017 09:33:25 -0700
Ben Pfaff <blp@ovn.org> wrote:

> Signed-off-by: Ben Pfaff <blp@ovn.org>
> ---
> This will be applied to both master and branch-2.8, to effectively release
> Open vSwitch 2.8.0.

Hi Ben,

I don't want to delay 2.8 even more but there was a conntrack change[1]
which breaks OpenShift.  I think Russell is discussing with Justin at
the moment what to do.

I don't know the urgency of this change from Justin's side, so my
suggestion would be to push it to 2.9 and in 2.8 add a warning
about the upcoming change.

[1] ofproto-dpif: Mark packets as "untracked" after  call to ct().

fbl


> 
> diff --git a/NEWS b/NEWS
> index 8e6fb79d38ca..b85c351dd961 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -1,4 +1,4 @@
> -v2.8.0 - xx xxx xxxx
> +v2.8.0 - 31 Aug 2017
>  ---------------------
>     - ovs-ofctl:
>       * ovs-ofctl can now accept and display port names in place of numbers.  By
> diff --git a/debian/changelog b/debian/changelog
> index 08ba2223a4c3..95a875fbfee3 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -4,9 +4,8 @@ openvswitch (2.8.0-1) unstable; urgency=medium
>     * Use debhelper for DEB_VERSION_UPSTREAM.  Thanks to Clint Byrum
>       <spamaps@debian.org> for this improvement.
>     * New upstream version
> -   - Nothing yet!  Try NEWS...
>  
> - -- Open vSwitch team <dev@openvswitch.org>  Fri, 04 Aug 2017 14:55:14 -0700
> + -- Open vSwitch team <dev@openvswitch.org>  Thu, 31 Aug 2017 09:32:16 -0700
>  
>  openvswitch (2.7.0-1) unstable; urgency=low
>     [ Open vSwitch team ]

On Thu, Aug 31, 2017 at 01:59:45PM -0300, Flavio Leitner wrote:
> On Thu, 31 Aug 2017 09:33:25 -0700
> Ben Pfaff <blp@ovn.org> wrote:
> 
> > Signed-off-by: Ben Pfaff <blp@ovn.org>
> > ---
> > This will be applied to both master and branch-2.8, to effectively release
> > Open vSwitch 2.8.0.
> 
> Hi Ben,
> 
> I don't want to delay 2.8 even more but there was a conntrack change[1]
> which breaks OpenShift.  I think Russell is discussing with Justin at
> the moment what to do.
> 
> I don't know the urgency of this change from Justin's side, so my
> suggestion would be to push it to 2.9 and in 2.8 add a warning
> about the upcoming change.
> 
> [1] ofproto-dpif: Mark packets as "untracked" after  call to ct().

OK, let's figure this out first.

> On Aug 31, 2017, at 9:33 AM, Ben Pfaff <blp@ovn.org> wrote:
> 
> Signed-off-by: Ben Pfaff <blp@ovn.org>
> ---
> This will be applied to both master and branch-2.8, to effectively release
> Open vSwitch 2.8.0.
> 
> diff --git a/NEWS b/NEWS
> index 8e6fb79d38ca..b85c351dd961 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -1,4 +1,4 @@
> -v2.8.0 - xx xxx xxxx
> +v2.8.0 - 31 Aug 2017
> ---------------------
>    - ovs-ofctl:
>      * ovs-ofctl can now accept and display port names in place of numbers.  By
> diff --git a/debian/changelog b/debian/changelog
> index 08ba2223a4c3..95a875fbfee3 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -4,9 +4,8 @@ openvswitch (2.8.0-1) unstable; urgency=medium
>    * Use debhelper for DEB_VERSION_UPSTREAM.  Thanks to Clint Byrum
>      <spamaps@debian.org> for this improvement.
>    * New upstream version
> -   - Nothing yet!  Try NEWS...

I normally copy the NEWS items into here.  Do you think that's unnecessary?

Acked-by: Justin Pettit <jpettit@ovn.org>

--Justin

On Thu, Aug 31, 2017 at 01:08:31PM -0700, Justin Pettit wrote:
> 
> > On Aug 31, 2017, at 9:33 AM, Ben Pfaff <blp@ovn.org> wrote:
> > 
> > Signed-off-by: Ben Pfaff <blp@ovn.org>
> > ---
> > This will be applied to both master and branch-2.8, to effectively release
> > Open vSwitch 2.8.0.
> > 
> > diff --git a/NEWS b/NEWS
> > index 8e6fb79d38ca..b85c351dd961 100644
> > --- a/NEWS
> > +++ b/NEWS
> > @@ -1,4 +1,4 @@
> > -v2.8.0 - xx xxx xxxx
> > +v2.8.0 - 31 Aug 2017
> > ---------------------
> >    - ovs-ofctl:
> >      * ovs-ofctl can now accept and display port names in place of numbers.  By
> > diff --git a/debian/changelog b/debian/changelog
> > index 08ba2223a4c3..95a875fbfee3 100644
> > --- a/debian/changelog
> > +++ b/debian/changelog
> > @@ -4,9 +4,8 @@ openvswitch (2.8.0-1) unstable; urgency=medium
> >    * Use debhelper for DEB_VERSION_UPSTREAM.  Thanks to Clint Byrum
> >      <spamaps@debian.org> for this improvement.
> >    * New upstream version
> > -   - Nothing yet!  Try NEWS...
> 
> I normally copy the NEWS items into here.  Do you think that's unnecessary?

There's nothing wrong with it, but it's pretty unusual in a Debian
changelog.  Usually they just list Debian-specific changes, not upstream
changes.

> Acked-by: Justin Pettit <jpettit@ovn.org>

Since you are getting that last revert in for Flavio, and you usually do
the releases, do you want to take over this patch, then?

> On Aug 31, 2017, at 1:14 PM, Ben Pfaff <blp@ovn.org> wrote:
> 
> On Thu, Aug 31, 2017 at 01:08:31PM -0700, Justin Pettit wrote:
>> 
>>> On Aug 31, 2017, at 9:33 AM, Ben Pfaff <blp@ovn.org> wrote:
>>> 
>>> Signed-off-by: Ben Pfaff <blp@ovn.org>
>>> ---
>>> This will be applied to both master and branch-2.8, to effectively release
>>> Open vSwitch 2.8.0.
>>> 
>>> diff --git a/NEWS b/NEWS
>>> index 8e6fb79d38ca..b85c351dd961 100644
>>> --- a/NEWS
>>> +++ b/NEWS
>>> @@ -1,4 +1,4 @@
>>> -v2.8.0 - xx xxx xxxx
>>> +v2.8.0 - 31 Aug 2017
>>> ---------------------
>>>   - ovs-ofctl:
>>>     * ovs-ofctl can now accept and display port names in place of numbers.  By
>>> diff --git a/debian/changelog b/debian/changelog
>>> index 08ba2223a4c3..95a875fbfee3 100644
>>> --- a/debian/changelog
>>> +++ b/debian/changelog
>>> @@ -4,9 +4,8 @@ openvswitch (2.8.0-1) unstable; urgency=medium
>>>   * Use debhelper for DEB_VERSION_UPSTREAM.  Thanks to Clint Byrum
>>>     <spamaps@debian.org> for this improvement.
>>>   * New upstream version
>>> -   - Nothing yet!  Try NEWS...
>> 
>> I normally copy the NEWS items into here.  Do you think that's unnecessary?
> 
> There's nothing wrong with it, but it's pretty unusual in a Debian
> changelog.  Usually they just list Debian-specific changes, not upstream
> changes.

Great!  Less work for me going forward.

>> Acked-by: Justin Pettit <jpettit@ovn.org>
> 
> Since you are getting that last revert in for Flavio, and you usually do
> the releases, do you want to take over this patch, then?

Sure.  I'm happy to do that.

Thanks,

--Justin

> On Aug 31, 2017, at 1:18 PM, Justin Pettit <jpettit@ovn.org> wrote:
> 
>> 
>> On Aug 31, 2017, at 1:14 PM, Ben Pfaff <blp@ovn.org> wrote:
>> 
>> Since you are getting that last revert in for Flavio, and you usually do
>> the releases, do you want to take over this patch, then?
> 
> Sure.  I'm happy to do that.

I pushed this to branch-2.8.

--Justin

On Fri, 1 Sep 2017 11:47:19 -0700
Justin Pettit <jpettit@ovn.org> wrote:

> > On Aug 31, 2017, at 1:18 PM, Justin Pettit <jpettit@ovn.org> wrote:
> >   
> >> 
> >> On Aug 31, 2017, at 1:14 PM, Ben Pfaff <blp@ovn.org> wrote:
> >> 
> >> Since you are getting that last revert in for Flavio, and you usually do
> >> the releases, do you want to take over this patch, then?  
> > 
> > Sure.  I'm happy to do that.  
> 
> I pushed this to branch-2.8.

We ran tests with branch-2.8 from a couple weeks ago and found
only the ct() behavior change as a blocker issue. Since the
patch was reverted, the issue is resolved.

We found SELinux issues which are important to RHEL/Fedora distros,
which Aaron posted fixes already and we can continue following up
if needed.

Thanks,