File: 0013-grpc-middleware-fixes-2.patch

package info (click to toggle)
etcd 3.5.16-8
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 15,868 kB
  • sloc: sh: 3,136; makefile: 477
file content (52 lines) | stat: -rw-r--r-- 1,651 bytes parent folder | download | duplicates (4)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
From: Mathias Gibbens <gibmat@debian.org>
Date: Thu, 1 Aug 2024 14:04:04 +0800
Subject: In the interest of unblocking the grpc transition,
 for now rip out some use of old grpc-middleware from a test

Forwarded: not-needed
---
 tests/integration/testing.go | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/tests/integration/testing.go b/tests/integration/testing.go
index e673751..38cf4b8 100644
--- a/tests/integration/testing.go
+++ b/tests/integration/testing.go
@@ -19,21 +19,17 @@ import (
 	"path/filepath"
 	"testing"
 
-	grpc_logsettable "github.com/grpc-ecosystem/go-grpc-middleware/logging/settable"
 	"go.etcd.io/etcd/client/pkg/v3/testutil"
 	clientv3 "go.etcd.io/etcd/client/v3"
 	"go.etcd.io/etcd/server/v3/embed"
 	"go.etcd.io/etcd/server/v3/verify"
 	"go.uber.org/zap/zapcore"
-	"go.uber.org/zap/zapgrpc"
 	"go.uber.org/zap/zaptest"
 )
 
-var grpc_logger grpc_logsettable.SettableLoggerV2
 var insideTestContext bool
 
 func init() {
-	grpc_logger = grpc_logsettable.ReplaceGrpcLoggerV2()
 }
 
 type testOptions struct {
@@ -88,7 +84,6 @@ func BeforeTest(t testutil.TB, opts ...TestOption) {
 
 	// Registering cleanup early, such it will get executed even if the helper fails.
 	t.Cleanup(func() {
-		grpc_logger.Reset()
 		insideTestContext = previousInsideTestContext
 		os.Chdir(previousWD)
 	})
@@ -97,7 +92,6 @@ func BeforeTest(t testutil.TB, opts ...TestOption) {
 		t.Fatal("already in test context. BeforeTest was likely already called")
 	}
 
-	grpc_logger.Set(zapgrpc.NewLogger(zaptest.NewLogger(t).Named("grpc")))
 	insideTestContext = true
 
 	// Integration tests should verify written state as much as possible.