[Home]

Summary:ASTERISK-17762: [patch] Confusing directory vestige /var/log/asterisk/cel-csv/
Reporter:Billy Chia (bchia)Labels:
Date Opened:2011-04-27 11:44:57Date Closed:2011-05-05 07:28:41
Priority:MinorRegression?No
Status:Closed/CompleteComponents:CEL/General
Versions:1.8.3 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) cel-csv.diff
Description:/var/log/asterisk/cel-csv/ directory is created during standard install but there is no mechanism to place files into it. (Unlike CDR where files are placed in ../cdr-csv/ by default.) The similarity between CDR and CEL makes the presence of this directory confusing.

The desired fix would be to remove the directory and update cel.conf.sample.

Currently says,

; Channel Event Logging is a mechanism to provide fine-grained event information
; that can be used to generate billing information. Such event information can
; be recorded to databases and files via pluggable backend modules.

My recommendation is say "Such event information is recorded to databases and files via pluggable backed modules." The verbage "can be" suggests that this is optional and that there is a default csv file like CDR.

Comments:By: Andrew Latham (lathama) 2011-04-27 12:48:10

Can you provide a patch?

By: Andrew Latham (lathama) 2011-04-27 12:58:30

There that was easy...

By: Digium Subversion (svnbot) 2011-05-05 07:27:58

Repository: asterisk
Revision: 317058

U   branches/1.8/Makefile
U   branches/1.8/configs/cel.conf.sample

------------------------------------------------------------------------
r317058 | lmadsen | 2011-05-05 07:27:57 -0500 (Thu, 05 May 2011) | 7 lines

Remove unused directory and clear up some documentation.

(closes issue ASTERISK-17762)
Reported by: bchia
Patches:
     cel-csv.diff uploaded by lathama (license 1028)
Tested by: lathama, Marquis42
------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=317058

By: Digium Subversion (svnbot) 2011-05-05 07:28:41

Repository: asterisk
Revision: 317059

_U  trunk/
U   trunk/Makefile
U   trunk/configs/cel.conf.sample

------------------------------------------------------------------------
r317059 | lmadsen | 2011-05-05 07:28:40 -0500 (Thu, 05 May 2011) | 15 lines

Merged revisions 317058 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
 r317058 | lmadsen | 2011-05-05 08:27:56 -0400 (Thu, 05 May 2011) | 7 lines
 
 Remove unused directory and clear up some documentation.
 
 (closes issue ASTERISK-17762)
 Reported by: bchia
 Patches:
       cel-csv.diff uploaded by lathama (license 1028)
 Tested by: lathama, Marquis42
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=317059