fix: provide default for var.location to improve backwards compatibility of ADC change.

[kevcube]

Currently, the logic was to use var.function_location if var.location was unset. But because var.location has no default, the var.location is unset scenario is impossible; terraform will ask for it to be set if this is the root module, and will error if this is a child module of the active configuration.

ref: terraform-google-modules/terraform-example-foundation#1458

[kevcube]

@anshukaira this is related to #194

[krprabhat-eng]

Can we somehow mark function_location as deprecated & promote the use of location. If we make location as optional variable, it will show under advanced fields in ADC. As we created a new major version for this breaking change, it should not impact modules consuming the old version.

[anshukaira]

Can we somehow mark function_location as deprecated & promote the use of location. If we make location as optional variable, it will show under advanced fields in ADC. As we created a new major version for this breaking change, it should not impact modules consuming the old version.

For now, i think it should be fine to add this, as this will be release as 0.8.1 i think.
ADC has a version fro 0.8.0 so i dont think this should have any impact.
I will discuss this once internally, what should we do in this case and get back here. For now, PR is LGTMed from my end

[github-actions]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

[kevcube]

Not staleOn Apr 14, 2026, at 19:17, github-actions[bot] ***@***.***> wrote:github-actions[bot] left a comment (GoogleCloudPlatform/terraform-google-cloud-functions#213) This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[kevcube]

@krprabhat-eng @anshukaira can this merge?

[github-actions]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days