Project

General

Profile

Actions

Bug #532

closed

default_cluster.php is a nightmare to configure

Added by Neil Voss almost 14 years ago. Updated over 12 years ago.

Status:
Won't Fix or Won't Do
Priority:
Normal
Assignee:
Amber Herold
Category:
-
Target version:
Start date:
05/21/2010
Due date:
% Done:

0%

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

Description

default_cluster.php is a nightmare to configure

(1) automate setting some variables
(2) rework it to include fewer options


Related issues 1 (0 open1 closed)

Related to Appion - Feature #1511: Update Config file template and config wizard to reflect new way of configuring clustersClosedAmber Herold01/10/2012

Actions
Actions #1

Updated by Neil Voss almost 14 years ago

  • Project changed from 5 to Appion
Actions #2

Updated by Amber Herold almost 14 years ago

  • Target version changed from Appion/Leginon 2.1.0 to Appion/Leginon Future Version
  • Affected Version set to Appion/Leginon 2.0.1
  • Show in known bugs set to No

We decided this is very low priority during 2.1.0 planning meeting.

Actions #3

Updated by Amber Herold almost 13 years ago

  • Status changed from New to Assigned
  • Assignee set to Amber Herold
  • Target version changed from Appion/Leginon Future Version to Appion/Leginon 2.2.0

I've made modifications to the way appion handles cluster config. Not checked in yet, will have Christopher review first.
Here is the idea.

1. default_cluster.php no longer exists. Installations will not have a seperate config file for each cluster. Instead, the main config.php file will include the constants required for each cluster in the processing hosts array with an entry for each one like this:

$PROCESSING_HOSTS[] = array('host' => 'garibaldi.scripps.edu', 
                'nproc' => 8,
                'nodesdef' => '16',
                'nodesmax' => '280',
                'ppndef' => '4',
                'ppnmax' => '8',
                'reconpn' => '4',
                'walltimedef' => '240',
                'walltimemax' => '240',
                'cputimedef' => '240',
                'cputimemax' => '240',
                'memorymax' => '30',
                'appionbin' => '~bcarr/appionbin/',
                'baseoutdir' => '', //sends appion procession output to a location under the users home directory on the remote host
                'remoteserver' => 'amibox03.scripps.edu',
                'dirsep' => '/' );

2. I created myamiweb/processing/inc/cluster.inc which is a small class that reads the config file params, provides accessor functions, and has some logic for setting the remote output path. Most of the functions that were included in the default_cluster file are no longer needed after our recent changes to the way jobs are submitted.

3. This class is used for populating the processing host parameters form, which is curerntly only used for refinements. The cluster class needs to know which host is being used when it is instantiated, so files use the PROCESSING_HOSTS array to look up the available hosts before creating a new cluster instance.

Actions #4

Updated by Amber Herold over 12 years ago

  • Status changed from Assigned to Won't Fix or Won't Do

This is no longer relevant as we have refactored the way we handle cluster configuration so this config file has been removed.

See r15878 where cluster.inc was added to take the place of default_cluster.php.
Also relevant are #1324 and #1511.

Actions

Also available in: Atom PDF