From 9ef7d35d815c8c12ee34e7a25c84d6ba4f73764f Mon Sep 17 00:00:00 2001 From: Pawan Date: Wed, 26 Aug 2020 20:37:28 +0530 Subject: [PATCH] fix(upgrade): Reverting back to old way of checking the volume status few customers are using old version of the driver where Status field is not present. So mount will fail after the upgrade to the 0.9 or later version. Reverting back to the checking if finalizer is set to check if volume is ready to be mounted. Signed-off-by: Pawan --- changelogs/unreleased/196-pawanpraka1 | 1 + pkg/zfs/mount.go | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) create mode 100644 changelogs/unreleased/196-pawanpraka1 diff --git a/changelogs/unreleased/196-pawanpraka1 b/changelogs/unreleased/196-pawanpraka1 new file mode 100644 index 0000000..06ce361 --- /dev/null +++ b/changelogs/unreleased/196-pawanpraka1 @@ -0,0 +1 @@ +Reverting back to old way of checking the volume status diff --git a/pkg/zfs/mount.go b/pkg/zfs/mount.go index aa911b1..5899cd3 100644 --- a/pkg/zfs/mount.go +++ b/pkg/zfs/mount.go @@ -139,7 +139,7 @@ func verifyMountRequest(vol *apis.ZFSVolume, mountpath string) error { vol.Spec.OwnerNodeID != NodeID { return status.Error(codes.Internal, "verifyMount: volume is owned by different node") } - if vol.Status.State != ZFSStatusReady { + if vol.Finalizers == nil { return status.Error(codes.Internal, "verifyMount: volume is not ready to be mounted") }