dial_queue_test.go 5.36 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"

Raúl Kripalani's avatar
Raúl Kripalani committed
10 11
	peer "github.com/libp2p/go-libp2p-peer"
	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

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

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

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

}

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

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

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

75 76 77 78 79 80 81 82 83 84 85 86 87
	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)
	}
88 89 90 91

	// 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++ {
92
		_ = dq.Consume()
93 94
	}

Raúl Kripalani's avatar
Raúl Kripalani committed
95 96 97 98 99 100
	// 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)
101 102 103

	// 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
104
	for i := 0; i < 3; i++ {
105 106 107 108
		hang <- struct{}{}
	}

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

Raúl Kripalani's avatar
Raúl Kripalani committed
111 112 113 114
	// unblock the rest.
	for i := 0; i < 10; i++ {
		hang <- struct{}{}
	}
115

Raúl Kripalani's avatar
Raúl Kripalani committed
116
	wg = new(sync.WaitGroup)
117 118 119
	// 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
120 121
	// enqueue more jobs.
	for i := 0; i < 6; i++ {
122 123 124 125
		in.EnqChan <- peer.ID(i)
	}

	// let's check we have 6 workers hanging.
Raúl Kripalani's avatar
Raúl Kripalani committed
126
	waitForWg(t, wg, 2*time.Second)
127 128 129
}

// 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
130
func TestDialQueueShrinksWithWhenIdle(t *testing.T) {
131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146
	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)
	}

147 148 149 150 151 152 153 154 155 156 157 158 159
	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)
	}
160 161 162

	// keep up to speed with backlog by releasing the dial function every time we acquire a channel.
	for i := 0; i < 13; i++ {
163
		ch := dq.Consume()
164 165 166 167 168 169 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
		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)
	}

200 201 202 203 204 205 206 207 208 209 210 211
	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)
	}
212 213 214

	// pick up three consumers.
	for i := 0; i < 3; i++ {
215
		_ = dq.Consume()
216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239
		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:
	}
}