Skip to content

Commit

Permalink
feat: added IMDSv2 as default for EC2 instances (#63)
Browse files Browse the repository at this point in the history
## Description
<!--- Describe your changes in detail -->

## Motivation and Context
<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here. -->

## Breaking Changes
<!-- Does this break backwards compatibility with the current major
version? -->
<!-- If so, please provide an explanation why it is necessary. -->

## How Has This Been Tested?
- [ ] I have updated at least one of the `examples/*` to demonstrate and
validate my change(s)
- [ ] I have tested and validated these changes using one or more of the
provided `examples/*` projects
<!--- Users should start with an existing example as its written, deploy
it, then check their changes against it -->
<!--- This will highlight breaking/disruptive changes. Once you have
checked, deploy your changes to verify -->
<!--- Please describe how you tested your changes -->
- [ ] I have executed `pre-commit run -a` on my pull request
<!--- Please see
https://github.com/antonbabenko/pre-commit-terraform#how-to-install for
how to install -->
  • Loading branch information
applike-ss authored Jun 5, 2024
1 parent f145cc3 commit df309a8
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 12 deletions.
14 changes: 7 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,18 +7,18 @@ Terraform module which creates an spotinst ocean ecs cluster
| Name | Version |
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3.0 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 4.67 |
| <a name="requirement_spotinst"></a> [spotinst](#requirement\_spotinst) | >= 1.87.1 |
| <a name="requirement_tls"></a> [tls](#requirement\_tls) | >= 4.0.4 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.52.0 |
| <a name="requirement_spotinst"></a> [spotinst](#requirement\_spotinst) | >= 1.175.0 |
| <a name="requirement_tls"></a> [tls](#requirement\_tls) | >= 4.0.5 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 4.67 |
| <a name="provider_aws.owner"></a> [aws.owner](#provider\_aws.owner) | >= 4.67 |
| <a name="provider_spotinst"></a> [spotinst](#provider\_spotinst) | >= 1.87.1 |
| <a name="provider_tls"></a> [tls](#provider\_tls) | >= 4.0.4 |
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.52.0 |
| <a name="provider_aws.owner"></a> [aws.owner](#provider\_aws.owner) | >= 5.52.0 |
| <a name="provider_spotinst"></a> [spotinst](#provider\_spotinst) | >= 1.175.0 |
| <a name="provider_tls"></a> [tls](#provider\_tls) | >= 4.0.5 |

## Modules

Expand Down
4 changes: 2 additions & 2 deletions examples/cluster/versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@ terraform {
required_providers {
aws = {
source = "hashicorp/aws"
version = ">= 4.67"
version = ">= 5.52.0"
configuration_aliases = [
aws.owner,
]
}

spotinst = {
source = "spotinst/spotinst"
version = "1.140.0"
version = "1.175.0"
}
}

Expand Down
5 changes: 5 additions & 0 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,11 @@ EOF
draining_timeout = 300
ebs_optimized = true

instance_metadata_options {
http_tokens = "required"
http_put_response_hop_limit = 2
}

autoscaler {
is_enabled = true
is_auto_config = local.is_auto_config
Expand Down
6 changes: 3 additions & 3 deletions versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,15 @@ terraform {
configuration_aliases = [
aws.owner,
]
version = ">= 4.67"
version = ">= 5.52.0"
}
spotinst = {
source = "spotinst/spotinst"
version = ">= 1.87.1"
version = ">= 1.175.0"
}
tls = {
source = "hashicorp/tls"
version = ">= 4.0.4"
version = ">= 4.0.5"
}
}
required_version = ">= 1.3.0"
Expand Down

0 comments on commit df309a8

Please sign in to comment.