Processed: Re: Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-03-03 Thread Debian Bug Tracking System
Processing control commands:

> tags -1 + pending
Bug #884711 [release.debian.org] stretch-pu: package dpdk/16.11.4-1+deb9u1
Added tag(s) pending.

-- 
884711: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884711
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-03-03 Thread Adam D. Barratt
Control: tags -1 + pending

On Sun, 2018-02-11 at 19:26 +, Luca Boccassi wrote:
> On Sat, 2018-02-10 at 09:47 +0100, Julien Cristau wrote:
> > Control: tags -1 confirmed
> > 
> > On Sun, Jan 14, 2018 at 20:14:14 +, Luca Boccassi wrote:
> > 
> > > Thank you for the review, I have reworked the debdiff - rather
> > > than
> > > taking the last 16.11.x version we uploaded in Sid and reverting
> > > a
> > > few
> > > changes, instead I've taken the version in Stretch and merged the
> > > LTS
> > > point releases and a couple of small changes, details below.
> > > 
> > 
> > Great, thanks.  Go ahead and upload to stretch.
> > 
> > Cheers,
> > Julien
> 
> Done, thanks.

Flagged for acceptance.

Regards,

Adam



Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-02-11 Thread Luca Boccassi
On Sat, 2018-02-10 at 09:47 +0100, Julien Cristau wrote:
> Control: tags -1 confirmed
> 
> On Sun, Jan 14, 2018 at 20:14:14 +, Luca Boccassi wrote:
> 
> > Thank you for the review, I have reworked the debdiff - rather than
> > taking the last 16.11.x version we uploaded in Sid and reverting a
> > few
> > changes, instead I've taken the version in Stretch and merged the
> > LTS
> > point releases and a couple of small changes, details below.
> > 
> 
> Great, thanks.  Go ahead and upload to stretch.
> 
> Cheers,
> Julien

Done, thanks.

There will be one LTS bugfix release every 3 months until at least
November. I think there is no whitelisting system like Ubuntu has, and
so I will need to create a release.debian.org bug each time, right?

-- 
Kind regards,
Luca Boccassi

signature.asc
Description: This is a digitally signed message part


Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-02-10 Thread Julien Cristau
Control: tags -1 confirmed

On Sun, Jan 14, 2018 at 20:14:14 +, Luca Boccassi wrote:

> Thank you for the review, I have reworked the debdiff - rather than
> taking the last 16.11.x version we uploaded in Sid and reverting a few
> changes, instead I've taken the version in Stretch and merged the LTS
> point releases and a couple of small changes, details below.
> 
Great, thanks.  Go ahead and upload to stretch.

Cheers,
Julien



Processed: Re: Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-02-10 Thread Debian Bug Tracking System
Processing control commands:

> tags -1 confirmed
Bug #884711 [release.debian.org] stretch-pu: package dpdk/16.11.4-1+deb9u1
Added tag(s) confirmed.

-- 
884711: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884711
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Processed: Re: Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-01-14 Thread Debian Bug Tracking System
Processing control commands:

> tags -1 -moreinfo
Bug #884711 [release.debian.org] stretch-pu: package dpdk/16.11.4-1+deb9u1
Removed tag(s) moreinfo.

-- 
884711: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884711
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-01-13 Thread Julien Cristau
Control: tag -1 moreinfo

Hi Luca,

On Mon, Dec 18, 2017 at 14:46:04 +, Luca Boccassi wrote:

> DPDK maintains LTS branches upstream for 2 years [1]. This is covered
> by regressions tests from Intel and AT on various networking hardware
> and virtualized environments.These tests are run to validate the
> release branch before tagging a new version. Other hardware vendors
> regularly test these LTS versions too. They are also used by other
> distros such as RedHat and Ubuntu.
> Only bug fixes that affect that release branch, and that have already
> been merged in the mainline, are allowed. No API/ABI changes of any
> kind are allowed. Full changelog can be found at the bottom of the
> release notes [2].
> Each major mainline release is followed by an LTS branch release, where
> the relevant bug fixes are backported, so there are usually 4 bugfix
> LTS releases per year. There are usually between 10 and 100 patches
> backported per release.
> 
> Stretch shipped with the first release of 16.11. Since then, there have
> been 4 bugfix LTS releases, up to 16.11.4 a few days ago.
> We have uploaded them to unstable/testing/stretch-backports.
> 
> We would like to take advantage of the upstream LTS maintenance and
> push these point releases to Stretch.
> 
> Attached is the full debdiff. I can provide a debdiff for each
> individual point release between 16.11 and 16.11.4 if required.
> There is only one change in dependency, python-elftools was demoted
> from Recommends to Suggests as it's not useful in most cases. I think
> this change is fine but can remove it if asked.
> 
> Patches to fix bugs reported on Ubuntu, and to make the build
> reproducible, are also included. I can remove them if asked.
> 
> The changelog is inlined at the bottom.
> 
> Although the main users are Debian users with custom software, there is
> a reverse build-dep in Stretch as well, collectd. It works fine with
> the new LTS version.
> 
Thanks for the detailed explanation, that helps.

A few comments:
- I'd rather do without the changes in debian/control
- likewise init script, pkg-config file (unless there's an explanation)
- the above doesn't seem to address the changes in debian/patches/, can
  you detail the reason behind each of those?
- I think from our perspective the reproducibility changes are adding
  more risk than anything else, and normally not appropriate for stable
- the change in prep-modules, generating dependencies on the exact version
  of linux-image rather than the ABI (probably with ">=" the build
  version), looks wrong.  I don't know if/how that is used by the package
  though.
- the debian/rules changes introduce noise, so if that can be trimmed
  down it would be welcome
- I can't speak to the upstream changes, but no objection in principle
  if that stuff is tested as you describe.  Is any of the LTS branch
  testing done in a Debian stable environment, either by upstream, you,
  or some other users of this code?

Cheers,
Julien



Processed: Re: Bug#884711: stretch-pu: package dpdk/16.11.4-1+deb9u1

2018-01-13 Thread Debian Bug Tracking System
Processing control commands:

> tag -1 moreinfo
Bug #884711 [release.debian.org] stretch-pu: package dpdk/16.11.4-1+deb9u1
Added tag(s) moreinfo.

-- 
884711: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884711
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems