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

stderr from Vagrant::SSH::Session should be included in error reports #285

Closed
charles-dyfis-net opened this issue Jan 24, 2011 · 3 comments

Comments

@charles-dyfis-net
Copy link

Right now, when an ssh exec! call fails, only the command which was run is emitted.

However, as Vagrant does not transfer commands to the client in a way which avoids unquoting or other modification in-flight, it is not guaranteed that re-running the command as given in the code will actually reproduce the failure. To permit the user to understand these errors, the actual message should be emitted with the error.

@mitchellh
Copy link
Contributor

Charles,

This is a good idea. I think your issue report here outlines a larger issue in Vagrant that there is poor internal logging. Logging in general needs to be much improved.

I'll consider this a first step in that process.

@charles-dyfis-net
Copy link
Author

I'm personally using 951bef8f151f0a121cb93f5f3f232a5a09ec1d1b for this issue.

@mitchellh
Copy link
Contributor

Failed SSH commands show output in error message. [closed by c1373f4]

skade pushed a commit to Asquera/vagrant that referenced this issue Apr 19, 2011
@ghost ghost locked and limited conversation to collaborators Apr 17, 2020
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants