fix(storage): clone the defaultRetry to avoid modifying it directly (#11533)

Fixes #11531

Also moved this code since it doesn't need to be done in the loop.

Author: BrennaEpp

storage/grpc_client.go

@@ -1698,6 +1698,15 @@ func (c *grpcStorageClient) OpenWriter(params *openWriterParams, opts ...storage
 
 	s := callSettings(c.settings, opts...)
 
+	retryDeadline := defaultWriteChunkRetryDeadline
+	if params.chunkRetryDeadline != 0 {
+		retryDeadline = params.chunkRetryDeadline
+	}
+	if s.retry == nil {
+		s.retry = defaultRetry.clone()
+	}
+	s.retry.maxRetryDuration = retryDeadline
+
 	// This function reads the data sent to the pipe and sends sets of messages
 	// on the gRPC client-stream as the buffer is filled.
 	go func() {
@@ -1723,16 +1732,6 @@ func (c *grpcStorageClient) OpenWriter(params *openWriterParams, opts ...storage
 					return err
 				}
 
-				retryDeadline := defaultWriteChunkRetryDeadline
-				if params.chunkRetryDeadline != 0 {
-					retryDeadline = params.chunkRetryDeadline
-				}
-
-				if gw.settings.retry == nil {
-					gw.settings.retry = defaultRetry
-				}
-				gw.settings.retry.maxRetryDuration = retryDeadline
-
 				var o *storagepb.Object
 				uploadBuff := func(ctx context.Context) error {
 					obj, err := gw.uploadBuffer(ctx, recvd, offset, doneReading)