Skip to content

Commit 612332c

Browse files
committed
chore: add disk stat error wrapping
While doing some debugging, realized we weren't doing any disk wrapping so it was difficult to work out *which* file on disk didn't exist. Signed-off-by: Justin Chadwell <[email protected]>
1 parent f835dd4 commit 612332c

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

util/disk/disk_openbsd.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,14 @@ package disk
55

66
import (
77
"syscall"
8+
9+
"github.com/pkg/errors"
810
)
911

1012
func GetDiskStat(root string) (DiskStat, error) {
1113
var st syscall.Statfs_t
1214
if err := syscall.Statfs(root, &st); err != nil {
13-
return DiskStat{}, err
15+
return DiskStat{}, errors.Wrapf(err, "could not stat fs at %s", root)
1416
}
1517

1618
return DiskStat{

util/disk/disk_unix.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,14 @@ package disk
55

66
import (
77
"syscall"
8+
9+
"github.com/pkg/errors"
810
)
911

1012
func GetDiskStat(root string) (DiskStat, error) {
1113
var st syscall.Statfs_t
1214
if err := syscall.Statfs(root, &st); err != nil {
13-
return DiskStat{}, err
15+
return DiskStat{}, errors.Wrapf(err, "could not stat fs at %s", root)
1416
}
1517

1618
return DiskStat{

util/disk/disk_windows.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,14 @@
44
package disk
55

66
import (
7+
"github.com/pkg/errors"
78
"golang.org/x/sys/windows"
89
)
910

1011
func GetDiskStat(root string) (DiskStat, error) {
1112
rootUTF16, err := windows.UTF16FromString(root)
1213
if err != nil {
13-
return DiskStat{}, err
14+
return DiskStat{}, errors.Wrapf(err, "could not encode %s", root)
1415
}
1516
var (
1617
totalBytes uint64
@@ -22,7 +23,7 @@ func GetDiskStat(root string) (DiskStat, error) {
2223
&freeAvailableBytes,
2324
&totalBytes,
2425
&totalFreeBytes); err != nil {
25-
return DiskStat{}, err
26+
return DiskStat{}, errors.Wrapf(err, "could not stat fs at %s", root)
2627
}
2728

2829
return DiskStat{

0 commit comments

Comments
 (0)