Disable notification for denied property.GetAll call#830
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #830 +/- ##
==========================================
- Coverage 70.47% 70.36% -0.11%
==========================================
Files 61 61
Lines 14159 14100 -59
==========================================
- Hits 9978 9922 -56
+ Misses 4181 4178 -3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
I can imagine scenario when rouge VM tries to call |
|
It disables notification for a denied call, it doesn't allow anything new. |
|
Yeah, I understand that nothing is being allowed by this change, just notifications being made silent. |
It's expected to be denied when some properties are allowed but not all. QubesOS/qubes-issues#10534
marmarek
force-pushed
the
getall-notify-no
branch
from
f05fe62 to
e0fe88d
Compare
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026062520-devel&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=2026050504-devel&flavor=update
Failed tests46 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/176874#dependencies 14 fixed
Unstable testsDetails
Performance TestsPerformance degradation:19 performance degradations
Remaining performance tests:51 tests
|
3 participants
It's expected to be denied when some properties are allowed but not all.
QubesOS/qubes-issues#10534