[Home]

Summary:ASTERISK-27846: ast_coredumper: Fix OUTPUT directory
Reporter:Ted G (tgwaste)Labels:
Date Opened:2018-05-09 20:49:44Date Closed:2018-05-31 05:17:00
Priority:MinorRegression?
Status:Closed/CompleteComponents:Addons/General
Versions:15.3.0 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:The OUTPUTDIR functionality seems to be broken in ast_coredumper:

gz/tmp# /var/lib/asterisk/scripts/ast_coredumper
OUTPUTDIR /some/directory doesn't exists or is not a directory

gz/tmp# /var/lib/asterisk/scripts/ast_coredumper /tmp/core
OUTPUTDIR /some/directory doesn't exists or is not a directory

gz/tmp# export OUTPUTDIR=/tmp ; echo $OUTPUTDIR
/tmp

gz/tmp# /var/lib/asterisk/scripts/ast_coredumper /tmp/core
OUTPUTDIR /some/directory doesn't exists or is not a directory

gz/tmp# asterisk -V
Asterisk 15.3.0

Looks like the problem is ast_debug_tools.conf is taking precedence over the environment setting.

[ -f /etc/asterisk/ast_debug_tools.conf ] && source /etc/asterisk/ast_debug_tools.conf
[ -f ~/ast_debug_tools.conf ] && source ~/ast_debug_tools.conf
[ -f ./ast_debug_tools.conf ] && source ./ast_debug_tools.conf

This doesn't seem like the appropriate behavior to me.  The user should be able to change the output directory on the command line and not have to edit a config file every time. Also, it seems like /tmp should be the default directory instead of /some/directory which would probably never exist.
Comments:By: Asterisk Team (asteriskteam) 2018-05-09 20:49:45.087-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Friendly Automation (friendly-automation) 2018-05-31 05:17:01.558-0500

Change 8991 merged by Joshua Colp:
ast_coredumper:  Fix output directory and variable precedence

[https://gerrit.asterisk.org/8991|https://gerrit.asterisk.org/8991]

By: Friendly Automation (friendly-automation) 2018-05-31 05:17:11.157-0500

Change 8994 merged by Joshua Colp:
ast_coredumper:  Fix output directory and variable precedence

[https://gerrit.asterisk.org/8994|https://gerrit.asterisk.org/8994]

By: Friendly Automation (friendly-automation) 2018-05-31 05:17:21.060-0500

Change 8993 merged by Joshua Colp:
ast_coredumper:  Fix output directory and variable precedence

[https://gerrit.asterisk.org/8993|https://gerrit.asterisk.org/8993]