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

Provider-aws latest release has a conflict with consul-cluster module #181

Closed
rafaelremondes opened this issue Jul 31, 2020 · 4 comments
Closed

Comments

@rafaelremondes
Copy link
Contributor

rafaelremondes commented Jul 31, 2020

In the last version of the aws provider, v3.0.0, following conflicts appear for resource resource/aws_autoscaling_group when running a terraform plan

Error: "availability_zones": conflicts with vpc_zone_identifier

  on .terraform/modules/consul_cluster/terraform-aws-consul-0.7.6/modules/consul-cluster/main.tf line 14, in resource "aws_autoscaling_group" "autoscaling_group":
  14: resource "aws_autoscaling_group" "autoscaling_group" {



Error: "vpc_zone_identifier": conflicts with availability_zones

  on .terraform/modules/consul_cluster/terraform-aws-consul-0.7.6/modules/consul-cluster/main.tf line 14, in resource "aws_autoscaling_group" "autoscaling_group":
  14: resource "aws_autoscaling_group" "autoscaling_group" {

Related to this change:
hashicorp/terraform-provider-aws#12927

Only one of the two parameters(vpc_zone_identifier, availability_zones) should be passed right?

@rafaelremondes rafaelremondes changed the title New provider aws release has conflict with this module Provider-aws latest release has a conflict with consult-cluster modules Jul 31, 2020
@rafaelremondes rafaelremondes changed the title Provider-aws latest release has a conflict with consult-cluster modules Provider-aws latest release has a conflict with consult-cluster module Jul 31, 2020
@brikis98
Copy link
Collaborator

brikis98 commented Aug 1, 2020

3.0.0 just came out, with some backwards incompatible changes, and we have not yet tested/upgraded this repo for it. As a short term workaround, you can pin your AWS provider to 2.x.x.

Only one of the two parameters(vpc_zone_identifier, availability_zones) should be passed right?

I think we want to pass both, but we probably need to change the default for our corresponding input variables in consul-cluster to be null. Would you be up for a PR that makes that change?

@rafaelremondes
Copy link
Contributor Author

Hey @brikis98

Yes, I pinned the provider-aws version and it worked.

Sure, I'll test it and open a PR. Thanks for the help

@rafaelremondes rafaelremondes changed the title Provider-aws latest release has a conflict with consult-cluster module Provider-aws latest release has a conflict with consul-cluster module Aug 1, 2020
@andrew-womeldorf
Copy link
Contributor

🙌 Thank you @rafaelremondes @brikis98 !

@brikis98
Copy link
Collaborator

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants