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

Add swap space in EC2 instances #131

Merged
merged 3 commits into from
Feb 17, 2021

Conversation

henriqueribeiro
Copy link
Contributor

@henriqueribeiro henriqueribeiro commented Feb 10, 2021

Issue #, if available:
https://github.com/aws-samples/aws-genomics-workflows/issues/124

Description of changes:

  • Add 4GB of swap space in EC2 instances

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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.

Suggest moving the swap provisioning to ecs-additions-common.sh

@@ -103,6 +103,14 @@ Resources:
- unzip

runcmd:
# add 4GB of swap space
Copy link
Contributor

Choose a reason for hiding this comment

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

Prefer to move these lines to ecs-additions-common.sh and keep the LaunchTemplate UserData as minimal as possible. When the GWFCore stack is launched the ecs-additions-* scripts are "installed" in a CodeCommit repo. Updates to the repo trigger redeployment to an artifacts location that referenced by the LaunchTemplate. This enables users to customize provisioning steps without having to teardown and relaunch the GWFCore stack.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Makes total sense. I will update it

@wleepang wleepang self-assigned this Feb 16, 2021
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.

Looks good!

@wleepang wleepang merged commit 0218be8 into aws-samples:master Feb 17, 2021
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.

2 participants