Merge pull request #59651 from wojtek-t/print_stderr

Automatic merge from submit-queue (batch tested with PRs 59447, 59594, 59651, 59389). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Print stderr from go tool pprof in profile gatherer

Merge pull request #59651 from wojtek-t/print_stderr
79e02dee · Kubernetes Submit Queue · GitHub · 5898d630 · 3e3dee68 · 79e02dee
Unverified Commit 79e02dee authored Feb 09, 2018 by Kubernetes Submit Queue Committed by GitHub Feb 09, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

profile_gatherer.go test/e2e/framework/profile_gatherer.go +4 -1

No files found.
--- a/test/e2e/framework/profile_gatherer.go
+++ b/test/e2e/framework/profile_gatherer.go
@@ -17,6 +17,7 @@ limitations under the License.
 package framework

 import (
+	"bytes"
 	"fmt"
 	"io/ioutil"
 	"os"
@@ -107,8 +108,10 @@ func gatherProfileOfKind(profileBaseName, kind string) error {
 	}
 	defer outfile.Close()
 	cmd.Stdout = outfile
+	stderr := bytes.NewBuffer(nil)
+	cmd.Stderr = stderr
 	if err := cmd.Run(); nil != err {
-		return fmt.Errorf("Failed to run 'go tool pprof': %v", err)
+		return fmt.Errorf("Failed to run 'go tool pprof': %v, stderr: %#v", err, stderr.String())
 	}
 	return nil
 }