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

Error: "enable_flow_logs": [REMOVED] This field is being removed in favor of log_config. If log_config is present, flow logs are enabled. Please remove this field #45

Closed
captain-kark opened this issue Dec 3, 2019 · 5 comments

Comments

@captain-kark
Copy link

Version 3.0.0 of the google cloud provider is removing this option.

https://www.terraform.io/docs/providers/google/guides/version_3_upgrade.html#resource-google_compute_subnetwork

@autero1
Copy link
Contributor

autero1 commented Dec 3, 2019

Thanks for noting this @captain-kark . PR welcome!

@captain-kark
Copy link
Author

I can start working on a couple of branches, I think it'll be worth it for me to move to 3.0.0 for everything I'm doing right now anyway.

What do you think about a couple of long running branches, for 2.x and 3.x?

@autero1
Copy link
Contributor

autero1 commented Dec 4, 2019

Hi @captain-kark , would be great if can contribute! I don't think we need to create long-running branches - instead we just pin the providers in the examples to > 3.x.

@avielb
Copy link

avielb commented Mar 9, 2020

please approve this

@robmorgan
Copy link
Contributor

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

No branches or pull requests

4 participants