[Bug 1983451] Re: [fwupd][SRU] Hide inhibited usb4 device in dell-dock plugin

[Lukas Märdian]

** Also affects: fwupd (Ubuntu Focal)
   Importance: Undecided
       Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1983451

Title:
  [fwupd][SRU] Hide inhibited usb4 device in dell-dock plugin

Status in OEM Priority Project:
  Confirmed
Status in OEM Priority Project focal series:
  Confirmed
Status in fwupd package in Ubuntu:
  Fix Released
Status in fwupd source package in Focal:
  New
Status in fwupd source package in Jammy:
  Confirmed
Status in fwupd source package in Kinetic:
  Fix Released

Bug description:
  In fwupd mechanism, a device might be probed by multiple plugins, so
  an "inhibited flag" might be applied to prevent it been probed several
  times.

  An "Update Error" message will show up once this flag has been applied
  [0], and this will confuse some users to let they think the firmware
  upgrade process failed , in the purpose of improving the user
  experience, a patch has been merged to upstream to hide the inhibited
  usb4 device[1].

  [Impact]
   * A new flag "hidden" has been added in the inhibited code snippet, once the device enumeration found this flag, it will not show it.
   * Since this misleading message has be hidden, the users won't be confused on the inhibited message.

  [Test Plan]
   * Prepare a dell-dock wd22tb4.

   * Prepare one pc or laptop with thunderbolt interface on it.

   * Before(include) fwupd upstream version 1.8.2 or Ubuntu version
  1.7.5-3, a "waning message"[0] will be observed on the inhibited
  device via the command "fwupdmgr get-devices".

   * Once this patch has been applied, the inhibited device will be
  hidden.

  [Where problems could occur]
   * The only problem might occur is once someone misuses this flag "hidden" when they use this device_inhibit function, then the devices will be hidden.
   * But after double confirm, I __do not__ see someone else has applied this flag on their plugin.

  [0] https://github.com/fwupd/fwupd/tree/main/plugins/dell-dock#update-behavior
  [1] https://github.com/fwupd/fwupd/commit/4bf53196fc1198a0688b49bcd16cd0126823f434

  For other risk and test plan, please check lp:1982103

To manage notifications about this bug go to:
https://bugs.launchpad.net/oem-priority/+bug/1983451/+subscriptions