terraform-aws-openshift icon indicating copy to clipboard operation
terraform-aws-openshift copied to clipboard

Install to an existing VPC

Open mathus13 opened this issue 6 years ago • 5 comments

Thank you for this. I am new to openshift/k8, and still pretty new to AWS. Using terraform to bootstrap this is so helpful.

I have a need to create the openshift cluster in a pre-existing vpc and subnet. I am wondering what the best method is to bypass the vpc creation and pass the vpc and subnet ids as a variable

mathus13 avatar Jan 11 '19 17:01 mathus13

Hi @mathus13 it will not be possible to add to an existing VPC without some modification to the Terraform code I'm afraid! If you are struggling to do it let me know and I will try to find some time to help.

dwmkerr avatar Jan 14 '19 10:01 dwmkerr

Thank you. I figured out how to do what I needed. Do you think it would be helpful to implement a VPC and/or Subnet variable using if statements? If so, I would be happy to submit a PR

mathus13 avatar Jan 14 '19 15:01 mathus13

Yes that would be awesome!

On Mon, 14 Jan 2019 at 3:21 PM, Shawn Barratt [email protected] wrote:

Thank you. I figured out how to do what I needed. Do you think it would be helpful to implement a VPC and/or Subnet variable using if statements? If so, I would be happy to submit a PR

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/dwmkerr/terraform-aws-openshift/issues/74#issuecomment-454042157, or mute the thread https://github.com/notifications/unsubscribe-auth/AB1nSFZjGfQBQbh41SUvLXD8g4aXL9Ehks5vDKCIgaJpZM4Z75R7 .

dwmkerr avatar Jan 15 '19 20:01 dwmkerr

@mathus13 any news about this issue?

arashkaffamanesh avatar Feb 12 '19 22:02 arashkaffamanesh

@arashkaffamanesh Time has been tight to write any kind of conditional, but I was able to make it work by emptying step 7 and replacing references to the created VPC and Subnet ids with passed variables.

I would consider this a closed issue, and if I can find time to write functionality for this I will post a PR.

mathus13 avatar Feb 20 '19 16:02 mathus13