Package: sosreport
Severity: normal
Tags: upstream

Dear Maintainer,

This bug doesn't affect Debian ( no DebianPlugin for docker) but I am filing 
this bug for sosreport in debian to stay aligned.

[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 <admwig...@gmail.com> 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 of sosreport nor other 
plugin.

[Other Info]

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

The bug has been reported upstream already[3] and two similar patches has been 
proposed[4].
Let's see which one will be accepted/merged.

[3] - Upstream Bug
https://github.com/sosreport/sos/issues/930

[4] - Pull Request(s)
PR #1
https://github.com/sosreport/sos/pull/1009
https://github.com/sosreport/sos/pull/1009/commits/f8a939d792d8ac683f341458b8cf77a35c4db929

PR #2
https://github.com/sosreport/sos/pull/1012
https://github.com/davecore82/sos/commit/7d063cf0a5949a9819b80f68e1862d95b8c3fa35

[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 <admwig...@gmail.com> 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


-- System Information:
Debian Release: 7.11
  APT prefers oldstable
  APT policy: (500, 'oldstable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.8.0-52-generic (SMP w/4 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash

Reply via email to