Project

General

Profile

Actions

Bug #864

closed

uploadimage.php of a new session does not work with cluster submitAppioJob function

Added by Anchi Cheng over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Eric Hou
Category:
-
Target version:
Start date:
09/08/2010
Due date:
% Done:

0%

Estimated time:
Affected Version:
Appion/Leginon 2.0.2
Show in known bugs:
No
Workaround:

submitAppionJob is redirected to justShowCommand when the jobtype is "uploadimage"


Description

I think this has never worked whether using old or new submitAppionJob function.

AppionJob submission with webcaller.py needs a place to put the job log file. There is no folder assigned at the php level for such a new Leginon session since the default comes from leginon.cfg of the processing server when the python script is run.

Actions #1

Updated by Anchi Cheng over 13 years ago

  • Workaround updated (diff)
Actions #2

Updated by Anchi Cheng over 13 years ago

  • Assignee changed from Anchi Cheng to Eric Hou
Actions #3

Updated by Amber Herold over 13 years ago

  • Target version set to Appion/Leginon 2.1.0
Actions #4

Updated by Eric Hou over 13 years ago

  • Status changed from Assigned to In Test
  • Assignee changed from Eric Hou to Anchi Cheng
Actions #5

Updated by Anchi Cheng over 13 years ago

  • Status changed from In Test to In Code Review

Test failed through guppy. $_SESSION does not have the key processinghost without getting it from $_POST. guppy's default appion (2.0) does not want jobtype but the trunk version on guppy (need Appion Wrapper) does.

I also don't like to have an entire new function uploadimageTable that has mostly repeated code from justShowCommandTable, because it will make it necessary to change both if we need to parse the parameters differently. committed as r14831 both fixes for the above bugs and changes to use extended justShowCommandTable function instead of uploadimageTable.

I think this ssh command directly is still not the permanent solution as it will be run from the head node of the cluster, not recommended or allowed in most cases. Christopher said that it will be o.k. in our case because this script does not use a lot of processor time.

Actions #6

Updated by Anchi Cheng over 13 years ago

  • Assignee changed from Anchi Cheng to Eric Hou
Actions #7

Updated by Eric Hou over 13 years ago

  • Status changed from In Code Review to Closed

I have tested this many times. And autoinstallation script is using it.

Thanks.

Actions

Also available in: Atom PDF