dial_queue_test.go 5.41 KB
Newer Older
1 2 3 4 5
package dht

import (
	"context"
	"sync"
Raúl Kripalani's avatar
Raúl Kripalani committed
6
	"sync/atomic"
7 8 9
	"testing"
	"time"

10
	"github.com/libp2p/go-libp2p-core/peer"
Raúl Kripalani's avatar
Raúl Kripalani committed
11
	queue "github.com/libp2p/go-libp2p-peerstore/queue"
12 13 14 15 16 17
)

func TestDialQueueGrowsOnSlowDials(t *testing.T) {
	in := queue.NewChanQueue(context.Background(), queue.NewXORDistancePQ("test"))
	hang := make(chan struct{})

Raúl Kripalani's avatar
Raúl Kripalani committed
18
	var cnt int32
19
	dialFn := func(ctx context.Context, p peer.ID) error {
Raúl Kripalani's avatar
Raúl Kripalani committed
20
		atomic.AddInt32(&cnt, 1)
21 22 23 24 25 26 27 28 29 30
		<-hang
		return nil
	}

	// Enqueue 20 jobs.
	for i := 0; i < 20; i++ {
		in.EnqChan <- peer.ID(i)
	}

	// remove the mute period to grow faster.
31 32 33 34 35 36 37 38 39 40 41 42 43
	config := dqDefaultConfig()
	config.maxIdle = 10 * time.Minute
	config.mutePeriod = 0
	dq, err := newDialQueue(&dqParams{
		ctx:    context.Background(),
		target: "test",
		in:     in,
		dialFn: dialFn,
		config: config,
	})
	if err != nil {
		t.Error("unexpected error when constructing the dial queue", err)
	}
44

45 46
	dq.Start()

47
	for i := 0; i < 4; i++ {
48
		_ = dq.Consume()
49 50 51
		time.Sleep(100 * time.Millisecond)
	}

Raúl Kripalani's avatar
Raúl Kripalani committed
52
	for i := 0; i < 20; i++ {
53
		if atomic.LoadInt32(&cnt) > int32(DefaultDialQueueMinParallelism) {
Raúl Kripalani's avatar
Raúl Kripalani committed
54 55 56 57
			return
		}
		time.Sleep(100 * time.Millisecond)
	}
58

Raúl Kripalani's avatar
Raúl Kripalani committed
59
	t.Errorf("expected 19 concurrent dials, got %d", atomic.LoadInt32(&cnt))
60 61 62 63

}

func TestDialQueueShrinksWithNoConsumers(t *testing.T) {
Raúl Kripalani's avatar
Raúl Kripalani committed
64 65
	// reduce interference from the other shrink path.

66 67 68
	in := queue.NewChanQueue(context.Background(), queue.NewXORDistancePQ("test"))
	hang := make(chan struct{})

Raúl Kripalani's avatar
Raúl Kripalani committed
69
	wg := new(sync.WaitGroup)
70 71 72 73 74 75 76
	wg.Add(13)
	dialFn := func(ctx context.Context, p peer.ID) error {
		wg.Done()
		<-hang
		return nil
	}

77 78 79 80 81 82 83 84 85 86 87 88 89
	config := dqDefaultConfig()
	config.maxIdle = 10 * time.Minute
	config.mutePeriod = 0
	dq, err := newDialQueue(&dqParams{
		ctx:    context.Background(),
		target: "test",
		in:     in,
		dialFn: dialFn,
		config: config,
	})
	if err != nil {
		t.Error("unexpected error when constructing the dial queue", err)
	}
90

91 92
	dq.Start()

93 94 95
	// acquire 3 consumers, everytime we acquire a consumer, we will grow the pool because no dial job is completed
	// and immediately returnable.
	for i := 0; i < 3; i++ {
96
		_ = dq.Consume()
97 98
	}

Raúl Kripalani's avatar
Raúl Kripalani committed
99 100 101 102 103 104
	// Enqueue 13 jobs, one per worker we'll grow to.
	for i := 0; i < 13; i++ {
		in.EnqChan <- peer.ID(i)
	}

	waitForWg(t, wg, 2*time.Second)
105 106 107

	// Release a few dialFn, but not all of them because downscaling happens when workers detect there are no
	// consumers to consume their values. So the other three will be these witnesses.
Raúl Kripalani's avatar
Raúl Kripalani committed
108
	for i := 0; i < 3; i++ {
109 110 111 112
		hang <- struct{}{}
	}

	// allow enough time for signalling and dispatching values to outstanding consumers.
Raúl Kripalani's avatar
Raúl Kripalani committed
113
	time.Sleep(1 * time.Second)
114

Raúl Kripalani's avatar
Raúl Kripalani committed
115 116 117 118
	// unblock the rest.
	for i := 0; i < 10; i++ {
		hang <- struct{}{}
	}
119

Raúl Kripalani's avatar
Raúl Kripalani committed
120
	wg = new(sync.WaitGroup)
121 122 123
	// we should now only have 6 workers, because all the shrink events will have been honoured.
	wg.Add(6)

Raúl Kripalani's avatar
Raúl Kripalani committed
124 125
	// enqueue more jobs.
	for i := 0; i < 6; i++ {
126 127 128 129
		in.EnqChan <- peer.ID(i)
	}

	// let's check we have 6 workers hanging.
Raúl Kripalani's avatar
Raúl Kripalani committed
130
	waitForWg(t, wg, 2*time.Second)
131 132 133
}

// Inactivity = workers are idle because the DHT query is progressing slow and is producing too few peers to dial.
Raúl Kripalani's avatar
Raúl Kripalani committed
134
func TestDialQueueShrinksWithWhenIdle(t *testing.T) {
135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150
	in := queue.NewChanQueue(context.Background(), queue.NewXORDistancePQ("test"))
	hang := make(chan struct{})

	var wg sync.WaitGroup
	wg.Add(13)
	dialFn := func(ctx context.Context, p peer.ID) error {
		wg.Done()
		<-hang
		return nil
	}

	// Enqueue 13 jobs.
	for i := 0; i < 13; i++ {
		in.EnqChan <- peer.ID(i)
	}

151 152 153 154 155 156 157 158 159 160 161 162 163
	config := dqDefaultConfig()
	config.maxIdle = 1 * time.Second
	config.mutePeriod = 0
	dq, err := newDialQueue(&dqParams{
		ctx:    context.Background(),
		target: "test",
		in:     in,
		dialFn: dialFn,
		config: config,
	})
	if err != nil {
		t.Error("unexpected error when constructing the dial queue", err)
	}
164

165 166
	dq.Start()

167 168
	// keep up to speed with backlog by releasing the dial function every time we acquire a channel.
	for i := 0; i < 13; i++ {
169
		ch := dq.Consume()
170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205
		hang <- struct{}{}
		<-ch
		time.Sleep(100 * time.Millisecond)
	}

	// wait for MaxIdlePeriod.
	time.Sleep(1500 * time.Millisecond)

	// we should now only have 6 workers, because all the shrink events will have been honoured.
	wg.Add(6)

	// enqueue more jobs
	for i := 0; i < 10; i++ {
		in.EnqChan <- peer.ID(i)
	}

	// let's check we have 6 workers hanging.
	waitForWg(t, &wg, 2*time.Second)
}

func TestDialQueueMutePeriodHonored(t *testing.T) {
	in := queue.NewChanQueue(context.Background(), queue.NewXORDistancePQ("test"))
	hang := make(chan struct{})
	var wg sync.WaitGroup
	wg.Add(6)
	dialFn := func(ctx context.Context, p peer.ID) error {
		wg.Done()
		<-hang
		return nil
	}

	// Enqueue a bunch of jobs.
	for i := 0; i < 20; i++ {
		in.EnqChan <- peer.ID(i)
	}

206 207 208 209 210 211 212 213 214 215 216 217
	config := dqDefaultConfig()
	config.mutePeriod = 2 * time.Second
	dq, err := newDialQueue(&dqParams{
		ctx:    context.Background(),
		target: "test",
		in:     in,
		dialFn: dialFn,
		config: config,
	})
	if err != nil {
		t.Error("unexpected error when constructing the dial queue", err)
	}
218

219 220
	dq.Start()

221 222
	// pick up three consumers.
	for i := 0; i < 3; i++ {
223
		_ = dq.Consume()
224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247
		time.Sleep(100 * time.Millisecond)
	}

	time.Sleep(500 * time.Millisecond)

	// we'll only have 6 workers because the grow signals have been ignored.
	waitForWg(t, &wg, 2*time.Second)
}

func waitForWg(t *testing.T, wg *sync.WaitGroup, wait time.Duration) {
	t.Helper()

	done := make(chan struct{})
	go func() {
		defer close(done)
		wg.Wait()
	}()

	select {
	case <-time.After(wait):
		t.Error("timeout while waiting for WaitGroup")
	case <-done:
	}
}