Skip to content

Commit

Permalink
Merge pull request #18 from byu-oit/t3-family
Browse files Browse the repository at this point in the history
Allow different instance types, change default to t3
  • Loading branch information
stuft2 authored Jun 21, 2021
2 parents f3c37e7 + 438a7b3 commit 47d052c
Showing 3 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -18,7 +18,7 @@ provider "aws" {
}
module "bastion" {
source = "github.com/byu-oit/terraform-aws-bastion.git?ref=v2.0.1"
source = "github.com/byu-oit/terraform-aws-bastion.git?ref=v2.1.0"
env = "prd"
vpc_vpn_to_campus = true
netid = "mynetid"
2 changes: 1 addition & 1 deletion main.tf
Original file line number Diff line number Diff line change
@@ -25,7 +25,7 @@ data "aws_ssm_parameter" "ami" {

resource "aws_instance" "bastion" {
ami = data.aws_ssm_parameter.ami.value
instance_type = "t2.micro"
instance_type = var.instance_type
key_name = aws_key_pair.key.key_name
subnet_id = module.acs["${var.subnet_type}_subnet_ids"][0]
vpc_security_group_ids = [aws_security_group.sg.id]
6 changes: 6 additions & 0 deletions variables.tf
Original file line number Diff line number Diff line change
@@ -29,3 +29,9 @@ variable "subnet_type" {
default = "public"
description = "Which subnet type should the bastion launch in? (e.g. public, private, data)"
}

variable "instance_type" {
type = string
default = "t3.micro"
description = "The EC2 instance type to deploy (defaults to t3.micro)"
}

0 comments on commit 47d052c

Please sign in to comment.