Ubuntu
sosreport package

sosreport doesn't collect all data from /etc/apparmor.d/

Bug #1366103 reported by Michael Cunningham
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
sosreport (Ubuntu)
Fix Released
Undecided
Unassigned
Trusty
In Progress
Medium
Unassigned
Vivid
Won't Fix
Undecided
Unassigned
Xenial
Fix Released
Undecided
Unassigned
Yakkety
Fix Released
Undecided
Unassigned
Zesty
Fix Released
Undecided
Unassigned

Bug Description

[SRU justification]
All the apparmor data needs to be collected

[Impact]
Problem in investigating AppArmor related issue without the data

[Fix]
Enhance the apparmor plugin to collect all the necessary data

[Test Case]
Without the fix, sosreport will only collect the following files :

./etc/apparmor.d/sbin.dhclient
./etc/apparmor/subdomain.conf
./etc/apparmor/init/network-interface-security/sbin.dhclient

With the fix, it will collect all of
./apparmor.d
./apparmor

and run :
ls -alh /etc/apparmor.d/libvirt
ls -alh /etc/apparmor.d/abstractions

[Regression]
Minimal, this plugin already runs on Yakkety, Xenial & Zesty

[Original description of the problem]
Sosreport appears to collect only sbin.dhclient from /etc/apparmor.d and ignores all other files/dirs.

I have observed this on 12.04, 13.10 and 14.04.

I would expect that it grabs all files/dirs from that directory.

See original description
Revision history for this message
Bryan Quigley (bryanquigley) wrote :

It also doesn't grab sudo apparmor_status

Revision history for this message
Bryan Quigley (bryanquigley) wrote :

Changing the plugin to:
    def setup(self):
        self.add_copy_specs([
            "/etc/apparmor*",
        ])

 self.add_cmd_outputs([
     "apparmor_status",
])

Does most of it, unfortunately, /etc/apparmor.d/cache looks like it will just waste our space.. (Aside, shouldn't cache not be stored in /etc?)

Revision history for this message
Bryan Quigley (bryanquigley) wrote :

https://github.com/sosreport/sos/pull/538

Changed in sosreport (Ubuntu):
status: New → Triaged
Louis Bouchard (louis)
Changed in sosreport (Ubuntu Yakkety):
status: New → Fix Released
Changed in sosreport (Ubuntu Xenial):
status: New → Fix Released
Changed in sosreport (Ubuntu Vivid):
status: New → Won't Fix
Changed in sosreport (Ubuntu Trusty):
status: New → Triaged
Changed in sosreport (Ubuntu Zesty):
status: Triaged → Fix Released
Changed in sosreport (Ubuntu Trusty):
importance: Undecided → Medium
assignee: nobody → Louis Bouchard (louis-bouchard)
Louis Bouchard (louis)
description: updated
Louis Bouchard (louis)
Changed in sosreport (Ubuntu Trusty):
status: Triaged → In Progress
Louis Bouchard (louis)
tags: added: sts-sru
Edward Hope-Morley (hopem)
tags: added: sts-sru-needed
removed: sts-sru
Louis Bouchard (louis)
Changed in sosreport (Ubuntu Trusty):
assignee: Louis Bouchard (louis) → nobody
Edward Hope-Morley (hopem)
tags: removed: sts-sru-needed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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