Bug 762095 (GLUSTER-363) - Logrotate doesn't work
Summary: Logrotate doesn't work
Keywords:
Status: CLOSED NOTABUG
Alias: GLUSTER-363
Product: GlusterFS
Classification: Community
Component: logging
Version: mainline
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Harshavardhana
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-11-09 08:01 UTC by Harshavardhana
Modified: 2015-03-23 01:03 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

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..
Note You need to log in before you can comment on or make changes to this bug.