[sru] remove juju-db stop/start service interactions
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
sosreport (Ubuntu) |
Fix Released
|
Low
|
Eric Desrochers | ||
Trusty |
Won't Fix
|
Low
|
Unassigned | ||
Xenial |
Fix Released
|
Medium
|
Eric Desrochers | ||
Bionic |
Fix Released
|
Medium
|
Eric Desrochers | ||
Cosmic |
Won't Fix
|
Low
|
Unassigned | ||
Disco |
Fix Released
|
Medium
|
Eric Desrochers | ||
Eoan |
Fix Released
|
Medium
|
Eric Desrochers |
Bug Description
[Impact]
The juju plugin will stop and start the juju-db service during data collection.
sosreport should not impact running services, or attempt to recover them.
This has been reported upstream[0] and will be fixed by the juju 2.x refactor[1]
This is a stop-gap tracking the removal of the juju-db service restart code in existing sosreport releases.
[0] - https:/
[1] - https:/
[Test Case]
* Make sure you are in the juju controller.
* Install sosreport (if not present already)
$ sudo apt-get install sosreport -y
* Look mongod PID before
$ pidof mongod
* Run sosreport, ensuring that the juju plugin is exercised
$ sosreport -a # Should detect it if /usr/bin/juju' or '/usr/bin/juju-run' are found
and/or
$ sosreport -o juju # In a snap package env, it's possible sosreport doesn't detect it and the plugin would need to be enforce.
* Confirm the juju-db service was not restarted, and mongoexport data captured by looking mongod PID after
$ pidof mongod
Check for errors while running, or in /tmp/sosreport-
The offending function ensure_
[Regression Potential]
* Risk is low.
* Change is limited in scope to the juju plugin.
* Worst-case scenario is that the mongoexport command will fail to collect any data, which won't affect core functionality of sosreport itself nor impact other sosreport plugins.
If any regression, it will be way less worse than current situation, where daemon are susceptible to be restarted. It's not sosreport responsability to take action on daemons, it should only monitor it and report back to users via its respective plugin.
[Other information]
We will temporary divert from the juju plugin found upstream and debian, while the refactoring is completed to avoid any situation where sosreport is run on a controller since it may have production impact on Ubuntu juju environment.
Once the refactoring of the juju plugin is completed upstream, we will make sure to update debian and put the juju plugin align with what found upstream and debian.
Actually, sosreport 3.8 micro-release is blocked waiting for this refactoring to be completed (LP: #1825010).
Changed in sosreport (Ubuntu): | |
milestone: | none → eoan-updates |
milestone: | eoan-updates → none |
Changed in sosreport (Ubuntu Xenial): | |
assignee: | nobody → Dan Hill (hillpd) |
Changed in sosreport (Ubuntu Bionic): | |
assignee: | nobody → Dan Hill (hillpd) |
Changed in sosreport (Ubuntu Cosmic): | |
assignee: | nobody → Dan Hill (hillpd) |
Changed in sosreport (Ubuntu Disco): | |
assignee: | nobody → Dan Hill (hillpd) |
Changed in sosreport (Ubuntu Eoan): | |
assignee: | nobody → Dan Hill (hillpd) |
Changed in sosreport (Ubuntu Trusty): | |
assignee: | nobody → Dan Hill (hillpd) |
importance: | Undecided → Low |
Changed in sosreport (Ubuntu Xenial): | |
importance: | Undecided → Low |
Changed in sosreport (Ubuntu Bionic): | |
importance: | Undecided → Low |
Changed in sosreport (Ubuntu Cosmic): | |
importance: | Undecided → Low |
Changed in sosreport (Ubuntu Disco): | |
importance: | Undecided → Low |
Changed in sosreport (Ubuntu Eoan): | |
importance: | Undecided → Low |
tags: | added: sts |
Changed in sosreport (Ubuntu Eoan): | |
status: | New → In Progress |
description: | updated |
description: | updated |
Changed in sosreport (Ubuntu Trusty): | |
status: | New → Won't Fix |
description: | updated |
Changed in sosreport (Ubuntu): | |
assignee: | Dan Hill (hillpd) → Eric Desrochers (slashd) |
status: | In Progress → Fix Released |
Changed in sosreport (Ubuntu Cosmic): | |
status: | New → Won't Fix |
Changed in sosreport (Ubuntu Eoan): | |
assignee: | Dan Hill (hillpd) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Disco): | |
assignee: | Dan Hill (hillpd) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Cosmic): | |
assignee: | Dan Hill (hillpd) → nobody |
Changed in sosreport (Ubuntu Bionic): | |
assignee: | Dan Hill (hillpd) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Xenial): | |
assignee: | Dan Hill (hillpd) → Eric Desrochers (slashd) |
Changed in sosreport (Ubuntu Disco): | |
status: | New → In Progress |
Changed in sosreport (Ubuntu Bionic): | |
status: | New → In Progress |
Changed in sosreport (Ubuntu Xenial): | |
status: | New → In Progress |
Changed in sosreport (Ubuntu Trusty): | |
assignee: | Dan Hill (hillpd) → nobody |
description: | updated |
description: | updated |
description: | updated |
tags: |
added: verification-done-bionic removed: verification-needed-bionic |
I marked Trusty as 'Won't Fix' as it reached EOL.