Skip to content

Commit 658134f

Browse files
committed
Remove setting reconcileID in helmrepo-oci logger
With the new controller-runtime, the reconcileID is automatically set per reconciliation and need not be set explicitly. Signed-off-by: Sunny <darkowlzz@protonmail.com>
1 parent 6a560d9 commit 658134f

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

controllers/helmrepository_controller_oci.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ import (
3333
"k8s.io/apimachinery/pkg/runtime"
3434
"k8s.io/apimachinery/pkg/types"
3535
kerrors "k8s.io/apimachinery/pkg/util/errors"
36-
"k8s.io/apimachinery/pkg/util/uuid"
3736
kuberecorder "k8s.io/client-go/tools/record"
3837
ctrl "sigs.k8s.io/controller-runtime"
3938
"sigs.k8s.io/controller-runtime/pkg/client"
@@ -113,9 +112,7 @@ func (r *HelmRepositoryOCIReconciler) SetupWithManagerAndOptions(mgr ctrl.Manage
113112

114113
func (r *HelmRepositoryOCIReconciler) Reconcile(ctx context.Context, req ctrl.Request) (result ctrl.Result, retErr error) {
115114
start := time.Now()
116-
log := ctrl.LoggerFrom(ctx).
117-
// Sets a reconcile ID to correlate logs from all suboperations.
118-
WithValues("reconcileID", uuid.NewUUID())
115+
log := ctrl.LoggerFrom(ctx)
119116

120117
// logger will be associated to the new context that is
121118
// returned from ctrl.LoggerInto.

0 commit comments

Comments
 (0)