-
Notifications
You must be signed in to change notification settings - Fork 4.9k
azurerm_private_endpoint: Retry on "RetryableError" and "StorageAccountOperationInProgress" errors during LRO
#28112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ountOperationInProgress" errors during LRO
|
This PR is being labeled as "stale" because it has not been updated for 30 or more days. If this PR is still valid, please remove the "stale" label. If this PR is blocked, please add it to the "Blocked" milestone. If you need some help completing this PR, please leave a comment letting us know. Thank you! |
|
This PR is being labeled as "stale" because it has not been updated for 30 or more days. If this PR is still valid, please remove the "stale" label. If this PR is blocked, please add it to the "Blocked" milestone. If you need some help completing this PR, please leave a comment letting us know. Thank you! |
|
This PR is being labeled as "stale" because it has not been updated for 30 or more days. If this PR is still valid, please remove the "stale" label. If this PR is blocked, please add it to the "Blocked" milestone. If you need some help completing this PR, please leave a comment letting us know. Thank you! |
|
This PR is being labeled as "stale" because it has not been updated for 30 or more days. If this PR is still valid, please remove the "stale" label. If this PR is blocked, please add it to the "Blocked" milestone. If you need some help completing this PR, please leave a comment letting us know. Thank you! |
|
This is still an issue, any ideas when this request will be resolved? |
|
Any updates on this? I really would like to see this fixed |
|
This PR is being labeled as "stale" because it has not been updated for 30 or more days. If this PR is still valid, please remove the "stale" label. If this PR is blocked, please add it to the "Blocked" milestone. If you need some help completing this PR, please leave a comment letting us know. Thank you! |
|
@magodo it is not stale. would you be so kind to provide an update? |
|
Kindly ping @jackofallops |
|
any updates? |
|
This PR got updated per the change on the underlying SDK at: hashicorp/go-azure-sdk#1221. Now it depends on another PR: hashicorp/go-azure-sdk#1224. |
azurerm_private_endpoint: Retry on "RetryableError" and "StorageAccountOperationInProgress" errors during LROazurerm_private_endpoint: Retry on "RetryableError" and "StorageAccountOperationInProgress" errors during LRO
|
@jackofallops Given #30254 is merged, this PR is now ready for review. |
wyattfry
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One very minor error message adjustment IMO. Otherwise looks good to go.
| } | ||
| } | ||
| return &pluginsdk.RetryError{ | ||
| Err: fmt.Errorf("waiting the creation of %s: %+v", id, err), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could the error message stay the same? Or did you mean "waiting for"?
| Err: fmt.Errorf("waiting the creation of %s: %+v", id, err), | |
| Err: fmt.Errorf("creating %s: %+v", id, err), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wyattfry Yes, I meant "waiting for", to differ from the error message for L359:
result, err := client.CreateOrUpdate(ctx, id, parameters)
if err != nil {
return &pluginsdk.RetryError{
Err: fmt.Errorf("creating %s: %+v", id, err),
Retryable: false,
}
}
wyattfry
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Community Note
Description
This PR makes
azurerm_private_endpointretry on "RetryableError" and "StorageAccountOperationInProgress" errors during LRO.Depends on hashicorp/go-azure-sdk#1124
PR Checklist
For example: “
resource_name_here- description of change e.g. adding propertynew_property_name_here”Changes to existing Resource / Data Source
Testing
Note about the [WARN] logs above.
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
azurerm_private_endpoint: Retry on "RetryableError" and "StorageAccountOperationInProgress" errors during LRO [GH-00000]This is a (please select all that apply):
Related Issue(s)
Fixes #21293
Note
If this PR changes meaningfully during the course of review please update the title and description as required.