Return 400 when can't create backup uri

pkg/cloud_provider/file/file.go

@@ -519,7 +519,8 @@ func (manager *gcfsServiceManager) GetBackup(ctx context.Context, backupUri stri
 func (manager *gcfsServiceManager) CreateBackup(ctx context.Context, obj *ServiceInstance, backupName string, backupLocation string) (*filev1beta1.Backup, error) {
 	backupUri, region, err := CreateBackupURI(obj, backupName, backupLocation)
 	if err != nil {
-		return nil, err
+		klog.Errorf("Failed to create backup URI from given name %s and location %s, error: %v", backupName, backupLocation, err.Error())
+		return nil, status.Error(codes.InvalidArgument, err.Error())
 	}
 
 	backupSource := fmt.Sprintf("projects/%s/locations/%s/instances/%s", obj.Project, obj.Location, obj.Name)
@@ -743,7 +744,7 @@ func CodeForError(err error) *codes.Code {
 }
 
 // This function returns the backup URI, the region that was picked to be the backup resource location and error.
-func CreateBackupURI(obj *ServiceInstance, backupName, backupLocation string) (string, string, error) {
+func CreateBackupURI(obj *ServiceInstance, backupName string, backupLocation string) (string, string, error) {
 	region, err := deduceRegion(obj, backupLocation)
 	if err != nil {
 		return "", "", err

pkg/csi_driver/controller.go

@@ -868,9 +868,11 @@ func (s *controllerServer) CreateSnapshot(ctx context.Context, req *csi.CreateSn
 	}
 
 	// Check for existing snapshot
-	backupUri, _, err := file.CreateBackupURI(filer, req.Name, util.GetBackupLocation(req.GetParameters()))
+	backupLocation := util.GetBackupLocation(req.GetParameters())
+	backupUri, _, err := file.CreateBackupURI(filer, req.Name, backupLocation)
 	if err != nil {
-		return nil, err
+		klog.Errorf("Failed to create backup URI from given name %s and location %s, error: %v", req.Name, backupLocation, err.Error())
+		return nil, status.Error(codes.InvalidArgument, err.Error())
 	}
 	backupInfo, err := s.config.fileService.GetBackup(ctx, backupUri)
 	if err != nil {