Skip to content
This repository has been archived by the owner on Aug 9, 2023. It is now read-only.

FSx and Cromwell Updates #186

Merged
merged 2 commits into from
Mar 17, 2022
Merged

FSx and Cromwell Updates #186

merged 2 commits into from
Mar 17, 2022

Conversation

spatel-gfb
Copy link
Contributor

Description of changes:

With this PR we are enabling FSx for Lustre deployment and integrating it with cromwell.
High level Changes:
gwfcore-root.template.yaml : Added parameters and resources for handling FSx creation. It also needs the Security Group handling for FSx and Batch.
gwfcore-fsx.template.yaml: the main FSx template which can create SSD/HDD/Scratch type of FSx.
gwfcore-launch-template.template.yaml: handled the bootstraping so that FSx is mounted on the Batch EC2 instances.
gwfcore-batch.template.yaml: Handling FSx Subnets
cromwell-resources.template.yaml: handling mount of FSx and also cromwell jar setup updates.
ecs-additions : This code was updated to correctly handle bootstraping and also to collect logs from the bootstrapping into s3.

@henriqueribeiro henriqueribeiro mentioned this pull request Mar 17, 2022
Copy link
Contributor

@markjschreiber markjschreiber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good. There are a few comments where I had some questions.

Copy link
Contributor

@wleepang wleepang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, looks good. Just one comment re: SPOT Bid Percentage

@wleepang wleepang merged commit c00006a into aws-samples:master Mar 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants