Get the current list of PVCs to delete before deleting them

There is a failure case where if a PVC fails to delete, that this line refuses
to get executed.

The root of the issue is that PVCs can be deleted at different times during the
"rmdata" job, we need to refresh the list of PVCs to be deleted before doing the
final pass through them.

Issue: [ch6499]

Author: Jonathan S. Katz

pgo-rmdata/rmdata/process.go

@@ -66,10 +66,6 @@ func Delete(request Request) {
 		return
 	}
 
-	pvcList, err := getPVCs(request)
-	if err != nil {
-		log.Error(err)
-	}
 	if request.IsBackup {
 		log.Info("rmdata.Process backup use case")
 		//the case of removing a backup using
@@ -106,9 +102,7 @@ func Delete(request Request) {
 
 	//handle the case of 'pgo delete cluster mycluster'
 	removeCluster(request)
-	err = kubeapi.Deletepgcluster(request.RESTClient,
-		request.ClusterName, request.Namespace)
-	if err != nil {
+	if err := kubeapi.Deletepgcluster(request.RESTClient, request.ClusterName, request.Namespace); err != nil {
 		log.Error(err)
 	}
 	removeServices(request)
@@ -117,6 +111,10 @@ func Delete(request Request) {
 	removePgtasks(request)
 	//removeClusterJobs(request)
 	if request.RemoveData {
+		pvcList, err := getPVCs(request)
+		if err != nil {
+			log.Error(err)
+		}
 		removePVCs(pvcList, request)
 	}