Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reconcile results directory and mount on server #1709

Merged
merged 1 commit into from
May 14, 2022
Merged

Reconcile results directory and mount on server #1709

merged 1 commit into from
May 14, 2022

Conversation

johnSchnake
Copy link
Contributor

@johnSchnake johnSchnake commented May 13, 2022

Issue #1688 would cause issues even if the old CLI was targeting
a new server. I've had multiple people this week report issues with
this so I wanted to resolve it and push a new release.

Added a new test to ensure we dont regress on this.

Signed-off-by: John Schnake [email protected]

@johnSchnake
Copy link
Contributor Author

Commented out the fix to see if the test fails first in CI.

@johnSchnake
Copy link
Contributor Author

Confirmed test repro'd error:

        time="2022-05-13T20:48:56Z" level=info msg="Detected done file, transmitting result file" resultFile=/tmp/results/e2e.tar.gz
        time="2022-05-13T20:48:56Z" level=error msg="error gathering host data: open /tmp/results/e2e.tar.gz: no such file or directory" trace="open /tmp/results/e2e.tar.gz: no such file or directory\ngithub.com/vmware-tanzu/sonobuoy/pkg/worker.handleWaitFile.func2\n\t/go/src/github.com/vmware-tanzu/sonobuoy/pkg/worker/worker.go:232\ngithub.com/vmware-
...

Issue #1688 would cause issues even if the old CLI was targeting
a new server. I've had multiple people this week report issues with
this so I wanted to resolve it and push a new release.

Added a new test to ensure we dont regress on this.

Signed-off-by: John Schnake <[email protected]>
@johnSchnake johnSchnake merged commit 6811726 into vmware-tanzu:main May 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant