Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

selinux: Allow log files to be located in /var/log/radosgw #7604

Merged
merged 1 commit into from Feb 11, 2016

Conversation

b-ranto
Copy link
Contributor

@b-ranto b-ranto commented Feb 11, 2016

We do suggest users to put their logs in /var/log/radosgw in the
documentation at times. We should also label that directory with
ceph_var_log_t so that ceph daemons can also write there.

The commit also updates the man page for this policy. This man page is
automatically generated by

  • sepolicy manpage -p . -d ceph_t

and have not been reloaded in a while. Hence, it contains few more
changes than the new radosgw directory.

Signed-off-by: Boris Ranto branto@redhat.com

We do suggest users to put their logs in /var/log/radosgw in the
documentation at times. We should also label that directory with
ceph_var_log_t so that ceph daemons can also write there.

The commit also updates the man page for this policy. This man page is
automatically generated by

* sepolicy manpage -p . -d ceph_t

and have not been reloaded in a while. Hence, it contains few more
changes than the new radosgw directory.

Signed-off-by: Boris Ranto <branto@redhat.com>
@b-ranto b-ranto force-pushed the wip-selinux-update-radosgw-log branch from a148b03 to bcf1204 Compare February 11, 2016 11:38
@b-ranto
Copy link
Contributor Author

b-ranto commented Feb 11, 2016

The failed build does not seem to be related to the SELinux changes -- it fails in a different area that these changes did not touch. The package seems to build fine in gitbuilders (at least where it finished already and where master builds fine).

liewegas added a commit that referenced this pull request Feb 11, 2016
selinux: allow log files to be located in /var/log/radosgw

Reviewed-by: Sage Weil <sage@redhat.com>
@liewegas liewegas merged commit 62ecb28 into master Feb 11, 2016
@liewegas liewegas deleted the wip-selinux-update-radosgw-log branch February 11, 2016 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants