This got trumped by security, so I rebased it and bumped the version for
you - attached. I then reviewed it, but given the clarified SRU policy
on test changes, does this still qualify?

>From https://wiki.ubuntu.com/StableReleaseUpdates#When_not_to_upload:

"Do not upload SRUs for bugs which do not affect users at runtime."

Does this affect users at runtime, and if not, is there some other
reason that we want this uploaded now, rather than bundling it with some
other useful changes? I note that the previous Bionic SRU for the same
issue did have other bundled changes.

** Patch added: "apport-xenial.debdiff"
   
https://bugs.launchpad.net/ubuntu/+source/apport/+bug/1766740/+attachment/5276165/+files/apport-xenial.debdiff

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to apport in Ubuntu.
https://bugs.launchpad.net/bugs/1766740

Title:
  apport autopkgtest regression due to kernel packaging changes

Status in apport package in Ubuntu:
  Fix Released
Status in apport source package in Xenial:
  Triaged
Status in apport source package in Bionic:
  Fix Released

Bug description:
  [SRU Justification]
  In bionic, linux-image-$kvers-$flavor is now built from the linux-signed 
source package on amd64, not from the linux source package.  This now causes a 
test to fail in the apport test suite:

  [...]
  ======================================================================
  FAIL: test_run_crash_kernel (__main__.T)
  run_crash() for a kernel error
  ----------------------------------------------------------------------
  Traceback (most recent call last):
    File "./test_ui.py", line 1305, in test_run_crash_kernel
      self.assertEqual(self.ui.opened_url, 'http://linux.bugs.example.com/%i' % 
se
  lf.ui.crashdb.latest_id())
  AssertionError: 'http://linux-signed.bugs.example.com/5' != 
'http://linux.bugs.e
  xample.com/5'
  - http://linux-signed.bugs.example.com/5
  ?             -------
  + http://linux.bugs.example.com/5

  ----------------------------------------------------------------------
  Ran 70 tests in 79.815s

  FAILED (failures=1)
  [...]

  
(https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac
  /autopkgtest-
  bionic/bionic/amd64/a/apport/20180424_222219_45dc8@/log.gz)

  This test needs to be fixed to know about the packaging change.

  Note that this packaging change is expected to soon be SRUed back to
  all Ubuntu releases, so apport's testsuite change should also be
  SRUed.

  [Test case]
  This passes if the autopkgtests pass.

  [Regression potential]
  The test could be changed to pass while leaving underlying issues in the 
code.  This has probably not happened here.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/apport/+bug/1766740/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to     : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to