Clean up provider specfic code, remove provider related logic

Author: dannawang0221

populator-machinery/controller.go

@@ -771,11 +771,6 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 					// We'll get called again later when the pvcPrime gets bounded
 					return nil
 				}
-				pv, err := params.KubeClient.CoreV1().PersistentVolumes().Get(ctx, pvcPrime.Spec.VolumeName, metav1.GetOptions{})
-				if err != nil {
-					return err
-				}
-				params.PV = pv
 
 				err = c.providerFunctionConfig.PopulateFn(ctx, *params)
 				if err != nil {
@@ -792,7 +787,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 					return err
 				}
 				if !complete {
-					//TODO: Revisited if there is a better way to requeue pvc than return an error
+					// TODO: Revisited if there is a better way to requeue pvc than return an error
 					// Return error to force reque pvc. We'll get called again later when the population operation complete
 					return fmt.Errorf(reasonWaitForDataPopulationFinished)
 				}