read deprecation message for resource deprecations#38135
read deprecation message for resource deprecations#38135DanielMSchmidt wants to merge 4 commits intomainfrom
Conversation
dbanck
left a comment
There was a problem hiding this comment.
I think we should also include the new deprecation message in the provider schema JSON so that other consumers, such as language servers, can display it. This should be relatively easy to do by updating:
terraform/internal/command/jsonprovider/attribute.go
Lines 13 to 24 in c8a05ab
c8a05ab to
1a39809
Compare
I agree – the only remark I'd have before we do that, would be to check this with large providers like AWS / Azure / Google. As they might have lots of these and this might cause their schema size to explode even further. I would at least like to test this locally to see how much that schema increases in size to get some rough percentage. |
|
For the Terraform AWS Provider: For Terraform Plugin SDKv2:% grep 'Deprecated: ' internal/service/*/*.go | wc -l
72For Terraform Plugin Framework:% grep DeprecationMessage internal/service/*/*.go | wc -l
23so, less than 100 deprecation messages so far and each message is usually less than 100 characters (often something like status is deprecated. Use state instead.). A tiny fraction of the overall provider schema JSON size. |

Fixes #
Target Release
1.15.x
Rollback Plan
Changes to Security Controls
Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.
CHANGELOG entry