CDR Tickets

Issue Number 3628
Summary Audio download not working
Created 2013-07-23 14:52:59
Issue Type Bug
Submitted By Osei-Poku, William (NIH/NCI) [C]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2013-07-26 15:39:23
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.110782
Description

The audio download utility is not working. We get a python script error each time we attempt to download. This is one of the tasks we were prevented from testing so this is the first time we're using it in the production environment.

Her is the error message:

"A problem occurred in a Python script.
d:\cdr\Log\tmpb67iyh.html contains the description of this error. "

Comment entered 2013-07-25 17:33:41 by alan

I tried this in DEV and in PROD and got the same results as William. It works in DEV but fails in PROD, logging an error report.

Looking at the error report and at the source code, it looks to me like the program might be failing and raising an exception on line 114, attempting to change to the directory that contains the audio files. The exception handler catches the exception and attempts to log it (line 191) but, unfortunately, one piece of the information it tries to write (the contents of a variable named dbgmsg) hasn't been defined yet, so that undefined variable report is what appears in the error report instead of the info we wanted. However I'm more than 50% sure (I'm really going out on a limb aren't I?) that the problem was failing to change to the directory.

If I'm right, then either the program has the directory name wrong (unlikely), the name is right but CBIIT hasn't created the directory on the PROD ftp server, or the name is right and the directory exists but it has the wrong permissions for our ftp login credentials to change to it. This should be a straightforward fix but it requires CBIIT to check the correct directory name, existence, and permissions.

Since Volker has been working with CBIIT on setting up the ftp servers, I'm hoping he'll be able to look at this. If he isn't available and a crisis is looming, I'll call CBIIT from home tomorrow and step them through the checks, or else do it on Tuesday.

If I'm wrong and this isn't the cause, well, we'll try plan B. However this is looking enough like the problem that I don't think there's any point formulating a plan B until we've checked it out.

Comment entered 2013-07-25 17:46:45 by Englisch, Volker (NIH/NCI) [C]

I will take a look at this later today.

Comment entered 2013-07-25 18:38:46 by alan

If we can get CBIIT to move the definition of dbgmsg from line 139 to 113, the try/except block you put in should work and we might get a message that will confirm or deny my theory about what's happening.

Comment entered 2013-07-25 18:52:20 by Englisch, Volker (NIH/NCI) [C]

The production share of the FTP server hasn't been setup correctly and a directory is still missing. Vanessa could have reported this when she was able to only upload the audio files to the parent directory instead of the Term_Audio directory that's missing.

I've requested to have the missing directories created and the audio files moved.

Alan, if you think we should still correct the problem of the program failing before the definition of the dbgmsg variable we could keep this issue open after the FTP directories have been created or create a new issue separately.

Comment entered 2013-07-25 22:34:55 by alan

I propose that we leave the issue open at least until Tuesday when we're both in and can talk about it.

If CBIIT fixes the server directory then the other changes will be very low priority. However we may see problems like this in the future if there are any changes in the network or server setups, or when the new staging server comes online, so we might want to get all of the debug logging stuff you put in working shape just in case its needed again. The effort to do that should be pretty minimal.

Comment entered 2013-07-26 13:46:00 by Englisch, Volker (NIH/NCI) [C]

The missing directories on PROD have been created. Could you please test if the audio download is working now?

Comment entered 2013-07-26 15:38:49 by Osei-Poku, William (NIH/NCI) [C]

It worked. Thanks!

Files Retrieved:

Week_095.zip
Week_096.zip

Comment entered 2013-07-26 16:13:23 by Englisch, Volker (NIH/NCI) [C]

Very good.

I suggest to notify Vanessa since she had uploaded the files to a different directory last time so she'll know not to try again to upload the next file to the parent directory. Additionally, since we've just created that Term_Audio directory it wouldn't be a bad idea to have her confirm that she's still able to upload to this directory (and all of the permissions are set properly).

Elapsed: 0:00:00.001513