V5 upgrade docs updated for Default Provider Values and Data Source 404 erroring

[NickElliot]

Release Note Template for Downstream PRs (will be copied)

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 1 file changed, 8 insertions(+))
Terraform Beta: Diff ( 1 file changed, 8 insertions(+))

[c2thorn]

Suggested change

`project`, `region`, and `zone` fields will now display their values during plan-time instead of defaulting to the unknown `(known after apply)` value normally displayed for `computed` fields. These values will be taken from either the Terraform resource config file, provider config, or local environment variables, depending on which variables are supplied by the user, matching the existing per-resource functionality for what default values are used in execution of a Terraform plan.

`project`, `region`, and `zone` fields will now display their values during plan-time instead of defaulting to the unknown `(known after apply)` value normally displayed for [Computed](https://developer.hashicorp.com/terraform/plugin/sdkv2/schemas/schema-behaviors#computed) fields. These values will be taken from either the Terraform resource config file, provider config, or local environment variables, depending on which variables are supplied by the user, matching the existing per-resource functionality for what default values are used in execution of a Terraform plan.

Not sure if adding a link to the docs for what "Computed" means will be more or less confusing for users. I'll defer to your opinion.

[NickElliot]

I'm not sure the level of expected technical specification in here -- if erring on the side of basic terminology, I'll rewrite it to describe it along the lines of "fields without fixed default values"

[c2thorn]

That sounds even better. Thanks!

[NickElliot]

The new wording is in, sounds good?

[c2thorn]

added another suggestion on second thought

[c2thorn]

Suggested change

`project`, `region`, and `zone` fields will now display their values during plan-time instead of the placeholder `(known after apply)` value normally displayed for fields without fixed default values. These values will be taken from either the Terraform resource config file, provider config, or local environment variables, depending on which variables are supplied by the user, matching the existing per-resource functionality for what default values are used in execution of a Terraform plan.

`project`, `region`, and `zone` fields will now display their values during plan-time instead of the placeholder `(known after apply)` value normally displayed for fields without provider default values. These values will be taken from either the Terraform resource config file, provider config, or local environment variables, depending on which variables are supplied by the user, matching the existing per-resource functionality for what default values are used in execution of a Terraform plan.

I understand fixed != API value here, but might as well be specific?

[NickElliot]

I think the problem is it conflicts with the other usage of provider as a term -- since it does use the "provder default" value, as in what is configured in the user's provider config. I thought about saying "universal default", "hard-coded default", or similar to mean "regardless of local configuration", but wasn't sure if that became more or less clear than "fixed" on its own

[c2thorn]

That makes sense. How about "fixed Terraform default values" ? If not, I'm fine with just "fixed". Users that care will probably already understand from the (known after apply) part.

[NickElliot]

That sounds good to me!

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 1 file changed, 8 insertions(+))
Terraform Beta: Diff ( 1 file changed, 8 insertions(+))

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 1 file changed, 8 insertions(+))
Terraform Beta: Diff ( 1 file changed, 8 insertions(+))