Project

General

Profile

Actions

Feature #1427

closed

add support for each processing host to have it's own wrapper path

Added by Amber Herold about 13 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Christopher Irving
Category:
-
Target version:
Start date:
09/21/2011
Due date:
% Done:

0%

Estimated time:

Description

This will allow betamyami to be installed on Garibaldi

Actions #1

Updated by Amber Herold about 13 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Amber Herold to Christopher Irving

For this to work, the config file needs to have an entry for the wrapperpath added to each processing host array entry like this:

$PROCESSING_HOSTS[] = array('host' => 'guppy.scripps.edu',
                            'nproc' => 8,
                            'nodesdef' => '2',
                            'nodesmax' => '8',
                            'ppndef' => '8',
                            'ppnmax' => '8',
                            'reconpn' => '8',
                            'walltimedef' => '2',
                            'walltimemax' => '2',
                            'cputimedef' => '2',
                            'cputimemax' => '2',
                            'memorymax' => '30',
                            'appionbin' => '/opt/myamisnap/bin/appion/',
                            'baseoutdir' => DEFAULT_APPION_PATH,
                            'localhelperhost' => 'guppy.scripps.edu',
                            'dirsep' => '/',
                            'wrapperpath' => '/opt/myamisnap/bin/appion' );

USE_APPION_WRAPPER must still be defined as true for the paths to be added to commands.

If the wrapperpath entry is not defined for a host in the config file, no path will be added to a command even if USE_APPION_WRAPPER is true.

Actions #2

Updated by Amber Herold about 13 years ago

Ok... it was silly not to revert to the Older config file if the newer format is not there. Fixed that.

Actions #3

Updated by Christopher Irving about 13 years ago

  • Status changed from In Code Review to In Test

Looks good to me.

Actions #4

Updated by Christopher Irving about 13 years ago

  • Status changed from In Test to Closed

Tested - works perfectly.

Actions

Also available in: Atom PDF