fix: derive the project attribute from resource or compute api or fallback to provider

[gurusai-voleti]

Fixes hashicorp/terraform-provider-google#17165

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

storage: derive the project attribute from resource or compute api or fallback to provider in `resource_storage_bucket`

[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.

google provider: Diff ( 1 file changed, 7 insertions(+), 9 deletions(-))
google-beta provider: Diff ( 1 file changed, 7 insertions(+), 9 deletions(-))

[modular-magician]

Tests analytics

Total tests: 109
Passed tests: 100
Skipped tests: 9
Affected tests: 0

Click here to see the affected service packages

• storage

🟢 All tests passed!

View the build log

[github-actions]

Hello! I am a robot. Tests will require approval from a repository maintainer to run.

@SirGitsalot, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look.

You can help make sure that review is quick by doing a self-review and by running impacted tests locally.

[SirGitsalot]

LGTM for the code change, but please update the docs too:

https://github.com/GoogleCloudPlatform/magic-modules/blob/main/mmv1/third_party/terraform/website/docs/d/storage_bucket.html.markdown

The "If no value is supplied in the configuration or through provider defaults then the data source will use the Compute API..." is not accurate anymore

[gurusai-voleti]

LGTM for the code change, but please update the docs too:

https://github.com/GoogleCloudPlatform/magic-modules/blob/main/mmv1/third_party/terraform/website/docs/d/storage_bucket.html.markdown

The "If no value is supplied in the configuration or through provider defaults then the data source will use the Compute API..." is not accurate anymore

Updated documentation

[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.

google provider: Diff ( 2 files changed, 8 insertions(+), 10 deletions(-))
google-beta provider: Diff ( 2 files changed, 8 insertions(+), 10 deletions(-))

[modular-magician]

Tests analytics

Total tests: 109
Passed tests: 100
Skipped tests: 9
Affected tests: 0

Click here to see the affected service packages

• storage

🟢 All tests passed!

View the build log