** Changed in: ubuntu-fan (Ubuntu Zesty)
   Importance: Undecided => Medium

** Changed in: ubuntu-fan (Ubuntu Zesty)
       Status: New => Fix Committed

-- 
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 Committed
Status in ubuntu-fan source package in Zesty:
  Fix Committed

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