-
Notifications
You must be signed in to change notification settings - Fork 642
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
[qemu] Fix crash on concurrent shutdown #3089
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3089 +/- ##
=======================================
Coverage 88.62% 88.62%
=======================================
Files 238 238
Lines 12146 12146
=======================================
Hits 10764 10764
Misses 1382 1382
|
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.
LGTM, thanks!
bors merge
3089: [qemu] Fix crash on concurrent shutdown r=townsend2010 a=ricab Co-authored-by: Ricardo Abreu <[email protected]>
e4cdadc
to
afb0bc0
Compare
Canceled. |
bors merge |
3089: [qemu] Fix crash on concurrent shutdown r=townsend2010 a=ricab Co-authored-by: Ricardo Abreu <[email protected]>
Build failed: |
FWIW, I restarted the build that failed and it succeeded. The last missing build (win11) is running here: https://github.com/canonical/multipass/actions/runs/5397356920 |
Ok, this passed the integration tests. Will merge manually. |
No description provided.