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 ... -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1691485 Title: fanatic disable-fan fails without docker installed To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/ubuntu-fan/+bug/1691485/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
