Validator: Remove optional and duplicated dialopts for streaming middleware

[prestonvanloon]

What type of PR is this?

Other

What does this PR do? Why is it needed?

This stream interceptor is already defined in ConstructDialOptions.

prysm/validator/client/service.go

Lines 258 to 309 in 5a299ae

	func ConstructDialOptions(
	maxCallRecvMsgSize int,
	withCert string,
	grpcRetries uint,
	grpcRetryDelay time.Duration,
	extraOpts ...grpc.DialOption,
	) []grpc.DialOption {
	var transportSecurity grpc.DialOption
	if withCert != "" {
	creds, err := credentials.NewClientTLSFromFile(withCert, "")
	if err != nil {
	log.Errorf("Could not get valid credentials: %v", err)
	return nil
	}
	transportSecurity = grpc.WithTransportCredentials(creds)
	} else {
	transportSecurity = grpc.WithInsecure()
	log.Warn("You are using an insecure gRPC connection. If you are running your beacon node and " +
	"validator on the same machines, you can ignore this message. If you want to know " +
	"how to enable secure connections, see: https://docs.prylabs.network/docs/prysm-usage/secure-grpc")
	}
	if maxCallRecvMsgSize == 0 {
	maxCallRecvMsgSize = 10 * 5 << 20 // Default 50Mb
	}
	dialOpts := []grpc.DialOption{
	transportSecurity,
	grpc.WithDefaultCallOptions(
	grpc.MaxCallRecvMsgSize(maxCallRecvMsgSize),
	grpc_retry.WithMax(grpcRetries),
	grpc_retry.WithBackoff(grpc_retry.BackoffLinear(grpcRetryDelay)),
	),
	grpc.WithStatsHandler(&ocgrpc.ClientHandler{}),
	grpc.WithUnaryInterceptor(middleware.ChainUnaryClient(
	grpc_opentracing.UnaryClientInterceptor(),
	grpc_prometheus.UnaryClientInterceptor,
	grpc_retry.UnaryClientInterceptor(),
	grpcutils.LogRequests,
	)),
	grpc.WithChainStreamInterceptor(
	grpcutils.LogStream,
	grpc_opentracing.StreamClientInterceptor(),
	grpc_prometheus.StreamClientInterceptor,
	grpc_retry.StreamClientInterceptor(),
	),
	grpc.WithResolvers(&multipleEndpointsGrpcResolverBuilder{}),
	}
	dialOpts = append(dialOpts, extraOpts...)
	return dialOpts
	}

Which issues(s) does this PR fix?

None reported

Other notes for review

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (develop@565d510). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #8549   +/-   ##
==========================================
  Coverage           ?   58.69%           
==========================================
  Files              ?      458           
  Lines              ?    32911           
  Branches           ?        0           
==========================================
  Hits               ?    19316           
  Misses             ?    10661           
  Partials           ?     2934