Merge pull request #40 from niladrih/no-bdc

cherry-pick(pre-upgrade): Removed error return when no BDC is found for a PV

cmd/provisioner-localpv/app/backward_compatability.go

@@ -24,6 +24,7 @@ import (
 	mconfig "github.com/openebs/maya/pkg/apis/openebs.io/v1alpha1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	clientset "k8s.io/client-go/kubernetes"
+	"k8s.io/klog"
 )
 
 // This function performs the preupgrade related tasks for 1.0 to 1.1
@@ -48,7 +49,12 @@ func addLocalPVFinalizerOnAssociatedBDCs(kubeClient *clientset.Clientset) error
 		bdcObj, err := blockdeviceclaim.NewKubeClient().WithNamespace(getOpenEBSNamespace()).
 			Get(bdcName, metav1.GetOptions{})
 		if err != nil {
-			return errors.Wrapf(err, "failed to get bdc %v", bdcName)
+			// BDCs may not exist if the PV reclaimPolicy is set
+			// to 'Retain' and the BDCs have been manually removed
+			// Ref: github.com/openebs/openebs/issues/3363
+			// TODO: Clean this part of the code up a bit.
+			klog.Warningf("failed to get bdc %v", bdcName)
+			continue
 		}
 
 		// Add finalizer only if deletionTimestamp is not set