Skip to content

Commit

Permalink
Merge pull request #9345 from afbjorklund/gotest-string
Browse files Browse the repository at this point in the history
Fix error in unittest, as pointed out by warning
  • Loading branch information
medyagh authored Sep 28, 2020
2 parents d6d1079 + 647c857 commit c56f64b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/drivers/kic/kic.go
Original file line number Diff line number Diff line change
Expand Up @@ -403,7 +403,7 @@ func killAPIServerProc(runner command.Runner) error {
pid, err := strconv.Atoi(rr.Stdout.String())
if err == nil { // this means we have a valid pid
glog.Warningf("Found a kube-apiserver running with pid %d, will try to kill the proc", pid)
if _, err = runner.RunCmd(exec.Command("pkill", "-9", string(pid))); err != nil {
if _, err = runner.RunCmd(exec.Command("pkill", "-9", fmt.Sprint(pid))); err != nil {
return errors.Wrap(err, "kill")
}
}
Expand Down

0 comments on commit c56f64b

Please sign in to comment.