Skip to content

Commit

Permalink
Merge pull request #433 from Capgemini/bastion-deploy-fix
Browse files Browse the repository at this point in the history
Bastion deploy fix
  • Loading branch information
enxebre committed Jul 28, 2015
2 parents d99c204 + e393cc5 commit 4117f06
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 2 additions & 2 deletions bootstrap/aws/config-default.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ export TF_VAR_atlas_artifact_version_master=${TF_VAR_atlas_artifact_version_mast
export TF_VAR_atlas_artifact_version_slave=${TF_VAR_atlas_artifact_version_slave:-6}

export TF_VAR_region=${TF_VAR_region:-eu-west-1}
export TF_VAR_master_size=${TF_VAR_master_size:-m1.medium}
export TF_VAR_slave_size=${TF_VAR_slave_size:-m1.medium}
export TF_VAR_master_size=${TF_VAR_master_size:-m3.medium}
export TF_VAR_slave_size=${TF_VAR_slave_size:-m3.medium}
export TF_VAR_slaves=${TF_VAR_slaves:-1}
export TF_VAR_availability_zones=${TF_VAR_availability_zones:-'eu-west-1a,eu-west-1b,eu-west-1c'}
export TF_VAR_public_subnet_availability_zone=${TF_VAR_public_subnet_availability_zone:-'eu-west-1a'}
Expand Down
1 change: 1 addition & 0 deletions terraform/aws/bastion-server.tf
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ resource "aws_instance" "bastion" {
"echo 1 | sudo tee /proc/sys/net/ipv4/conf/all/forwarding",
/* Install docker */
"curl -sSL https://get.docker.com/ubuntu/ | sudo sh",

This comment has been minimized.

Copy link
@shirkevich

shirkevich Jul 28, 2015

why you install docker here? isn't it already installed in packer?

This comment has been minimized.

Copy link
@tayzlor

tayzlor Jul 28, 2015

Member

The bastion host didn't use the packer build previously since it only has docker and a vpn running on it we just build it off a stock ubuntu AMI in AWS. There's was no need for it to come included with Mesos etc...

Looks like that changed in e0164f3

I'm not sure if thats a good thing, since we dont need all the extra stuff in here

"sudo service docker start",
/* Initialize open vpn data container */
"sudo mkdir -p /etc/openvpn",
"sudo docker run --name ovpn-data -v /etc/openvpn busybox",
Expand Down

0 comments on commit 4117f06

Please sign in to comment.