Fix node controller panic (#48)

Code2Life · web-flow · commit c1424d9523c9 · 2025-02-17T02:01:03.000Z
* fix: typo

* fix: nil panic issue
diff --git a/internal/controller/node_controller.go b/internal/controller/node_controller.go
@@ -108,7 +108,7 @@ func (r *NodeReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.
 				if err := controllerutil.SetControllerReference(node, gpuNode, r.Scheme); err != nil {
 					return ctrl.Result{}, fmt.Errorf("failed to set controller reference: %w", err)
 				}
-				_, e := controllerutil.CreateOrUpdate(ctx, r.Client, gpuNode, nil)
+				_, e := controllerutil.CreateOrUpdate(ctx, r.Client, gpuNode, func() error { return nil })
 				if e != nil {
 					return ctrl.Result{}, fmt.Errorf("failed to create or patch GPUNode: %w", e)
 				}

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ func (r *NodeReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.`
`108`	`108`	`if err := controllerutil.SetControllerReference(node, gpuNode, r.Scheme); err != nil {`
`109`	`109`	`return ctrl.Result{}, fmt.Errorf("failed to set controller reference: %w", err)`
`110`	`110`	`}`
`111`		`- _, e := controllerutil.CreateOrUpdate(ctx, r.Client, gpuNode, nil)`
	`111`	`+ _, e := controllerutil.CreateOrUpdate(ctx, r.Client, gpuNode, func() error { return nil })`
`112`	`112`	`if e != nil {`
`113`	`113`	`return ctrl.Result{}, fmt.Errorf("failed to create or patch GPUNode: %w", e)`
`114`	`114`	`}`