Skip to content

updater: use qubes-vm-update to update dom0 non-interactively#264

Open
piotrbartman wants to merge 11 commits intoQubesOS:mainfrom
piotrbartman:updater/dom0
Open

updater: use qubes-vm-update to update dom0 non-interactively#264
piotrbartman wants to merge 11 commits intoQubesOS:mainfrom
piotrbartman:updater/dom0

Conversation

@piotrbartman
Copy link
Member

@piotrbartman piotrbartman commented May 28, 2025

GUI updater shows dom0 update progress

for test requires: QubesOS/qubes-core-admin-client/pull/404
requires: QubesOS/qubes-core-admin-linux/pull/191

@codecov
Copy link

codecov bot commented May 30, 2025

Codecov Report

Attention: Patch coverage is 92.85714% with 4 lines in your changes missing coverage. Please review.

Project coverage is 93.04%. Comparing base (3a85bee) to head (6c2c9dd).

Files with missing lines Patch % Lines
qui/updater/progress_page.py 60.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #264      +/-   ##
==========================================
+ Coverage   92.99%   93.04%   +0.04%     
==========================================
  Files          64       64              
  Lines       13034    12894     -140     
==========================================
- Hits        12121    11997     -124     
+ Misses        913      897      -16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

for vm in qapp.domains:
if vm.klass == "AdminVM":
try:
if settings.hide_skipped and bool(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I might be confused here, but I think the check for skip-update is needed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's in lines 123–126 (dom0 is now treated like the other VMs).

@marmarek
Copy link
Member

This has conflicts now...

@qubesos-bot
Copy link

qubesos-bot commented Jun 23, 2025

OpenQA test summary

Complete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025121602-4.3&flavor=pull-requests

Test run included the following:

New failures, excluding unstable

Compared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025111104-4.3&flavor=update

  • system_tests_network_updates
    • TC_00_Dom0Upgrade_whonix-gateway-18: test_000_update (error)
      subprocess.CalledProcessError: Command 'timeout=120; while ! tor-ci...

Failed tests

1 failures
  • system_tests_network_updates
    • TC_00_Dom0Upgrade_whonix-gateway-18: test_000_update (error)
      subprocess.CalledProcessError: Command 'timeout=120; while ! tor-ci...

Fixed failures

Compared to: https://openqa.qubes-os.org/tests/158999#dependencies
Nothing fixed

Unstable tests

Details

Performance Tests

Performance degradation:

No issues

Remaining performance tests:

No remaining performance tests

@marmarek
Copy link
Member

opening updater from the widget failed with:

Jun 23 10:25:51.247349 dom0 qui-updates[3269]: gdk_window_thaw_toplevel_updates: assertion 'window->update_and_descendants_freeze_count > 0' failed

I'm not sure if it's related to this PR, I'll restart the test..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants