Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

Comments

Ecs cluster custom instances#53

Merged
achille-roussel merged 12 commits intosegmentio:masterfrom
Edencia:ecs-cluster-custom-cloud-file
Dec 3, 2016
Merged

Ecs cluster custom instances#53
achille-roussel merged 12 commits intosegmentio:masterfrom
Edencia:ecs-cluster-custom-cloud-file

Conversation

@backnight
Copy link
Contributor

@backnight backnight commented Oct 9, 2016

I did this to be able to push a custom "something" (cloud-init file, script, etc) to the cluster instances.
It is useful for example if you want to do extra configuration of your servers.

Example of use :

// Extra configuration file for initializing cluster instances
data "template_file" "extra_config" {
  template = "${file("${path.module}/extra-setup.yml.tpl")}"

  vars {
    ...
  }
}

// Init cluster
module "stack" {
  [...]
  extra_cloud_config_content = "${data.template_file.extra_config.rendered}"
}

@backnight
Copy link
Contributor Author

You are using an outdated version of Terraform in CircleCI (0.6.16), the current version is 0.7.5. Please change it in order to make tests possible!

@yields
Copy link
Contributor

yields commented Oct 9, 2016

Hey do you mind re-basing master is on latest TF now? also would be nice if you kept terraform fmt out of the way, maybe it should be the last commit, so we know what changed :).

@backnight
Copy link
Contributor Author

backnight commented Oct 9, 2016

Is everything ok? Let me know if you need anything else.

@backnight
Copy link
Contributor Author

About terraform fmt in main.tf file: the only change is the 2 conf vars added at the end.

@backnight
Copy link
Contributor Author

Enjoy!

@backnight
Copy link
Contributor Author

Up ?

@achille-roussel
Copy link
Contributor

Looks good to me 👍 Thanks for your contribution.

@achille-roussel achille-roussel merged commit c5fd758 into segmentio:master Dec 3, 2016
@backnight backnight deleted the ecs-cluster-custom-cloud-file branch December 11, 2016 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants