Merge ~rodsmith/plainbox-provider-checkbox:fix-lxd-test-2 into plainbox-provider-checkbox:master

Proposed by Rod Smith
Status: Merged
Approved by: Jeff Lane 
Approved revision: 0eaba95f40f993d82ce0ef0986eff1958b7552f7
Merged at revision: 54aacfe6071a68ebce87205aae119aefc2a8af6a
Proposed branch: ~rodsmith/plainbox-provider-checkbox:fix-lxd-test-2
Merge into: plainbox-provider-checkbox:master
Diff against target: 12 lines (+1/-1)
1 file modified
units/miscellanea/jobs.pxu (+1/-1)
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Review via email: mp+372996@code.launchpad.net

Commit message

Work around a problem caused by sosreport running as root and creating root-owned files in ~/.config/lxc.

Description of the change

This MR uses the "-n" option to sosreport to disable sosreport's LXD test, thus working around the sosreport bug. I've filed bug reports against sosreport directly:

https://bugs.launchpad.net/ubuntu/+source/sosreport/+bug/1844663

https://github.com/sosreport/sos/issues/1787

To post a comment you must log in.
Revision history for this message
Rod Smith (rodsmith) wrote :
Revision history for this message
Jeff Lane  (bladernr) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/miscellanea/jobs.pxu b/units/miscellanea/jobs.pxu
2index ca726a6..1641790 100644
3--- a/units/miscellanea/jobs.pxu
4+++ b/units/miscellanea/jobs.pxu
5@@ -421,7 +421,7 @@ estimated_duration: 20.0
6 id: miscellanea/sosreport
7 user: root
8 requires: executable.name == 'sosreport'
9-command: sosreport --batch --tmp-dir $PLAINBOX_SESSION_SHARE
10+command: sosreport --batch -n lxd --tmp-dir $PLAINBOX_SESSION_SHARE
11 _summary: Generate baseline sosreport
12 _description: Generates a baseline sosreport of logs and system data
13

Subscribers

People subscribed via source and target branches