Skip to content

Commit 80ab7a0

Browse files
committed
chore: remove spammy 'clean up unused volumes' logs
43f4e31 included a change to the output cleanup flow of the `VolumeConfigController` which added a log whenever the controller runloop ran: 'cleaning up unused volumes'. This patch removes these spammy/not relevant entry to prevent it from cluttering up the logs. Signed-off-by: Laura Brehm <[email protected]>
1 parent 74d3590 commit 80ab7a0

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

internal/app/machined/pkg/controllers/block/volume_config.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ func (ctrl *VolumeConfigController) Run(ctx context.Context, r controller.Runtim
131131
}
132132
}
133133

134-
if err := ctrl.cleanupUnusedVolumes(ctx, r, volumeConfigsByID, volumeMountRequestsByID, logger); err != nil {
134+
if err := ctrl.cleanupUnusedVolumes(ctx, r, volumeConfigsByID, volumeMountRequestsByID); err != nil {
135135
return fmt.Errorf("error cleaning up unused volumes: %w", err)
136136
}
137137
}
@@ -281,10 +281,7 @@ func (ctrl *VolumeConfigController) cleanupUnusedVolumes(
281281
r controller.Runtime,
282282
volumeConfigsByID map[string]*block.VolumeConfig,
283283
volumeMountRequestsByID map[string]*block.VolumeMountRequest,
284-
l *zap.Logger,
285284
) error {
286-
l.Info("cleaning up unused volumes")
287-
// Clean up unused volume configs
288285
for _, volumeConfig := range volumeConfigsByID {
289286
okToDestroy, err := r.Teardown(ctx, volumeConfig.Metadata())
290287
if err != nil {

0 commit comments

Comments
 (0)