fix: remove retry in read as retry for create/update is already handled in respective methods

[gurusai-voleti]

remove retry in read as retry for create/update is already handled in respective methods

Fixes: https://github.com/hashicorp/terraform-provider-google/issues/13464

Release Note Template for Downstream PRs (will be copied)

storage: removed retry on 404s during refresh for `google_storage_bucket`, preventing hanging when refreshing deleted buckets

[github-actions[bot]]

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

@rileykarson, 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.

[rileykarson]

What bug(s) is this addressing / what's the context?

[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, 1 insertion(+), 10 deletions(-)) google-beta provider: Diff ( 1 file changed, 1 insertion(+), 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

[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, 1 insertion(+), 10 deletions(-)) google-beta provider: Diff ( 1 file changed, 1 insertion(+), 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

[github-actions[bot]]

@rileykarson This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.

[kautikdk]

Hi @gurusai-voleti, As per the changes, we are planning to remove retry logic for read operations. Are we ensured that it is matching storage buckets SLOs? If not can we consider shorter retry time?

[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, 1 insertion(+), 10 deletions(-)) google-beta provider: Diff ( 1 file changed, 1 insertion(+), 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