Skip to content

Commit 8bbdca4

Browse files
committed
Allow long-running replication full syncs
1 parent 755d88c commit 8bbdca4

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

‎internal/controller/replication_controller.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ func NewReplicationController(client kubernetes.Interface, cfg ReplicationContro
9191
cfg.ReconcileInterval = 30 * time.Second
9292
}
9393
if cfg.HTTPTimeout <= 0 {
94-
cfg.HTTPTimeout = 30 * time.Second
94+
cfg.HTTPTimeout = time.Hour
9595
}
9696
if cfg.AgentLabelSelector == "" {
9797
cfg.AgentLabelSelector = "app.kubernetes.io/component=node"

‎internal/controller/replication_controller_test.go‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,16 @@ import (
1515
"github.com/shipstuff/simple-volume/internal/agent"
1616
)
1717

18+
func TestNewReplicationControllerDefaultsHTTPTimeoutToFullSyncWindow(t *testing.T) {
19+
controller := NewReplicationController(fake.NewSimpleClientset(), ReplicationControllerConfig{})
20+
if controller.cfg.HTTPTimeout != time.Hour {
21+
t.Fatalf("HTTPTimeout = %v, want %v", controller.cfg.HTTPTimeout, time.Hour)
22+
}
23+
if controller.http.Timeout != time.Hour {
24+
t.Fatalf("http client timeout = %v, want %v", controller.http.Timeout, time.Hour)
25+
}
26+
}
27+
1828
func TestDesiredReplicationsDiscoversActiveAndReplicaAgents(t *testing.T) {
1929
storageClass := "simple-volume"
2030
client := fake.NewSimpleClientset(

0 commit comments

Comments
 (0)