updater: use qubes-vm-update to update dom0 non-interactively#264
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #264 +/- ##
==========================================
+ Coverage 92.63% 92.71% +0.08%
==========================================
Files 66 66
Lines 13672 13618 -54
==========================================
- Hits 12665 12626 -39
+ Misses 1007 992 -15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| for vm in qapp.domains: | ||
| if vm.klass == "AdminVM": | ||
| try: | ||
| if settings.hide_skipped and bool( |
There was a problem hiding this comment.
I might be confused here, but I think the check for skip-update is needed?
There was a problem hiding this comment.
It's in lines 123–126 (dom0 is now treated like the other VMs).
|
This has conflicts now... |
634aaed to
6c2c9dd
Compare
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026051515-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 testsNo failures! Fixed failuresCompared to: https://openqa.qubes-os.org/tests/176874#dependencies Unstable testsDetails
Performance TestsPerformance degradation:No issues Remaining performance tests:No remaining performance tests |
|
opening updater from the widget failed with: I'm not sure if it's related to this PR, I'll restart the test.. |
6c2c9dd to
49a10e0
Compare
|
This needs a rebase (hopefully last one). |
update tests
GUI updater shows dom0 update progress
for test requires: QubesOS/qubes-core-admin-client/pull/404
requires: QubesOS/qubes-core-admin-linux/pull/191