Skip to content

Commit 0433a83

Browse files
bundle/deploy/lock: drop redundant Errorf on failed lock acquire
The error is returned to the caller; logging it here just produces a duplicate line in the user-facing output. Drop the log; preserve the permission-denied branch and the bare return. Co-authored-by: Isaac
1 parent 00a7e9a commit 0433a83

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

bundle/deploy/lock/workspace_filesystem.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,6 @@ func (l *workspaceFilesystemLock) Acquire(ctx context.Context) error {
4747
log.Infof(ctx, "Acquiring deployment lock (force: %v)", l.force)
4848
err = lk.Lock(ctx, l.force)
4949
if err != nil {
50-
log.Errorf(ctx, "Failed to acquire deployment lock: %v", err)
51-
5250
// If we get a permission or "doesn't exist" error from the API this
5351
// indicates we either don't have permissions or the path is invalid.
5452
if errors.Is(err, fs.ErrPermission) || errors.Is(err, fs.ErrNotExist) {

0 commit comments

Comments
 (0)