Closed kevinkirkup closed 4 years ago
Hi @kevinkirkup,
Thanks for bringing this up.
This is the expected behavior when using the old SDK with the new provider protocol. In the future there will be a new SDK that can make full use of the new plugin protocol.
What is happening is that when a provider gets the proposed new state to plan, the only fields it can technically replace must be marked as Computed
(defaults are essentially a computed field that is filled in during plan). However that doesn't align with how the old SDK worked, and we need to make allowances for that in core by only logging a warning.
These warnings have still proven extremely useful in diagnosing provider issues, since they often highlight where providers are returning the incorrect values which cause problems later on. Unfortunately in the case of Default
settings, Terraform has no visibility into that and can't distinguish them from any other unexpected value.
I'll leave this open for now to make it easier for others to try and locate information about the warnings.
Going to close this issue as there isn't anything actionable left and folks should still get search hits for closed issues anyway.
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
Terraform Version
Terraform Configuration Files
N/A
Debug Output
In the schema, the arguments are set as
Optional
with aDefault
:Terraform reports a warning:
Crash Output
Expected Behavior
This should have been fine.
Actual Behavior
Terraform reports a warning
Steps to Reproduce
Run the Acceptance Tests for Pureport Provider.
Additional Context
References
None that I could find.