Skip to content

Commit

Permalink
Merge pull request #823 from mackerelio/check-log-reduce-errors-when-…
Browse files Browse the repository at this point in the history
…a-file-in-log-dir-have-just-been-removed

Reduce check-log errors when a file in the log directory has been removed at the moment of running check-log
  • Loading branch information
ne-sachirou committed Feb 26, 2024
2 parents a842bce + b9fb42a commit 05124ff
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions check-log/lib/check-log.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import (
"context"
"crypto/md5"
"encoding/json"
"errors"
"fmt"
"io"
"log"
Expand Down Expand Up @@ -564,20 +565,21 @@ func saveState(f string, state *state) error {
var errFileNotFoundByInode = fmt.Errorf("old file not found")

func findFileByInode(inode uint, dir string) (string, error) {
entries, err := os.ReadDir(dir)
if err != nil {
return "", err
}
entries, readDirErr := os.ReadDir(dir)
for _, entry := range entries {
fi, err := entry.Info()
if err != nil {
if err != nil && !errors.Is(err, os.ErrNotExist) {
return "", err
}
if detectInode(fi) == inode {
return filepath.Join(dir, fi.Name()), nil
}
}
return "", errFileNotFoundByInode
err := errFileNotFoundByInode
if readDirErr != nil {
err = errors.Join(err, readDirErr)
}
return "", err
}

func openOldFile(f string, state *state) (*os.File, error) {
Expand Down

0 comments on commit 05124ff

Please sign in to comment.