cleanup: fix golint

Author: andyzhangx

iscsi/iscsi_test.go

@@ -723,7 +723,8 @@ func TestConnectorPersistance(t *testing.T) {
 		return makeFakeExecCommand(0, string(mockedOutput))(cmd, args...)
 	}).Reset()
 
-	c.Persist("/tmp/connector.json")
+	err := c.Persist("/tmp/connector.json")
+	assert.Nil(err)
 	c2, err := GetConnectorFromFile("/tmp/connector.json")
 	assert.Nil(err)
 	assert.NotNil(c2)

iscsi/iscsiadm.go

@@ -168,21 +168,21 @@ func Login(tgtIQN, portal string) error {
 func Logout(tgtIQN, portal string) error {
 	debug.Println("Begin Logout...")
 	args := []string{"-m", "node", "-T", tgtIQN, "-p", portal, "-u"}
-	iscsiCmd(args...)
-	return nil
+	_, err := iscsiCmd(args...)
+	return err
 }
 
 // DeleteDBEntry deletes the iscsi db entry for the specified target
 func DeleteDBEntry(tgtIQN string) error {
 	debug.Println("Begin DeleteDBEntry...")
 	args := []string{"-m", "node", "-T", tgtIQN, "-o", "delete"}
-	iscsiCmd(args...)
-	return nil
+	_, err := iscsiCmd(args...)
+	return err
 }
 
 // DeleteIFace delete the iface
 func DeleteIFace(iface string) error {
 	debug.Println("Begin DeleteIFace...")
-	iscsiCmd([]string{"-m", "iface", "-I", iface, "-o", "delete"}...)
-	return nil
+	_, err := iscsiCmd([]string{"-m", "iface", "-I", iface, "-o", "delete"}...)
+	return err
 }