upgrade test shell invoker logs isn't showing up in a subtest log output

[dprotaso]

Noticed during some upgrade test runs that the subtests don't have verbose logs - so it's unclear what the failure is

Looking at the raw output will https://gist.github.com/dprotaso/514807299f699353acc8d4adf8144789 show you there are logs being printed to stdout/stderr directly

For the upgrade tests we should direct this output to the subtest it was invoked

[dprotaso]

The default values should be overridden for upgrade tests

https://github.com/knative/hack/blob/f2f9b6f91263b25040acf4a98cafc9b6d7db0230/shell/executor.go#L81-L86

	if config.Out == nil {
		config.Out = os.Stdout
	}
	if config.Err == nil {
		config.Err = os.Stderr
	}

[cardil]

I think I'll move this issue to the hack repo.

It looks I can't do it.

[cardil]

/transfer hack

[cardil]

/reopen

There are PRs needs to be done.

[knative-prow]

@cardil: Reopened this issue.

Details

In response to this:

/reopen

There are PRs needs to be done.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.