IndependentPlugin is not working

Bug #1917651 reported by Eric Desrochers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
sosreport (Ubuntu)
Fix Released
High
Ponnuvel Palaniyappan
Focal
Fix Released
High
Ponnuvel Palaniyappan
Groovy
Fix Released
High
Ponnuvel Palaniyappan
Hirsute
Fix Released
High
Ponnuvel Palaniyappan

Bug Description

[IMPACT]
Regression found while doing the verification testing of (LP: #1913284)

As described by my colleague Ponnuvel:

"
The problem is that IndependentPlugin is not working in Focal's sosreport. The bcache plugin uses IndependentPlugin (as in, it's not tied to any specific Distro).

IndependentPlugin was broken at some point since Bionic and has been fixed upstream. (It's working on Bionic in 3.X series).

 sosreport | 3.9.1-1ubuntu0.18.04.3 | bionic-updates
 sosreport | 4.0-1~ubuntu0.20.04.4 | focal-proposed
 sosreport | 4.0-1ubuntu2.2 | groovy-proposed
 sosreport | 4.0-1ubuntu9 | hirsute

However, the sos report package on Focal, Groovy, and Hirsute all have the broken code.
"

It currently impacts plugins relying on "IndependentPlugin" to run such as the bcache plugin.

"IndependentPlugin" is a class for plugins that can run on any platform.

[TEST PLAN]
The patch includes a fix for the IndependentPlugin. Currently, bcache plugin uses the IndependentPlugin. So sosreport collection has to be tested on a machine with bcache deployment. The cherry-picked commit includes a number of other changes, so --all-logs and -a options would need to be used to ensure there's no other breakages.

[WHERE PROBLEM COULD OCCUR]

The IndependentPlugin may still not work - in that case, it'd only affect bcache plugin. Worse, the changes could affect other Plugin types and cause more regressions - affecting data collection for multiple plugins.

[OTHER INFORMATION]

Upstream bug:
https://github.com/sosreport/sos/pull/2018

Upstream commit:
https://github.com/sosreport/sos/commit/a36e1b83040f3f2c63912d4601f4b33821cd4afb

Tags: seg sts
Eric Desrochers (slashd)
description: updated
tags: added: seg sts
Changed in sosreport (Ubuntu):
assignee: nobody → Ponnuvel Palaniyappan (pponnuvel)
Changed in sosreport (Ubuntu Focal):
assignee: nobody → Ponnuvel Palaniyappan (pponnuvel)
Changed in sosreport (Ubuntu Groovy):
assignee: nobody → Ponnuvel Palaniyappan (pponnuvel)
status: New → In Progress
Changed in sosreport (Ubuntu Focal):
status: New → In Progress
Changed in sosreport (Ubuntu):
status: New → In Progress
description: updated
description: updated
Eric Desrochers (slashd)
description: updated
description: updated
Revision history for this message
Ponnuvel Palaniyappan (pponnuvel) wrote :

Attaching debdiff for focal.

Revision history for this message
Ponnuvel Palaniyappan (pponnuvel) wrote :

Attached debdiff for groovy.

Revision history for this message
Eric Desrochers (slashd) wrote :

Thanks Ponnuvel.

May I ask two more things:

* Debdiff for the active devel release (hirsute).
* Fill the SRU tmpl above that I already started.

- Eric

description: updated
description: updated
Changed in sosreport (Ubuntu Hirsute):
importance: Undecided → High
Changed in sosreport (Ubuntu Groovy):
importance: Undecided → High
Changed in sosreport (Ubuntu Focal):
importance: Undecided → High
Revision history for this message
Ponnuvel Palaniyappan (pponnuvel) wrote :

Attached debdiff for hirsute.

description: updated
Revision history for this message
Ponnuvel Palaniyappan (pponnuvel) wrote :

Done both - thanks, Eric!

Revision history for this message
Eric Desrochers (slashd) wrote :

I just noticed upstream released v4.1 which contains the fix for the broken IndependentPlugin.

I'll start working to push this new release in Debian/Unstable and then sync with Hirsute.

Revision history for this message
Eric Desrochers (slashd) wrote :

So the plan for now is to sync Hirsute with Debian/unstable once Debian approves it.

Then I'll SRU the fix in stable release with the debdiff you have provided, and then plan an upgrade to 4.0 to 4.1 on stable release later on.

- Eric

Revision history for this message
Eric Desrochers (slashd) wrote :

Fixed in 4.1 version of sos now found in in H/G/F/B.

Changed in sosreport (Ubuntu Hirsute):
status: In Progress → Fix Released
Changed in sosreport (Ubuntu Groovy):
status: In Progress → Confirmed
status: Confirmed → Fix Released
Changed in sosreport (Ubuntu Focal):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.