SAP / terraform-provider-btp

Terraform provider for SAP BTP
https://registry.terraform.io/providers/SAP/btp/latest
Apache License 2.0
89 stars 18 forks source link

fix: added retry logic for reading resource subaccount entitlement #910

Closed diya-dhan closed 2 months ago

diya-dhan commented 2 months ago

Purpose

Closes #902

Implemented the retry logic in the Read function of the resource subaccount entitlement to overcome the rate limiting issue faced. This is accomplished by placing the GET call within a loop, enabling the function to manage rate limiting errors accordingly.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

go test ./...

What to Check

Verify that the following are valid:

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

lechnerc77 commented 2 months ago

Sonar Cloud Coverage accepted