Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

module: Add EBS extra volume support #45

Merged
merged 2 commits into from
Feb 6, 2019
Merged

module: Add EBS extra volume support #45

merged 2 commits into from
Feb 6, 2019

Conversation

jieyu
Copy link
Contributor

@jieyu jieyu commented Feb 5, 2019

Allow extra EBS volumes to be created for each AWS instance.

Copy link
Contributor

@fatz fatz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Allow extra EBS volumes to be created for each AWS instance.
@@ -121,6 +121,7 @@ module "dcos-infrastructure" {
private_agents_os = "${var.private_agents_os}"
private_agents_root_volume_size = "${var.private_agents_root_volume_size}"
private_agents_root_volume_type = "${var.private_agents_root_volume_type}"
private_agents_extra_volumes = ["${var.private_agents_extra_volumes}"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

with this we need to have a minimum ref to the next version of instance

@fatz
Copy link
Contributor

fatz commented Feb 6, 2019

min version of infra needs to be raised as mentioned in dcos-terraform-terraform-aws-infrastructure#20 next patch num is safe to use

This is because new variables are introduced.
@jieyu
Copy link
Contributor Author

jieyu commented Feb 6, 2019

@fatz PTAL again. I bump the dependency version

Copy link
Contributor

@fatz fatz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ci failed cause of a missing description update which is available in master. Approving and merging.

@fatz fatz merged commit 7c2d807 into master Feb 6, 2019
@Rohithzr
Copy link

Rohithzr commented May 8, 2019

have a question: How to use this?

@fatz
Copy link
Contributor

fatz commented May 8, 2019

@Rohithzr you can take a look into the instance module https://github.com/dcos-terraform/terraform-aws-instance#example

@Rohithzr
Copy link

@fatz thanks, I have one question though, does this method also mounts those volumes to the instance?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants