New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add default_remote_cerequirements #36

Merged
merged 1 commit into from Aug 17, 2015

Conversation

Projects
None yet
2 participants
@djw8605
Member

djw8605 commented Aug 7, 2015

Add default_remote_cerequirements. This enables the administrator to
specify routes with the attribute default_remote_cerequirements, which
will then be appended to the remote_cerequirements. Therefore, to add
a new type of resource (such as a gpu), the admin needs to:

  1. Add to the default_remote_cerequirements in the router entry so
    that the local submit file (.sh) can pick up the new requirement. For
    a GPU, it would be:
set_default_remote_cerequirements = "RequestGpus == 1"; \
  1. Add to the [pbs|lsf|sge|…]_local_submit_attributes.sh an if
    statement to capture and use this attribute. Again, for a GPU and
    Slurm:
if [ -n "$RequestGpus" ]; then
    echo "#SBATCH --gres=gpu:$RequestGpus"
    echo "module load cuda/6.0"
fi
Add default_remote_cerequirements
Add default_remote_cerequirements.  This enables the administrator to
specify routes with the attribute default_remote_cerequirements, which
will then be appended to the remote_cerequirements.  Therefore, to add
a new type of resource (such as a gpu), the admin needs to:

1. Add to the default_remote_cerequirements in the router entry so
that the local submit file (.sh) can pick up the new requirement.  For
a GPU, it would be:

    set_default_remote_cerequirements = "RequestGpus == 1"; \

2. Add to the [pbs|lsf|sge|…]_local_submit_attributes.sh an if
statement to capture and use this attribute.  Again, for a GPU and
Slurm:

    if [ -n "$RequestGpus" ]; then
        echo "#SBATCH --gres=gpu:$RequestGpus"
        echo "module load cuda/6.0"
     fi
@djw8605

This comment has been minimized.

Show comment
Hide comment
@djw8605

djw8605 Aug 7, 2015

Member

@brianhlin, could you take a look at this pull request.

Member

djw8605 commented Aug 7, 2015

@brianhlin, could you take a look at this pull request.

@brianhlin

This comment has been minimized.

Show comment
Hide comment
@brianhlin

brianhlin Aug 17, 2015

Member

Looks good to me

Member

brianhlin commented Aug 17, 2015

Looks good to me

brianhlin added a commit that referenced this pull request Aug 17, 2015

@brianhlin brianhlin merged commit 3b4f06c into opensciencegrid:master Aug 17, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment