DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: fix ignore for the linter - reason was missing

Browse source
This commit is contained in:
Ramon Silveira Borges 2024-03-24 21:59:09 +01:00
parent 699a0d5c00
commit 0dc0086b8d
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
test/e2e/settings/grpc.go
View file

@ -86,7 +86,6 @@ var _ = framework.DescribeSetting("GRPC", func() {
return strings.Contains(server, "grpc_pass grpc://upstream_balancer;") return strings.Contains(server, "grpc_pass grpc://upstream_balancer;")
}) })
//nolint:goconst //string interpolation
conn, err := grpc.Dial(f.GetNginxIP()+":443", conn, err := grpc.Dial(f.GetNginxIP()+":443",
grpc.WithTransportCredentials( grpc.WithTransportCredentials(
credentials.NewTLS(&tls.Config{ credentials.NewTLS(&tls.Config{
@ -96,7 +95,7 @@ var _ = framework.DescribeSetting("GRPC", func() {
), ),
) )
assert.Nil(ginkgo.GinkgoT(), err, "error creating a connection") assert.Nil(ginkgo.GinkgoT(), err, "error creating a connection")
defer conn.Close() //nolint:errcheck defer conn.Close() //nolint:errcheck // Checking the error here is not valuable
client := pb.NewGRPCBinClient(conn) client := pb.NewGRPCBinClient(conn)
ctx := context.Background() ctx := context.Background()