Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Commit

Permalink
demonstration of a possible oversight
Browse files Browse the repository at this point in the history
  • Loading branch information
squeedee committed Nov 8, 2023
1 parent d4d2708 commit a24f659
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions reconcilers/sequence.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ func (r Sequence[T]) Reconcile(ctx context.Context, resource T) (Result, error)
for i, reconciler := range r {
log := logr.FromContextOrDiscard(ctx).
WithName(fmt.Sprintf("%d", i))
ctx := logr.NewContext(ctx, log)
subCtx := logr.NewContext(ctx, log)

result, err := reconciler.Reconcile(ctx, resource)
result, err := reconciler.Reconcile(subCtx, resource)
aggregateResult = AggregateResults(result, aggregateResult)
if err != nil {
return result, err
Expand Down

0 comments on commit a24f659

Please sign in to comment.