golint-sl

Appearance

reconciler

About 266 wordsLess than 1 minute

2025-01-16

Enforces Kubernetes reconciler best practices.

Category

Kubernetes

What It Checks

This analyzer ensures reconciler functions follow Kubernetes controller patterns:

  • Proper error handling
  • Correct requeue behavior
  • Resource not found handling

Why It Matters

Reconcilers have subtle requirements. Incorrect patterns cause:

  • Infinite reconcile loops
  • Missed updates
  • Resource leaks
  • Controller crashes

Examples

Bad: Not Handling NotFound

func (r *Reconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
    obj := &myv1.MyResource{}
    if err := r.Get(ctx, req.NamespacedName, obj); err != nil {
        return ctrl.Result{}, err  // Error logged for normal deletions!
    }
    // ...
}

Good: Ignore NotFound

func (r *Reconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
    obj := &myv1.MyResource{}
    if err := r.Get(ctx, req.NamespacedName, obj); err != nil {
        return ctrl.Result{}, client.IgnoreNotFound(err)
    }
    // ...
}

Bad: Tight Requeue Loop

func (r *Reconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
    // ... work ...
    return ctrl.Result{Requeue: true}, nil  // Tight loop, no backoff!
}

Good: Explicit Backoff

func (r *Reconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
    // ... work ...
    return ctrl.Result{RequeueAfter: 30 * time.Second}, nil
}

Good: Let Controller-Runtime Handle Backoff

func (r *Reconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
    // ... work ...
    if err != nil {
        return ctrl.Result{}, err  // Exponential backoff on error
    }
    return ctrl.Result{}, nil  // No requeue if successful
}

Configuration

# .golint-sl.yaml
analyzers:
  reconciler: true  # enabled by default

When to Disable

  • Non-Kubernetes projects
  • Projects not using controller-runtime
analyzers:
  reconciler: false

See Also