Wrap errors with more context

[obitech]

I think we could provide a bit more context around the returned errors. Right now there are a lot of these kind of error returns:

	if err := r.Create(ctx, newRule); err != nil {
		return ctrl.Result{}, err
	}

// ...

	if err := r.Update(ctx, newRule); err != nil {
		return ctrl.Result{}, err
	}

In this specific example it might not be immediately clear in the logs if an error occurred during the the Create or Update step.

My suggestion would be to wrap errors with minimal context.

[obitech]

Again, happy to provide a PR? 😌

[metalmatze]

Yes, wrapping with fmt.Errorf("error: %w", err) sounds really good.
Logging might be a similar issue where sometimes it's missing context.