[Bug 1888352] Re: use builtin dump_acpi_tables.py in hookutils

[Timo Aaltonen]

Hello Yuan-Chen, or anyone else affected,

Accepted apport into focal-proposed. The package will build now and be
available at
https://launchpad.net/ubuntu/+source/apport/2.20.11-0ubuntu27.9 in a few
hours, and then in the -proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed.  Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested, what testing has been
performed on the package and change the tag from verification-needed-
focal to verification-done-focal. If it does not fix the bug for you,
please add a comment stating that, and change the tag to verification-
failed-focal. In either case, without details of your testing we will
not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

** Changed in: apport (Ubuntu Focal)
       Status: In Progress => Fix Committed

** Tags added: verification-needed verification-needed-focal

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to apport in Ubuntu.
https://bugs.launchpad.net/bugs/1888352

Title:
  use builtin dump_acpi_tables.py in hookutils

Status in Apport:
  New
Status in OEM Priority Project:
  Confirmed
Status in apport package in Ubuntu:
  Fix Released
Status in apport source package in Focal:
  Fix Committed
Status in apport source package in Groovy:
  Fix Released

Bug description:
  1. add apcidump to hookutils.py:attach_hardware and use the builtin dump_acpi_tables.py.
  2. remove explicit acpidump from oem-getlogs to use the builtin one.
  3. refine usage string in oem-getlogs.

  To SRU to focal:

  [Impact]

   * for OEM project, lts is been used. And collect log is important.
     With built-in tool to get acpidump, we don't need to install
     extra tool in the end-customer's machine. That make it much
     easier for the oem process.

   * By call built-in utility, we no lounger need to install extra
     tools to collect data that's both complete and convenient for
     HWE people to work on.

  [Test Case]

   * before this applied, run oem-getlog without install acpidump. 
     we can't get the dump data. After this applied, we can just dump
     the data HWE need.

   * test step: install the new package, run

     sudo -E oem-getlogs [-c case_id] to get the

     use apport-unpack to unpack the apport file.

     check the acpidump file. HWE people know much better on check
     the acpidump file. Give the dump_acpi_tables.py just updated and SRUed
     by HWE/ACPI/UEFI expert, it's pretty safe to do so.

  [Regression Potential]

   * Given the modification change the way to collect log, even it
     failed, it won't break apport itself. Just the collected log
     might contain data not so valid.

   * Given acpidump mostly used by HWE/ACIP/UEFI expert, and they
     just reviewed and updated the dump_acpi_tables.py script, I
     believe it will have good quality.

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