add context to resourceManager functions

Signed-off-by: Rohan CJ <rohantmp@gmail.com>
openshift · Dec 6, 2021 · 01882ac · 01882ac
1 parent 69b0c1f
commit 01882ac
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/controllers/lvmcluster_controller.go b/controllers/lvmcluster_controller.go
@@ -122,14 +122,14 @@ type resourceManager interface {
 	getName() string
 
 	// ensureCreated should check the resources managed by this unit
-	ensureCreated(*LVMClusterReconciler, lvmv1alpha1.LVMCluster) error
+	ensureCreated(*LVMClusterReconciler, context.Context, lvmv1alpha1.LVMCluster) error
 
 	// ensureDeleted should wait for the resources to be cleaned up
-	ensureDeleted(*LVMClusterReconciler, lvmv1alpha1.LVMCluster) error
+	ensureDeleted(*LVMClusterReconciler, context.Context, lvmv1alpha1.LVMCluster) error
 
 	// updateStatus should optionally update the CR's status about the health of the managed resource
 	// each unit will have updateStatus called induvidually so
 	// avoid status fields like lastHeartbeatTime and have a
 	// status that changes only when the operands change.
-	updateStatus(*LVMClusterReconciler, lvmv1alpha1.LVMCluster) error
+	updateStatus(*LVMClusterReconciler, context.Context, lvmv1alpha1.LVMCluster) error
 }
diff --git a/doc/design/README.md b/doc/design/README.md
@@ -27,15 +27,15 @@ type resourceManager interface {
 	getName() string
 
 	// ensureCreated should check the resources managed by this unit
-	ensureCreated(*LVMClusterReconciler, lvmv1alpha1.LVMCluster) error
+	ensureCreated(*LVMClusterReconciler, context.Context, lvmv1alpha1.LVMCluster) error
 
 	// ensureDeleted should wait for the resources to be cleaned up
-	ensureDeleted(*LVMClusterReconciler, lvmv1alpha1.LVMCluster) error
+	ensureDeleted(*LVMClusterReconciler, context.Context, lvmv1alpha1.LVMCluster) error
 
 	// updateStatus should optionally update the CR's status about the health of the managed resource
 	// each unit will have updateStatus called induvidually so
 	// avoid status fields like lastHeartbeatTime and have a
 	// status that changes only when the operands change.
-	updateStatus(*LVMClusterReconciler, lvmv1alpha1.LVMCluster) error
+	updateStatus(*LVMClusterReconciler, context.Context, lvmv1alpha1.LVMCluster) error
 }
 ```