-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
qvm-start-daemon: fix TypeError in message #287
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #287 +/- ##
==========================================
- Coverage 74.60% 74.58% -0.02%
==========================================
Files 51 51
Lines 7899 7901 +2
==========================================
Hits 5893 5893
- Misses 2006 2008 +2 ☔ View full report in Codecov by Sentry. |
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.2&build=2024051221-4.2&flavor=pull-requests New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.2&build=2024050210-4.2&flavor=update
Failed tests8 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/98585#dependencies 1 fixed
Unstable tests
|
Not really an issue with this PR, but I think the fix belongs here:
At this point While at it, I think pre- and non-pre- events wants splitting to do just cleanup in the pre- event and just starting in non-pre events. Otherwise at least dom0 instance (but others, if qrexec policy allows) will try to do things twice. |
No description provided.