Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

Make plugin retries configurable through input parameters#40

Merged
tbroden84 merged 6 commits intomainfrom
plugins-retries
Dec 12, 2023
Merged

Make plugin retries configurable through input parameters#40
tbroden84 merged 6 commits intomainfrom
plugins-retries

Conversation

@tbroden84
Copy link
Copy Markdown
Contributor

@tbroden84 tbroden84 commented Nov 24, 2023

WHY are these changes introduced?

Increased reports about Error: ReadWithRetry failed, status: 400, message: map[error:Timeout talking to backend] when reading plugins information. Current retries (5 attempts and increased sleep) is not sufficient.

Trello

WHAT is this pull request doing?

  • Add sleep/timeout input parameters
  • Update how retries works for both official and community plugins
  • Clean up code

HOW can this pull request be tested?

Run together with our CloudAMQP Terraform provider.

@tbroden84 tbroden84 merged commit 783d2b1 into main Dec 12, 2023
@tbroden84 tbroden84 deleted the plugins-retries branch December 12, 2023 09:44
tbroden84 added a commit that referenced this pull request Dec 12, 2023
### WHY are these changes introduced?

Increased reports about `Error: ReadWithRetry failed, status: 400, message: map[error:Timeout talking to backend]` when reading plugins information. Current retries (5 attempts and increased sleep) is not sufficient.

### WHAT is this pull request doing?

- Add sleep/timeout input parameters
- Update how retries works for both official and community plugins
- Clean up code

### HOW can this pull request be tested?

Run together with our CloudAMQP Terraform provider.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant