Skip to content

Commit

Permalink
chore: change default factor of mirroring backoff
Browse files Browse the repository at this point in the history
  • Loading branch information
fraidev committed Sep 28, 2024
1 parent d344743 commit 1211362
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
5 changes: 3 additions & 2 deletions crates/fluvio-sc/src/controllers/mirroring/controller.rs
Original file line number Diff line number Diff line change
Expand Up @@ -284,8 +284,9 @@ impl<C: MetadataItem> RemoteMirrorController<C> {

fn create_backoff() -> ExponentialBackoff {
ExponentialBackoffBuilder::default()
.min(Duration::from_millis(100))
.max(Duration::from_secs(5))
.factor(1.1)
.min(Duration::from_secs(1))
.max(Duration::from_secs(30))
.build()
.unwrap()
}
Expand Down
5 changes: 3 additions & 2 deletions crates/fluvio-spu/src/mirroring/remote/controller.rs
Original file line number Diff line number Diff line change
Expand Up @@ -509,8 +509,9 @@ where

fn create_backoff() -> ExponentialBackoff {
ExponentialBackoffBuilder::default()
.min(Duration::from_millis(100))
.max(Duration::from_secs(5))
.factor(1.1)
.min(Duration::from_secs(1))
.max(Duration::from_secs(30))
.build()
.unwrap()
}
Expand Down
4 changes: 2 additions & 2 deletions tests/cli/mirroring_smoke_tests/e2e/fluvio-core.bats
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ setup_file() {
}

@test "Home status at remote 1 should show the home cluster connected" {
sleep 30
sleep 60
run timeout 15s "$FLUVIO_BIN" home status

assert_success
Expand Down Expand Up @@ -165,7 +165,7 @@ setup_file() {
}

@test "Home status at remote 2 should show the home cluster connected" {
sleep 30
sleep 60
run timeout 15s "$FLUVIO_BIN" home status

assert_success
Expand Down

0 comments on commit 1211362

Please sign in to comment.