Skip to content

Commit

Permalink
Try to improve logs
Browse files Browse the repository at this point in the history
  • Loading branch information
ljakimczuk committed May 16, 2024
1 parent 93616d8 commit fbea9ec
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion service/controller/app/resource/clients/resource.go
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,6 @@ func (r *Resource) addClientsToContext(ctx context.Context, cr v1alpha1.App) err
K8s: clients.K8sClient,
Helm: clients.HelmClient,
}
r.logger.Debugf(ctx, "clients %#q", clients)

return nil
}
Expand Down
4 changes: 2 additions & 2 deletions service/watcher/helmreleasestatus/helmreleasestatus.go
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ func (c *HelmReleaseStatusWatcher) doWatchStatus(ctx context.Context, client dyn
}

// The HelmRelease CR is named after the App CR and placed in the same namespace,
// hence its metadata can be used to locate the former.
// hence its metadata can be used to locate the latter.
app := v1alpha1.App{}
err = c.k8sClient.CtrlClient().Get(ctx,
types.NamespacedName{Name: helmRelease.Name, Namespace: helmRelease.Namespace},
Expand All @@ -200,7 +200,7 @@ func (c *HelmReleaseStatusWatcher) doWatchStatus(ctx context.Context, client dyn
continue
}

c.logger.Debugf(ctx, "status set for app '%s/%s'", app.Namespace, app.Name)
c.logger.LogCtx(ctx, "level", "debug", "message", fmt.Sprintf("status set for app '%s/%s'", app.Namespace, app.Name))
}
}

Expand Down

0 comments on commit fbea9ec

Please sign in to comment.