-
Notifications
You must be signed in to change notification settings - Fork 81
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
uhyve Unable to create VM #90
Comments
Do you use Linux? Is |
Hi Thanks for your help i needed reboot kubernetes/minikube#6468 (comment) |
@gilesbradshaw Does it work with minikube? I never tested it. |
ah sorry I don't know I was just reffing that as where I found suggestion to reboot :) |
@stlankes I think we should handle the error more gracefully. Understanding the panic is probably not that easy for most people. I am reopening the error as a reminder. Maybe someone finds some time to fix this. |
42: improve error message r=jbreitbart a=stlankes PR to solve issuer hermit-os/hermit-rs#90 Co-authored-by: Stefan Lankes <slankes@eonerc.rwth-aachen.de>
I'll merge hermit-os/uhyve#42, but it does not fix the issue I had in mind. In general there are two types of errors:
In case of a bug it is great if there is a stack trace so it can be tracked down, however in case of a user error the stack trace is confusing the user and uhyve should just terminate and print the appropriate message. |
Do you prefer something like this? |
I am not sure. That change means we treat every error the appears within |
any ideas?
The text was updated successfully, but these errors were encountered: