-
Notifications
You must be signed in to change notification settings - Fork 78
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
Bump Terraform provider version to v3.45.0 #392
Conversation
/test-examples="examples/storage/managementpolicy.yaml" |
/test-examples="examples/databricks/accessconnector.yaml" |
/test-examples="examples/storage/managementpolicy.yaml" |
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
1 similar comment
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
/test-examples="examples/operationalinsights/workspace.yaml" |
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
70ea6ec
to
d4bb7c2
Compare
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
1 similar comment
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
/test-examples="examples/securityinsights/sentinelloganalyticsworkspaceonboarding.yaml" |
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
d4bb7c2
to
94c294a
Compare
/test-examples="examples/securityinsights/sentinelloganalyticsworkspaceonboarding.yaml" |
1 similar comment
/test-examples="examples/securityinsights/sentinelloganalyticsworkspaceonboarding.yaml" |
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
94c294a
to
0ab52ac
Compare
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
1 similar comment
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
0ab52ac
to
4975e46
Compare
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
1 similar comment
/test-examples="examples/kusto/clustermanagedprivateendpoint.yaml" |
4975e46
to
102f92b
Compare
102f92b
to
5a4e483
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @svscheg LGTM!
Description of your changes
Bumps Terraform provider version to v3.45.0
Fixes #391
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
crddiff
reports breaking changes in the following resources:state schema version
reports breaking changes in the following resources:kusto.ClusterManagedPrivateEndpoint -> /~https://github.com/upbound/provider-azure/actions/runs/4331215148
Schema changes fix: changing the external-name syntax - will need to make the
C
lower case in theClusters
path component in this example ('Cluster' -> 'cluster')operationalinsights.Workspace -> /~https://github.com/upbound/provider-azure/actions/runs/4312609967
Manual intervention for
state schema version
changes:`manual-intervention` with issue - /~https://github.com/upbound/official-providers/issues/629 Schema changes fix: changing the external-name syntax - will need to make the `R` lower case in the `Redis ` path component in this example ('Redis' -> 'redis')
Resource failed with existing bug:
manual-intervention
- /~https://github.com/upbound/official-providers/issues/629Schema changes fix: changing the external-name syntax - will need to make the
R
lower case in theRedis
path component in this example ('Redis' -> 'redis')Resource failed with existing bug:
manual-intervention
- azure.cache:RedisLinkedServer
creation fails with StatusCode=400 #16Schema changes fix: changing the external-name syntax - will need to make the
R
lower case in theRedis
path component in this example ('Redis' -> 'redis')Also fixed the external name, will use next configuration: confi.TemplatedStringAsIdentifier("", "/subscriptions/{{ .setup.configuration.subscription_id }}/resourceGroups/{{ .parameters.resource_group_name }}/providers/Microsoft.Cache/redis/{{ .parameters.target_redis_cache_name }}/linkedServers/{{ .parameters.linked_redis_cache_name }}"),