Ensure PID files are cleaned up after migration (#2905)

Now that we now longer create PID files for new workloads, ensure they
get cleaned up during the status migration process.

Author: dmjb

pkg/workloads/statuses/file_status.go

@@ -430,12 +430,11 @@ func (*fileStatusManager) migratePIDFromFile(workloadName string, containerInfo
 	}
 	logger.Debugf("found PID %d in PID file for workload %s, will update status file", pid, workloadName)
 
-	// TODO: reinstate this once we decide to completely get rid of PID files.
 	// Delete the PID file after successful migration
-	/*if err := process.RemovePIDFile(baseName); err != nil {
+	if err := process.RemovePIDFile(baseName); err != nil {
 		logger.Warnf("failed to remove PID file for workload %s (base name: %s): %v", workloadName, baseName, err)
 		// Don't return false here - the migration succeeded, cleanup just failed
-	}*/
+	}
 
 	return pid, true
 }