Bug 762095 (GLUSTER-363)

Summary:	Logrotate doesn't work
Product:	[Community] GlusterFS	Reporter:	Harshavardhana <fharshav>
Component:	logging	Assignee:	Harshavardhana <fharshav>
Status:	CLOSED NOTABUG	QA Contact:
Severity:	low	Docs Contact:
Priority:	low
Version:	mainline	CC:	amarts, cww, gluster-bugs
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Harshavardhana 2009-11-09 06:14:35 UTC

Testing was done on Gluster Storage Platform

By manually sending "SIGHUP" glusterfs never used to
refresh the logfile unless it used to open the same 
file with append mode. 

Patch sent for reviews to fix this issue.

Comment 1 Harshavardhana 2009-11-09 08:01:26 UTC

Logrotate functionality doesn't work 

kill -HUP doesn't work properly.

Comment 2 Anand Avati 2009-11-12 04:18:26 UTC

PATCH: http://patches.gluster.com/patch/2139 in master (Fixed logrotate functionality)

Comment 3 Amar Tumballi 2009-11-13 17:41:25 UTC

I just did a git pull, and noticed I don't have this patch included in that.. but both patchwork, and bugs show its committed.. Avati can you have a look on it??

Comment 4 Amar Tumballi 2009-11-13 17:57:25 UTC

[2009-11-13 12:49:53] C [logging.c:78:gf_log_logrotate] logrotate: rename failed for str.log -> str.log-20091113 (No such file or directory)

Noticed that this feature (and also the previous logrotate) doesn't work for 'relative' path log files. Works fine for log files with absolute path. No need to hurry on the fix.. but is nice to have working logrotate for this case too..