Skip to content
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

Added ECS_ENABLE_SPOT_INSTANCE_DRAINING to README #2207

Merged
merged 1 commit into from
Sep 17, 2019

Conversation

sparrc
Copy link
Contributor

@sparrc sparrc commented Sep 16, 2019

see #2205

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sparrc sparrc added this to the 1.32.0 milestone Sep 16, 2019
README.md Outdated
@@ -178,6 +178,7 @@ additional details on each available environment variable.
| `ECS_EXCLUDE_UNTRACKED_IMAGE` | `alpine:latest` | Comma seperated list of `imageName:tag` of images that should not be deleted by the ECS agent if `ECS_ENABLE_UNTRACKED_IMAGE_CLEANUP` is enabled. | | |
| `ECS_DISABLE_DOCKER_HEALTH_CHECK` | `false` | Whether to disable the Docker Container health check for the ECS Agent. | `false` | `false` |
| `ECS_NVIDIA_RUNTIME` | nvidia | The Nvidia Runtime to be used to pass Nvidia GPU devices to containers. | nvidia | Not Applicable |
| `ECS_ENABLE_SPOT_INSTANCE_DRAINING` | `true` | If true, if the container instance receives a [spot interruption notice](https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/spot-interruptions.html), agent will set the instance's status to [DRAINING](https://docs.aws.amazon.com/AmazonECS/latest/developerguide/container-instance-draining.html), which gracefully shuts down and replaces all tasks running on the instance. It is recommended that this be set to `true` when using spot instances. | `false` | `false` |
Copy link
Contributor

Choose a reason for hiding this comment

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

*replaces tasks as part of service

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍, modified sentence to indicate that only tasks that are part of a service are replaced

@sparrc sparrc force-pushed the spot-draining-readme branch from f5c7125 to e4d44c7 Compare September 16, 2019 20:05
@sparrc sparrc force-pushed the spot-draining-readme branch from e4d44c7 to 43df207 Compare September 16, 2019 22:44
Copy link
Contributor

@sharanyad sharanyad left a comment

Choose a reason for hiding this comment

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

this will need a changelog update too. But can be a different PR

@sparrc sparrc merged commit a593930 into aws:dev Sep 17, 2019
@sparrc sparrc deleted the spot-draining-readme branch September 17, 2019 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants