Bug 1351375

Summary:	Rados Gateway (RGW) logrotate script references wrong path
Product:	[Red Hat Storage] Red Hat Ceph Storage	Reporter:	Michael J. Kidd <linuxkidd>
Component:	Distribution	Assignee:	ceph-eng-bugs <ceph-eng-bugs>
Status:	CLOSED WORKSFORME	QA Contact:	ceph-qe-bugs <ceph-qe-bugs>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	1.3.2	CC:	branto, kdreyer
Target Milestone:	rc
Target Release:	2.0
Hardware:	Unspecified
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-06-30 16:55:35 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Michael J. Kidd 2016-06-29 22:39:44 UTC

Description of problem:
In the ceph-radosgw package associated with:
https://access.redhat.com/errata/RHBA-2016:0313

Which 'fixes' BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1305981

The default log path for the RadosGW was updated to be /var/log/ceph/radosgw/, however the logrotate script still references /var/log/radosgw/.

This will cause the RGW to not be 'HUP'd whenever the log rotation occurs and RGW logging will stop writing at the first log rotation event.

Version-Release number of selected component (if applicable):
1.3.2 and all prior.


How reproducible:
Every time

Steps to Reproduce:
1. Install ceph-radosgw package
2. Confirm /etc/logrotate.d/radosgw script points to the rotation path of:
 /var/log/radosgw/*.log

Actual results:
 - Log rotation fails to properly rotate and/or restart the RGW process.

Expected results:
 - Log rotation to rotate the RGW logs in the new default path, and HUP RGW to re-open the log file after rotation.

Additional info:
n/a

Comment 2 Ken Dreyer (Red Hat) 2016-06-30 16:48:23 UTC

I'm having trouble seeing how bz 1305981 altered the default logging path to /var/log/ceph/radosgw. Michael, can you please provide more information there?

Boris, can you confirm?

Comment 3 Michael J. Kidd 2016-06-30 16:55:35 UTC

Sorry Ken, et al
  I mis-read the other BZ, and had someone else mention it in a way that suggested everything now went to /var/log/ceph/radosgw, so my pre-disposition to that thought skewed my reading.  I'm closing this BZ.. it's based on an incorrect reading of the other BZ.

Comment 4 Ken Dreyer (Red Hat) 2016-06-30 16:58:20 UTC

Thanks!