-
-
Notifications
You must be signed in to change notification settings - Fork 82
Deserialize device in events only if there is a handler registered #402
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?
Conversation
Deserializing device parameter may trigger a call to get remaining details. Don't do that if there are no handlers for such event anyway. Specifically, this avoids calling admin.vm.device.usb.Available (and others) by sys-audio whenever new device is plugged in.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #402 +/- ##
=======================================
Coverage 76.10% 76.11%
=======================================
Files 53 53
Lines 9287 9289 +2
=======================================
+ Hits 7068 7070 +2
Misses 2219 2219 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025121419-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025111104-4.3&flavor=update
Failed tests12 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/158999#dependencies 18 fixed
Unstable testsDetailsPerformance TestsPerformance degradation:11 performance degradations
Remaining performance tests:84 tests
|
Deserializing device parameter may trigger a call to get remaining
details. Don't do that if there are no handlers for such event anyway.
Specifically, this avoids calling admin.vm.device.usb.Available (and
others) by sys-audio whenever new device is plugged in.