<p>George Joseph <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/8386">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins2: Verified
  Kevin Harwell: Looks good to me, but someone else must approve
  Corey Farrell: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  George Joseph: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ast_coredumper:  Minor fixes<br><br>* Fix --tarball-config so the option doesn't cause an error.<br><br>* Allow for missing /etc/os-release.<br><br>* Add a sleep between tarballing the coredump and removing the<br>  output directory to allow the filesystem to settle.<br><br>Change-Id: I73e03b13087978bcc7f6bc9f45753990f82d9d77<br>---<br>M contrib/scripts/ast_coredumper<br>1 file changed, 6 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/contrib/scripts/ast_coredumper b/contrib/scripts/ast_coredumper<br>index 884ede7..38d95cc 100755<br>--- a/contrib/scripts/ast_coredumper<br>+++ b/contrib/scripts/ast_coredumper<br>@@ -216,6 +216,7 @@<br> RUNNING=false<br> latest=false<br> tarball_coredumps=false<br>+tarball_config=false<br> delete_coredumps_after=false<br> tarball_results=false<br> delete_results_after=false<br>@@ -280,6 +281,9 @@<br>                 ;;<br>    --tarball-coredumps)<br>          tarball_coredumps=true<br>+               ;;<br>+   --tarball-config)<br>+            tarball_config=true<br>           ;;<br>    --delete-coredumps-after)<br>             delete_coredumps_after=true<br>@@ -436,7 +440,7 @@<br> <br>           ln -s ${cf} ${dest}/tmp/${cfname}<br>             cp ${outputdir}/${cfname}*.txt ${dest}/tmp/<br>-          cp /etc/os-release ${dest}/etc/<br>+              [ -f /etc/os-release ] && cp /etc/os-release ${dest}/etc/<br>             if $tarball_config ; then<br>                     cp -a /etc/asterisk ${dest}/etc/<br>              fi<br>@@ -445,6 +449,7 @@<br>               cp -a /usr/sbin/asterisk ${dest}/usr/sbin<br>             rm -rf ${tf}<br>          tar -chzf ${tf} --transform="s/^[.]/${cfname}/" -C ${dest} .<br>+               sleep 3<br>               rm -rf ${dest}<br>                echo "Created $tf"<br>  elif $tarball_results ; then<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/8386">change 8386</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/8386"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I73e03b13087978bcc7f6bc9f45753990f82d9d77 </div>
<div style="display:none"> Gerrit-Change-Number: 8386 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>