Code review comment for lp:~xfactor973/apparmor-profiles/ceph-apparmor-profiles

Revision history for this message
Steve Beattie (sbeattie) wrote :

Hi, thanks for fixing up the pid/pids variables. I've gone ahead and merged this after also copying the profiles to the 16.04 and 16.10 trees -- I poked around very briefly at a 16.04 ceph install and didn't see anything radically different in FS layout that would give cause for concern (we'll obivously take updates if there things that need to be updated).

While reviewing, I did have one question; both profiles have:

  owner /etc/ceph/* rw,

Is it expected that ceph will need to write to arbitrary files under /etc/ceph/? What's the usage here?

Thanks for the contribution!

review: Approve

« Back to merge proposal