Skip to content

Commit

Permalink
Fixed more paths. Ensured params to CloudFormation are strings. Made …
Browse files Browse the repository at this point in the history
…clear in doco that RIOS needs to be installed for AWS Batch scripts.
  • Loading branch information
gillins committed Apr 19, 2024
1 parent 1debed7 commit 61ac3b3
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 5 deletions.
2 changes: 2 additions & 0 deletions doc/source/awsbatch.rst
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ This directory contains an example of using RIOS with AWS Batch.
Please ensure you understand the concepts used and the changes (and costs!)
associated with these examples before you use them.

The helper scripts below assume RIOS is installed on the machine you are using them on.

Files
-----

Expand Down
4 changes: 2 additions & 2 deletions tools/awsbatch/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ default: all
# grab the current RIOS source tree and make it available to the
# docker COPY command
dist:
cd ../../../..;python3 setup.py sdist --formats=gztar
cp ../../../../dist/rios-$(RIOS_VER).tar.gz .
cd ../..;python3 setup.py sdist --formats=gztar
cp ../../dist/rios-$(RIOS_VER).tar.gz .

# Login to ECR, build package and push to ECR
all: dist
Expand Down
File renamed without changes.
6 changes: 3 additions & 3 deletions tools/awsbatch/createbatch.py
Original file line number Diff line number Diff line change
Expand Up @@ -89,13 +89,13 @@ def createBatch(stackname, region, ecrName, vCPUs, maxMem, maxJobs,
addParam(params, 'ECR_Name', ecrName)

if vCPUs is not None:
addParam(params, 'VCPUS', vCPUs)
addParam(params, 'VCPUS', str(vCPUs))

if maxMem is not None:
addParam(params, 'MaxMemory', maxMem)
addParam(params, 'MaxMemory', str(maxMem))

if maxJobs is not None:
addParam(params, 'MaxJobs', maxJobs)
addParam(params, 'MaxJobs', str(maxJobs))

if instanceType is not None:
addParam(params, 'InstanceType', instanceType)
Expand Down

0 comments on commit 61ac3b3

Please sign in to comment.