Browse Source

Merge pull request #3242 from garenchan/ck-dev1

fix 3238: handle errors for GenerateDirUuid method
pull/3250/head
Chris Lu 3 years ago
committed by GitHub
parent
commit
201023dd15
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 7
      weed/storage/disk_location.go

7
weed/storage/disk_location.go

@ -44,13 +44,11 @@ func GenerateDirUuid(dir string) (dirUuidString string, err error) {
dirUuidString = dirUuid.String() dirUuidString = dirUuid.String()
writeErr := util.WriteFile(fileName, []byte(dirUuidString), 0644) writeErr := util.WriteFile(fileName, []byte(dirUuidString), 0644)
if writeErr != nil { if writeErr != nil {
glog.Warningf("failed to write uuid to %s : %v", fileName, writeErr)
return "", fmt.Errorf("failed to write uuid to %s : %v", fileName, writeErr) return "", fmt.Errorf("failed to write uuid to %s : %v", fileName, writeErr)
} }
} else { } else {
uuidData, readErr := os.ReadFile(fileName) uuidData, readErr := os.ReadFile(fileName)
if readErr != nil { if readErr != nil {
glog.Warningf("failed to read uuid from %s : %v", fileName, readErr)
return "", fmt.Errorf("failed to read uuid from %s : %v", fileName, readErr) return "", fmt.Errorf("failed to read uuid from %s : %v", fileName, readErr)
} }
dirUuidString = string(uuidData) dirUuidString = string(uuidData)
@ -65,7 +63,10 @@ func NewDiskLocation(dir string, maxVolumeCount int, minFreeSpace util.MinFreeSp
} else { } else {
idxDir = util.ResolvePath(idxDir) idxDir = util.ResolvePath(idxDir)
} }
dirUuid, _ := GenerateDirUuid(dir)
dirUuid, err := GenerateDirUuid(dir)
if err != nil {
glog.Fatalf("cannot generate uuid of dir %s: %v", dir, err)
}
location := &DiskLocation{ location := &DiskLocation{
Directory: dir, Directory: dir,
DirectoryUuid: dirUuid, DirectoryUuid: dirUuid,

Loading…
Cancel
Save