-
Notifications
You must be signed in to change notification settings - Fork 626
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
breaking changes for 4.x #1646
Comments
Seems
|
thanks @Nmishin - added. |
1 addition: |
thanks, added. |
Another addition that needs |
thanks, updated |
It seems that |
@andrewkrain please open a new issue following the template provided for the resources you're having issues with so we can address those specifically and just link back to this larger tracking ticket. |
Seems like |
thanks @Cyb3r-Jak3; added. |
More at cloudflare/terraform-provider-cloudflare#1767. The `account_id` argument is currently optional, but has been made mandatory in preparation for terraform-provider-cloudflare v4.x.x, more at cloudflare/terraform-provider-cloudflare#1646.
This functionality has been released in v4.0.0 of the Terraform Cloudflare Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
Tracking for all the planned cleanup in 4.x
Internal cleanup
UsingAccount
in favour of explicit account ID parameters. (provider: remove account_id global configuration option #2139)userBaseUrl
.client.AccountID
. (remove internal reliance onclient.AccountID
#2154)Schema updates
cloudflare_byo_ip_prefix
prefix_id
should be internalid
, not a custom attribute.Deletions/modifications
cloudflare_spectrum_application
needs to support a list of config options (not just IP list) inedge_ips
(resource/cloudflare_spectrum_application: allow nesting ofedge_ips
#2219)cloudflare_zone
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_load_balancer_pool
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_load_balancer_monitor
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_account_member
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_workers_kv_namespace
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_workers_kv
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_workers_script
to swapaccount_id
toRequired: true
(from optional) (enforceaccount_id
for resources that previously depended on globalaccount_id
#2153)cloudflare_access_rule
(resource/cloudflare_access_rule: remove ability to create user level rules #2157)cloudflare_load_balancer
(resource/cloudflare_access_rule: remove ability to create user level rules #2157)cloudflare_load_balancer_monitor
(resource/cloudflare_access_rule: remove ability to create user level rules #2157)cloudflare_load_balancer_pool
(resource/cloudflare_access_rule: remove ability to create user level rules #2157)Additions
cloudflare_zone
needs to supportaccount_id
(account_id
support for members and zone #1767)cloudflare_account_member
needs to supportaccount_id
(account_id
support for members and zone #1767)cloudflare_access_rule
needs to supportaccount_id
(Access rulesaccount_id
support #1790)cloudflare_load_balancer_pool
needs to supportaccount_id
(resource/cloudflare_load_balancer_{monitor,pool}: add support for explicitaccount_id
s #1986)cloudflare_load_balancer_monitor
needs to supportaccount_id
(resource/cloudflare_load_balancer_{monitor,pool}: add support for explicitaccount_id
s #1986)cloudflare_workers_kv
needs to supportaccount_id
(Bump github.com/cloudflare/cloudflare-go from 0.54.0 to 0.55.0 #2049)cloudflare_workers_kv_namespace
needs to supportaccount_id
(Bump github.com/cloudflare/cloudflare-go from 0.54.0 to 0.55.0 #2049)cloudflare_worker_script
needs to supportaccount_id
(Bump github.com/cloudflare/cloudflare-go from 0.56.0 to 0.57.0 #2102)only supported at the zone level for nowcloudflare_worker_route
needs to supportaccount_id
cloudflare_worker_cron_trigger
needs to supportaccount_id
Deprecations
cloudflare_argo_tunnel
=>cloudflare_tunnel
(resource/cloudflare_argo_tunnel: rename tocloudflare_tunnel
#2135)cloudflare_access_bookmark
resource in favour of configuration oncloudflare_access_application
(resource/cloudflare_access_bookmark: remove resource #2136).cloudflare_waf_group
(resource) (remove legacy WAF resources and data sources #2138)cloudflare_waf_override
(resource) (remove legacy WAF resources and data sources #2138)cloudflare_waf_package
(resource) (remove legacy WAF resources and data sources #2138)cloudflare_waf_rule
(resource) (remove legacy WAF resources and data sources #2138)cloudflare_waf_groups
(datasource) (remove legacy WAF resources and data sources #2138)cloudflare_waf_packages
(datasource) (remove legacy WAF resources and data sources #2138)cloudflare_waf_rules
(datasource) (remove legacy WAF resources and data sources #2138)cloudflare_ip_list
(replaced bycloudflare_list
) (resource/cloudflare_ip_list: removed in favour ofcloudflare_list
#2137)The text was updated successfully, but these errors were encountered: