From c265e434a555c579eb8206a1e70013828b9ccc1e Mon Sep 17 00:00:00 2001
From: Travis Glenn Hansen <travisghansen@yahoo.com>
Date: Tue, 2 Nov 2021 13:17:45 -0600
Subject: [PATCH] do not send 0 for snapshot size_bytes

Signed-off-by: Travis Glenn Hansen <travisghansen@yahoo.com>
---
 src/driver/controller-client-common/index.js | 2 +-
 src/driver/controller-synology/index.js      | 4 ++--
 src/driver/controller-zfs-ssh/index.js       | 4 ++--
 src/driver/freenas/api.js                    | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/driver/controller-client-common/index.js b/src/driver/controller-client-common/index.js
index f08e1e2..db251c3 100644
--- a/src/driver/controller-client-common/index.js
+++ b/src/driver/controller-client-common/index.js
@@ -670,7 +670,7 @@ class ControllerClientCommonDriver extends CsiBaseDriver {
          * The purpose of this field is to give CO guidance on how much space
          * is needed to create a volume from this snapshot.
          */
-        size_bytes: 0,
+        //size_bytes: 0,
         snapshot_id,
         source_volume_id: source_volume_id,
         //https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/timestamp.proto
diff --git a/src/driver/controller-synology/index.js b/src/driver/controller-synology/index.js
index df25669..a06736c 100644
--- a/src/driver/controller-synology/index.js
+++ b/src/driver/controller-synology/index.js
@@ -853,7 +853,7 @@ class ControllerSynologyDriver extends CsiBaseDriver {
            * The purpose of this field is to give CO guidance on how much space
            * is needed to create a volume from this snapshot.
            */
-          size_bytes: 0,
+          //size_bytes: 0,
           snapshot_id: `/lun/${lun.lun_id}/${snapshot.uuid}`, // add shanpshot_uuid //fixme
           source_volume_id: source_volume_id,
           //https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/timestamp.proto
@@ -880,7 +880,7 @@ class ControllerSynologyDriver extends CsiBaseDriver {
          * The purpose of this field is to give CO guidance on how much space
          * is needed to create a volume from this snapshot.
          */
-        size_bytes: 0,
+        //size_bytes: 0,
         snapshot_id: `/lun/${lun.lun_id}/${response.body.data.snapshot_uuid}`,
         source_volume_id: source_volume_id,
         //https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/timestamp.proto
diff --git a/src/driver/controller-zfs-ssh/index.js b/src/driver/controller-zfs-ssh/index.js
index 480faf6..e0aad78 100644
--- a/src/driver/controller-zfs-ssh/index.js
+++ b/src/driver/controller-zfs-ssh/index.js
@@ -1874,7 +1874,7 @@ class ControllerZfsSshBaseDriver extends CsiBaseDriver {
                * In that vein, I think it's best to return 0 here given the
                * unknowns of 'cow' implications.
                */
-              size_bytes: 0,
+              //size_bytes: 0,
 
               // remove parent dataset details
               snapshot_id: row["name"].replace(
@@ -2173,7 +2173,7 @@ class ControllerZfsSshBaseDriver extends CsiBaseDriver {
          * In that vein, I think it's best to return 0 here given the
          * unknowns of 'cow' implications.
          */
-        size_bytes: 0,
+        //size_bytes: 0,
 
         // remove parent dataset details
         snapshot_id: properties.name.value.replace(
diff --git a/src/driver/freenas/api.js b/src/driver/freenas/api.js
index eab6388..48ec19c 100644
--- a/src/driver/freenas/api.js
+++ b/src/driver/freenas/api.js
@@ -3556,7 +3556,7 @@ class FreeNASApiDriver extends CsiBaseDriver {
                * In that vein, I think it's best to return 0 here given the
                * unknowns of 'cow' implications.
                */
-              size_bytes: 0,
+              //size_bytes: 0,
 
               // remove parent dataset details
               snapshot_id: row["name"].replace(
@@ -3946,7 +3946,7 @@ class FreeNASApiDriver extends CsiBaseDriver {
          * In that vein, I think it's best to return 0 here given the
          * unknowns of 'cow' implications.
          */
-        size_bytes: 0,
+        //size_bytes: 0,
 
         // remove parent dataset details
         snapshot_id: properties.name.value.replace(