jewel: build/ops: backport 'logrotate: Run as root/ceph' #11201

Merged
merged 1 commit into from Oct 20, 2016

Projects

None yet

3 participants

@b-ranto
Contributor
b-ranto commented Sep 22, 2016 edited

http://tracker.ceph.com/issues/17382

Currently, we run the logrotate scripts as ceph/ceph but that way we
cannot rotate the scripts created by qemu (they are root/ceph and 644).
The original su line was introduced in commit 73d7bed because logrotate
complained that the directory was writable by a non-root group and it
needed a su line to supress that error. This way, we keep the group
settings and we can access and rotate the qemu logs as well.

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

@smithfarm smithfarm added this to the jewel milestone Sep 22, 2016
@smithfarm smithfarm self-assigned this Sep 22, 2016
@smithfarm
Contributor

added Backport tracker URL to PR description

@b-ranto b-ranto logrotate: Run as root/ceph
Currently, we run the logrotate scripts as ceph/ceph but that way we
cannot rotate the scripts created by qemu (they are root/ceph and 644).
The original su line was introduced in commit 73d7bed because logrotate
complained that the directory was writable by a non-root group and it
needed a su line to supress that error. This way, we keep the group
settings and we can access and rotate the qemu logs as well.

Signed-off-by: Boris Ranto <branto@redhat.com>
de672a0
@dachary dachary changed the title from Backport 'logrotate: Run as root/ceph' to jewel: build/ops: ackport 'logrotate: Run as root/ceph' Oct 10, 2016
@dachary dachary changed the title from jewel: build/ops: ackport 'logrotate: Run as root/ceph' to jewel: build/ops: backport 'logrotate: Run as root/ceph' Oct 10, 2016
@dachary
Member
dachary commented Oct 20, 2016

It passed the upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17487#note-22) suites.

@dachary dachary merged commit 3cb4bf4 into jewel Oct 20, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
@b-ranto b-ranto deleted the wip-backport-logrotate-jewel branch Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment