-
Notifications
You must be signed in to change notification settings - Fork 60
prime95 fix #555
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
base: main
Are you sure you want to change the base?
prime95 fix #555
Conversation
{ | ||
await this.WaitAsync(explicitTimeout, cancellationToken); | ||
process.SafeKill(); | ||
// await process.WaitForExitAsync(cancellationToken); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure this doesn't add delay to unit tests, if it does, find a way to mock it
…osoft/VirtualClient into users/saibulusu/prime95-fix
…osoft/VirtualClient into users/saibulusu/prime95-fix
Does this work for you? if not check my changes in #559 |
It still doesn't work when I use the process manager kill method, it produces the 137 OOM error when I use a VM of size Standard D64ds v6 (64 vcpus, 256 GiB memory). |
Error message: "Process must exit before requested information can be determined."
original: 22d56276-2f46-47f6-b3e5-c2469e9e566e
wait for exit: 6729cb33-92fd-43ce-bd12-81e0b2bb1788
5 seconds: 5e8cd34b-9518-427f-8ae0-05047148ba32