liuzhe02 / bigbluebutton

Automatically exported from code.google.com/p/bigbluebutton
0 stars 0 forks source link

Processing Temp Directory not Removed #1697

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
When post processing a recorded meeting a temp directory is created.  This 
directory is not removed if the post-processing script fails.

This directory was created: 
/var/bigbluebutton/recording/process/presentation/68997a347e267f85240c9dbf29442b
150eb25952-1386630525306/temp

But the post processing script failed (due to bug #1410) and the temp directory 
was not cleaned up, here are the last few lines in the logs.

-----
I, [2013-12-09T18:52:33.622478 #9433]  INFO -- : Exit status: 1
E, [2013-12-09T18:52:33.622627 #9433] ERROR -- : sox failed, exit code 1
E, [2013-12-09T18:52:33.622717 #9433] ERROR -- : 
/usr/local/bigbluebutton/core/lib/recordandplayback/edl/audio.rb:124:in `block 
in render'
E, [2013-12-09T18:52:33.622799 #9433] ERROR -- : 
/usr/local/bigbluebutton/core/lib/recordandplayback/edl/audio.rb:85:in `each'
E, [2013-12-09T18:52:33.622876 #9433] ERROR -- : 
/usr/local/bigbluebutton/core/lib/recordandplayback/edl/audio.rb:85:in `render'
E, [2013-12-09T18:52:33.622954 #9433] ERROR -- : 
/usr/local/bigbluebutton/core/lib/recordandplayback/generators/audio_processor.r
b:39:in `process'
E, [2013-12-09T18:52:33.623031 #9433] ERROR -- : 
/usr/local/bigbluebutton/core/scripts/process/presentation.rb:61:in `<main>'
-----

Original issue reported on code.google.com by david.bu...@edoceo.com on 10 Dec 2013 at 9:44

GoogleCodeExporter commented 9 years ago

Original comment by ffdixon@gmail.com on 15 Dec 2013 at 2:22

GoogleCodeExporter commented 9 years ago

Original comment by ritza...@gmail.com on 27 Jun 2014 at 4:06

GoogleCodeExporter commented 9 years ago

Original comment by ffdixon@gmail.com on 30 Jul 2014 at 9:39

GoogleCodeExporter commented 9 years ago
This is actually something that we're not going to be changing.

If the recording processing scripts fail, we are leaving the temporary files 
present, since they can be useful for debugging the issues that caused the 
processing scripts to fail.

Original comment by calvin.walton@kepstin.ca on 30 Jul 2014 at 9:44