refine error handling

Signed-off-by: YangKeao <[email protected]>

Author: YangKeao

go.mod

@@ -2,7 +2,10 @@ module github.com/chaos-mesh/chaos-driver
 
 go 1.17
 
-require github.com/spf13/cobra v1.2.1
+require (
+	github.com/pkg/errors v0.8.1
+	github.com/spf13/cobra v1.2.1
+)
 
 require (
 	github.com/inconshreveable/mousetrap v1.0.0 // indirect

go.sum

@@ -187,6 +187,7 @@ github.com/modern-go/reflect2 v0.0.0-20180701023420-4b7aa43c6742/go.mod h1:bx2lN
 github.com/modern-go/reflect2 v1.0.1/go.mod h1:bx2lNnkwVCuqBIxFjflWJWanXIb3RllmbCylyMrvgv0=
 github.com/pascaldekloe/goe v0.0.0-20180627143212-57f6aae5913c/go.mod h1:lzWF7FIEvWOWxwDKqyGYQf6ZUaNfKdP144TG7ZOy1lc=
 github.com/pelletier/go-toml v1.9.3/go.mod h1:u1nR/EPcESfeI/szUZKdtJ0xRNbUoANCkoOuaOx1Y+c=
+github.com/pkg/errors v0.8.1 h1:iURUrRGxPUNPdy5/HRSm+Yj6okJ6UtLINN0Q9M4+h3I=
 github.com/pkg/errors v0.8.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0=
 github.com/pkg/sftp v1.10.1/go.mod h1:lYOWFsE0bwd1+KfKJaKeuokY15vzFx25BLbzYYoAxZI=
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=

pkg/client/client.go

@@ -19,11 +19,12 @@ package client
 import "C"
 
 import (
-	"errors"
 	"math"
 	"os"
 	"syscall"
 	"unsafe"
+
+	"github.com/pkg/errors"
 )
 
 type Client struct {
@@ -38,11 +39,11 @@ var ErrFailToGetStat = errors.New("failed to get stat")
 func New() (*Client, error) {
 	fd, err := syscall.Open("/dev/chaos", syscall.O_RDWR, 0)
 	if err != nil {
-		return nil, err
+		return nil, errors.Wrap(err, "open /dev/chaos")
 	}
 
 	if C.get_version(C.int(fd)) != COMPATIBLE_VERSION {
-		return nil, ErrIncompatibleVersion
+		return nil, errors.WithStack(ErrIncompatibleVersion)
 	}
 
 	return &Client{
@@ -61,11 +62,11 @@ func (c *Client) InjectIOEMDelay(devPath string, op int, pidNs uint, delay int64
 	if len(devPath) > 0 {
 		info, err := os.Stat(devPath)
 		if err != nil {
-			return 0, err
+			return 0, errors.Wrapf(err, "stat file: %s", devPath)
 		}
 		stat, ok := info.Sys().(*syscall.Stat_t)
 		if !ok {
-			return 0, ErrFailToGetStat
+			return 0, errors.Wrap(ErrFailToGetStat, "dev info is not a stat")
 		}
 
 		dev = C.uint32_t(stat.Rdev)
@@ -76,7 +77,7 @@ func (c *Client) InjectIOEMDelay(devPath string, op int, pidNs uint, delay int64
 
 	id := C.add_injection(C.int(c.fd), 0, unsafe.Pointer(&ioem_injection), 0, unsafe.Pointer(&delay_arg))
 	if id < 0 {
-		return 0, syscall.Errno(-id)
+		return 0, errors.Wrap(syscall.Errno(-id), "add injection")
 	}
 
 	return int(id), nil
@@ -85,7 +86,7 @@ func (c *Client) InjectIOEMDelay(devPath string, op int, pidNs uint, delay int64
 func (c *Client) Recover(id int) error {
 	err := C.del_injection(C.int(c.fd), C.int(id))
 	if err != 0 {
-		return syscall.Errno(err)
+		return errors.Wrap(syscall.Errno(err), "recover injection")
 	}
 
 	return nil