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

Change several defaults to null #74

Closed
wants to merge 1 commit into from
Closed

Change several defaults to null #74

wants to merge 1 commit into from

Conversation

yod527
Copy link

@yod527 yod527 commented Aug 26, 2020

Changed default to null, of next vars:

  • subnet_ids
  • availability_zones

Meet exact the same issue as in consul: hashicorp/terraform-aws-consul#181

Changed default to null, of next vars:
 - subnet_ids
 - availability_zones

Meet exact the same issue as in consul: hashicorp/terraform-aws-consul#181
@yod527 yod527 requested a review from robmorgan as a code owner August 26, 2020 11:05
@hashicorp-cla
Copy link

hashicorp-cla commented Aug 26, 2020

CLA assistant check
All committers have signed the CLA.

@yardbirdsax
Copy link

Just wanted to note that to get this fully working for me I also had to update references to the terraform-aws-consul module to point to the fixed version. You can see those changes at this commit.

@brikis98
Copy link
Collaborator

Thanks for the PR! I believe these same changes were done in #73 and merged already, so I'm going to close this one. Give https://github.com/hashicorp/terraform-aws-nomad/releases/tag/v0.6.5 a shot.

@brikis98 brikis98 closed this Aug 31, 2020
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.

4 participants