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

minimal terraform templates #1

Merged
merged 1 commit into from Jul 17, 2018

Conversation

Projects
None yet
2 participants
@spohnan
Contributor

spohnan commented Jul 13, 2018

builds vpc, load balancer, ecs cluster with a single preloaded container and an optional bastion server

@jwilcoxen45

This comment has been minimized.

Contributor

jwilcoxen45 commented Jul 13, 2018

Noted on this request! Thank you! I will review thorough on Monday morning.

At the end of the deployment the following resources will have been provisioned.
* VPC with two public and two private subnets
* An optional bastion host which allows for console access to resources in private subnets

This comment has been minimized.

@jwilcoxen45

jwilcoxen45 Jul 16, 2018

Contributor

Nice use of bastion setup!

EC2 -> Load Balancers link in the left-hand menu -> Select the ALB and copy the DNS hostname to access Nginx
![architecture](diagrams/architecture.png)

This comment has been minimized.

@jwilcoxen45

jwilcoxen45 Jul 16, 2018

Contributor

Nice started diagrams.

// ~~ Nginx container ~~~
data "template_file" "nginx_task_definition" {

This comment has been minimized.

@jwilcoxen45

jwilcoxen45 Jul 16, 2018

Contributor

Is this intended to be replaced with Java/spring container? Or stuff java app within container? Not picking up the reason for using nginx container.

This comment has been minimized.

@spohnan

spohnan Jul 17, 2018

Contributor

yes, nginx is just a placeholder that was used to verify that the networking and security groups were set up correctly. The application load balancer should be capable of whatever proxy capabilities will be needed and this initial container will be replaced with Jenkins, Tomcat etc. in future commits.

This comment has been minimized.

@jwilcoxen45

jwilcoxen45 Jul 17, 2018

Contributor

Alright, then we will need an ECR too to push images. I am sure that is in your thought process too.

@jwilcoxen45 jwilcoxen45 merged commit 0f5ea51 into ICFI:master Jul 17, 2018

tbadaczewski pushed a commit that referenced this pull request Jul 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment