← Back to team overview

group.of.nepali.translators team mailing list archive

[Bug 1693574] Re: Docker plugin uses the wrong command for Ubuntu

 

** Changed in: sosreport
       Status: New => Fix Released

-- 
You received this bug notification because you are a member of नेपाली
भाषा समायोजकहरुको समूह, which is subscribed to Xenial.
Matching subscriptions: Ubuntu 16.04 Bugs
https://bugs.launchpad.net/bugs/1693574

Title:
  Docker plugin uses the wrong command for Ubuntu

Status in sosreport:
  Fix Released
Status in sosreport package in Ubuntu:
  Confirmed
Status in sosreport source package in Trusty:
  Confirmed
Status in sosreport source package in Xenial:
  Confirmed
Status in sosreport source package in Yakkety:
  Confirmed
Status in sosreport source package in Zesty:
  Confirmed
Status in sosreport source package in Artful:
  Confirmed
Status in sosreport package in Debian:
  Confirmed

Bug description:
  [Impact]

  The docker plugin in sosreport doesn't collect informations for
  Ubuntu.

  In docker.py for the Ubuntu Plugin, the docker command is set to
  'docker.io'[1] which seems to be a old workaround that has been
  removed[2], thus the docker plugin is no longer calling an existing
  docker binary which as a result doesn't collect docker informations in
  Ubuntu.

  [1]- Plugin : docker.py

  # Name collision with another package requires docker binary rename
  docker_cmd = 'docker.io'

  [2]- docker.io package : debian/changelog

  docker.io (1.3.1~dfsg1-2) unstable; urgency=medium
  * Remove deprecated /usr/bin/docker.io symlink
  - added as a temporary shim in 1.0.0~dfsg1-1 (13 Jun 2014)
  - unused by package-installed files in 1.2.0~dfsg1-1 (13 Sep 2014)
  -- Tianon Gravi <admwiggin@xxxxxxxxx> Fri, 07 Nov 2014 13:11:34 -0700

  [Test Case]

  # Install sosreport from the Ubuntu archive
  $ sudo apt-get update
  $ sudo apt-get install sosreport

  # Run sosreport with docker plugin only or all loaded plugins (for the
  sake of testing docker plugin the "-o" is more appropriate

  Docker plugin only :
  $ sosreport -o docker

  All loaded plugins:
  $ sosreport -a

  # Extract the sosreport*.tar.xz file generated by the above command
  found in /tmp

  # Look in "/path_to_sosreport/sos_commands/docker/" and note that all
  files are empty with "0" size.

  [Regression Potential]

  No regression, this is a trivial fix that will allow sosreport to
  start collecting the docker informations again.

  If for some reasons, a regression is found, it would be 100% limited
  to docker plugin of sosreport and not affecting the functionality the
  core functionality nor other plugins.

  [Other Info]

  Additional informations about the context can be found here :
  https://bugs.launchpad.net/ubuntu/+source/docker.io/+bug/1338768

  * Upstream Bug
  https://github.com/sosreport/sos/issues/930

  * Pull Request
  https://github.com/sosreport/sos/pull/1009
  https://github.com/sosreport/sos/pull/1009/commits/f8a939d792d8ac683f341458b8cf77a35c4db929

  * Debian Bug :
  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=863374
  (Patch submitted to Debian on June 2 2017)

  [Original Description]

  This Launchpad bug is based on an existing GitHub bug[1].

  The docker plugin is currently broken in sosreport. In docker.py[2]
  for Ubuntu the docker command is set to 'docker.io'.

      docker_cmd = 'docker.io'

  The docker.io package used to have a docker.io binary but this was
  reverted in this debian/changelog from Ubuntu Xenial/16.04 (LTS):

   docker.io (1.3.1~dfsg1-2) unstable; urgency=medium
      * Remove deprecated /usr/bin/docker.io symlink
        - added as a temporary shim in 1.0.0~dfsg1-1 (13 Jun 2014)
        - unused by package-installed files in 1.2.0~dfsg1-1 (13 Sep 2014)
     -- Tianon Gravi <admwiggin@xxxxxxxxx>  Fri, 07 Nov 2014 13:11:34 -0700

  The Docker Community and Enterprise Editions (docker-ce and docker-ee
  packages) also use the plain docker binary and this change is
  necessary to make the sosreport docker module work again.

  There are currently two different pull requests that fix this issue:

  https://github.com/sosreport/sos/pull/1009
  https://github.com/sosreport/sos/pull/1012

  We should SRU this fix once one of them is accepted.

  [1] https://github.com/sosreport/sos/issues/930
  [2] https://github.com/sosreport/sos/blob/master/sos/plugins/docker.py

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