Hello Stefan, or anyone else affected,

Accepted ubuntu-fan into yakkety-proposed. The package will build now
and be available at https://launchpad.net/ubuntu/+source/ubuntu-
fan/0.12.0.2 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, and change the tag
from verification-needed to verification-done. If it does not fix the
bug for you, please add a comment stating that, and change the tag to
verification-failed.  In either case, details of your testing will help
us make a better decision.

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

** Tags removed: verification-done

** Tags added: verification-needed

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to ubuntu-fan in Ubuntu.
https://bugs.launchpad.net/bugs/1691485

Title:
  fanatic disable-fan fails without docker installed

Status in ubuntu-fan package in Ubuntu:
  Fix Released
Status in ubuntu-fan source package in Xenial:
  In Progress
Status in ubuntu-fan source package in Yakkety:
  Fix Committed
Status in ubuntu-fan source package in Zesty:
  Fix Released

Bug description:
  Release: 17.04/Zesty
  Version: 0.12.2

  When disabling fan, the code tries to check whether docker is
  configured. However that test is attempting to decide which docker
  version is installed (in order to handle the new network options).
  This fails but is incorrectly evaluated as docker being configured.
  And that prevents fan from being de-configured.

  Fix: Test for docker being installed and if not return false to the
  configured check.

  SRU Justification:

  Impact: The function which does docker config updates/tests currently
  does not check for the presence of the docker command itself. And
  while there are error messages to stderr a call to check for fan being
  configured for docker accidentally results in returning true.

  Fix: Adding a check for the docker command being present early in the
  function and return an error in that case (which is a false in the is-
  docker-configured case). Since the function should really always fail
  if there is no docker present the risk of regressions should be low.

  Testcase:
   - [docker is not installed]
   - fanatic enable-fan ...
   - fanatic disable-fan ...

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ubuntu-fan/+bug/1691485/+subscriptions

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

Reply via email to