Docker plugin uses the wrong command for Ubuntu
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
sosreport |
Fix Released
|
Unknown
|
|||
sosreport (Debian) |
Fix Released
|
Unknown
|
|||
sosreport (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Trusty |
Fix Committed
|
Medium
|
Eric Desrochers | ||
Xenial |
Fix Committed
|
Medium
|
Eric Desrochers | ||
Zesty |
Won't Fix
|
Medium
|
Eric Desrochers | ||
Artful |
Fix Committed
|
Medium
|
Eric Desrochers |
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 <email address hidden> Fri, 07 Nov 2014 13:11:34 -0700
[Test Case]
# Install sosreport from the Ubuntu archive
$ apt-get update
$ apt-get install sosreport
# Install Docker from the Ubuntu archive
$ apt-get install docker.io
# 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 and verify that information is gathered properly
[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:/
* Upstream Bug
https:/
* Pull Request
https:/
https:/
* Debian Bug :
https:/
(Patch submitted to Debian on June 2 2017)
* Ubuntu Bug :
Working on a backport into stable release (SRU) which will include this change via (LP: #1734983).
[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 <email address hidden> 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:/
https:/
We should SRU this fix once one of them is accepted.
[1] https:/
[2] https:/
description: | updated |
Changed in sosreport (Ubuntu Trusty): | |
assignee: | nobody → David Coronel (davecore) |
Changed in sosreport (Ubuntu Xenial): | |
assignee: | nobody → David Coronel (davecore) |
Changed in sosreport (Ubuntu Yakkety): | |
assignee: | nobody → David Coronel (davecore) |
Changed in sosreport (Ubuntu Zesty): | |
assignee: | nobody → David Coronel (davecore) |
Changed in sosreport (Ubuntu Artful): | |
assignee: | nobody → David Coronel (davecore) |
Changed in sosreport (Ubuntu Trusty): | |
importance: | Undecided → Medium |
Changed in sosreport (Ubuntu Xenial): | |
importance: | Undecided → Medium |
Changed in sosreport (Ubuntu Yakkety): | |
importance: | Undecided → Medium |
Changed in sosreport (Ubuntu Zesty): | |
importance: | Undecided → Medium |
Changed in sosreport (Ubuntu Artful): | |
importance: | Undecided → Medium |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
Changed in sosreport (Debian): | |
status: | Unknown → Confirmed |
description: | updated |
Changed in sosreport: | |
status: | Unknown → New |
Changed in sosreport (Ubuntu Xenial): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Yakkety): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Artful): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Zesty): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
tags: | added: patch |
description: | updated |
tags: | added: sts sts-sru-needed |
Changed in sosreport (Ubuntu Xenial): | |
status: | New → Confirmed |
Changed in sosreport (Ubuntu Yakkety): | |
status: | New → Confirmed |
Changed in sosreport (Ubuntu Zesty): | |
status: | New → Confirmed |
Changed in sosreport (Ubuntu Artful): | |
status: | New → Confirmed |
Changed in sosreport (Ubuntu Trusty): | |
status: | New → Confirmed |
tags: | removed: sts-sru-needed |
Changed in sosreport: | |
status: | New → Fix Released |
Changed in sosreport (Ubuntu Artful): | |
status: | Confirmed → In Progress |
Changed in sosreport (Ubuntu Zesty): | |
status: | Confirmed → In Progress |
Changed in sosreport (Ubuntu Yakkety): | |
status: | Confirmed → In Progress |
Changed in sosreport (Ubuntu Xenial): | |
status: | Confirmed → In Progress |
Changed in sosreport (Ubuntu Trusty): | |
status: | Confirmed → In Progress |
Changed in sosreport (Ubuntu Yakkety): | |
status: | In Progress → Won't Fix |
Changed in sosreport (Ubuntu Artful): | |
assignee: | Eric Desrochers (slashd) → David Coronel (davecore) |
Changed in sosreport (Ubuntu Zesty): | |
assignee: | Eric Desrochers (slashd) → David Coronel (davecore) |
Changed in sosreport (Ubuntu Xenial): | |
assignee: | Eric Desrochers (slashd) → David Coronel (davecore) |
Changed in sosreport (Ubuntu): | |
assignee: | Eric Desrochers (slashd) → David Coronel (davecore) |
Changed in sosreport (Ubuntu Trusty): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Xenial): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Zesty): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Artful): | |
assignee: | David Coronel (davecore) → Eric Desrochers (slashd) |
Changed in sosreport (Debian): | |
status: | Confirmed → Fix Released |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
tags: |
added: verification-needed verification-needed-artful verification-needed-trusty verification-needed-xenial removed: patch |
tags: |
added: verification-done-artful verification-done-trusty verification-done-xenial removed: verification-needed-artful verification-needed-trusty verification-needed-xenial |
tags: |
added: verification-done removed: verification-needed |
Changed in sosreport (Ubuntu Zesty): | |
status: | Fix Committed → Won't Fix |
Debdiff for Xenial