Merge pull request #37729 from zmerlynn/fix-hack-e2e-up-dump

Automatic merge from submit-queue

hack/e2e.go: Dump cluster logs in case of Up failure

**What this PR does / why we need it**: A failure in `Up` currently results in no attempt to grab cluster logs. This fixes that hole. (Sigh, a ton of holes for this diagnosis path.)
This commit is contained in:
Kubernetes Submit Queue 2016-11-30 21:59:06 -08:00 committed by GitHub
commit 0ed38ac647

View File

@ -236,6 +236,11 @@ func run(deploy deployer) error {
}
// Start the cluster using this version.
if err := xmlWrap("Up", deploy.Up); err != nil {
if *dump != "" {
xmlWrap("DumpClusterLogs", func() error {
return DumpClusterLogs(*dump)
})
}
return fmt.Errorf("starting e2e cluster: %s", err)
}
if *dump != "" {