(chore): Fix the disk-fill fault (#715)

ispeakc0de · web-flow · commit 6a386d141060 · 2024-10-15T22:15:14.000+05:30
Signed-off-by: Shubham Chaudhary &lt;shubham.chaudhary@harness.io&gt;
diff --git a/chaoslib/litmus/disk-fill/helper/disk-fill.go b/chaoslib/litmus/disk-fill/helper/disk-fill.go
@@ -185,8 +185,9 @@ func fillDisk(t targetDetails, bs int) error {
 	out, err := cmd.CombinedOutput()
 	if err != nil {
 		log.Error(err.Error())
+		return cerrors.Error{ErrorCode: cerrors.ErrorTypeChaosInject, Source: t.Source, Target: fmt.Sprintf("{podName: %s, namespace: %s, container: %s}", t.Name, t.Namespace, t.TargetContainer), Reason: string(out)}
 	}
-	return cerrors.Error{ErrorCode: cerrors.ErrorTypeChaosInject, Source: t.Source, Target: fmt.Sprintf("{podName: %s, namespace: %s, container: %s}", t.Name, t.Namespace, t.TargetContainer), Reason: string(out)}
+	return nil
 }
 
 // getEphemeralStorageAttributes derive the ephemeral storage attributes from the target pod

Original file line number	Diff line number	Diff line change
`@@ -185,8 +185,9 @@ func fillDisk(t targetDetails, bs int) error {`
`185`	`185`	`out, err := cmd.CombinedOutput()`
`186`	`186`	`if err != nil {`
`187`	`187`	`log.Error(err.Error())`
	`188`	`+ return cerrors.Error{ErrorCode: cerrors.ErrorTypeChaosInject, Source: t.Source, Target: fmt.Sprintf("{podName: %s, namespace: %s, container: %s}", t.Name, t.Namespace, t.TargetContainer), Reason: string(out)}`
`188`	`189`	`}`
`189`		`- return cerrors.Error{ErrorCode: cerrors.ErrorTypeChaosInject, Source: t.Source, Target: fmt.Sprintf("{podName: %s, namespace: %s, container: %s}", t.Name, t.Namespace, t.TargetContainer), Reason: string(out)}`
	`190`	`+ return nil`
`190`	`191`	`}`
`191`	`192`
`192`	`193`	`// getEphemeralStorageAttributes derive the ephemeral storage attributes from the target pod`