Skip to content

feat(retries): Use config params for retry parameters - #1467

Merged
JuArce merged 13 commits into
stagingfrom
1401-1414-retry-config-no-jitter
Nov 22, 2024
Merged

feat(retries): Use config params for retry parameters#1467
JuArce merged 13 commits into
stagingfrom
1401-1414-retry-config-no-jitter

Conversation

@PatStiles

@PatStilesPatStiles commented Nov 21, 2024

Copy link
Copy Markdown
Contributor

Use config file for retry parameters

Description

Use config file for retry parameters

Type of change

Please delete options that are not relevant.

  • New feature

@PatStiles
PatStiles changed the base branch from testnet to stagingNovember 21, 2024 20:26
@PatStiles
PatStilesforce-pushed the 1401-1414-retry-config-no-jitter branch from 32f2ea3 to 1c9f3a8CompareNovember 21, 2024 21:07
@uri-99uri-99 changed the title feat(retries): Use config file for retry parametersfeat(retries): Use config params for retry parametersNov 21, 2024
uri-99
uri-99 suggested changes Nov 21, 2024
View reviewed changes
Comment threadcore/chainio/avs_writer.go Outdated
Comment threadcore/chainio/avs_writer.go Outdated
Comment threadcore/chainio/avs_writer.go
Comment threadcore/chainio/avs_writer.go Outdated
Comment threadcore/retry_test.go Outdated
Comment threadcore/retry_test.go Outdated
Comment threadcore/retry_test.go Outdated
Comment threadcore/retry_test.go Outdated
Comment threadcore/retry_test.go Outdated
Comment threadcore/retry_test.go Outdated
@JuArce
JuArce added this pull request to the merge queueNov 22, 2024
Merged via the queue into staging with commit ffc14a2Nov 22, 2024
@JuArce
JuArce deleted the 1401-1414-retry-config-no-jitter branch November 22, 2024 15:11
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MauroToscanoMauroToscanoMauroToscano approved these changes
@JuArceJuArceJuArce approved these changes
+1 more reviewer
@uri-99uri-99uri-99 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@PatStiles@MauroToscano@uri-99@JuArce