qubes-core-admin icon indicating copy to clipboard operation
qubes-core-admin copied to clipboard

Add free-form text to qube for notes, comments, ...

Open alimirjamali opened this issue 9 months ago • 3 comments

Core and API part of adding free-form text to each qube for comments, notes, descriptions, remarks, reminders, etc.

fixes: https://github.com/QubesOS/qubes-issues/issues/899

alimirjamali avatar Feb 16 '25 16:02 alimirjamali

Black complains

marmarek avatar Feb 20 '25 16:02 marmarek

Codecov Report

Attention: Patch coverage is 94.28571% with 2 lines in your changes missing coverage. Please review.

Project coverage is 70.45%. Comparing base (1473c37) to head (96cc2ac). Report is 26 commits behind head on main.

Files with missing lines Patch % Lines
qubes/backup.py 66.66% 1 Missing :warning:
qubes/vm/qubesvm.py 92.30% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #656      +/-   ##
==========================================
+ Coverage   70.25%   70.45%   +0.19%     
==========================================
  Files          59       61       +2     
  Lines       12747    12932     +185     
==========================================
+ Hits         8956     9111     +155     
- Misses       3791     3821      +30     
Flag Coverage Δ
unittests 70.45% <94.28%> (+0.19%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar Feb 20 '25 17:02 codecov[bot]

Black complains

Black complains are resolved. And the Codecov is advising on the required unittests. I guess I can proceed with the unittests? And I should assure that proper errors are raised if necessary.

alimirjamali avatar Feb 20 '25 18:02 alimirjamali

OpenQA test summary

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

Test run included the following:

  • https://github.com/QubesOS/qubes-core-admin-client/pull/351 (https://github.com/QubesOS/qubes-core-admin-client/pull/351/commits/08f6820cec6fd9399e97b3a2f31ba37cc92123f5)
  • https://github.com/QubesOS/qubes-manager/pull/420 (https://github.com/QubesOS/qubes-manager/pull/420/commits/d90b48c163917dd518dedcdd8bbd245801de92e8)
  • https://github.com/QubesOS/qubes-manager/pull/398 (https://github.com/QubesOS/qubes-manager/pull/398/commits/679eff400bd0de4887f0bb61de34250a1f842709)
  • https://github.com/QubesOS/qubes-core-agent-linux/pull/570 (https://github.com/QubesOS/qubes-core-agent-linux/pull/570/commits/81430ea4b7d8bd156287b925ea61e935e92a884a)
  • https://github.com/QubesOS/qubes-gui-agent-linux/pull/229 (https://github.com/QubesOS/qubes-gui-agent-linux/pull/229/commits/5a5054451decf83a867a3c7e1accdd074880a2c1)
  • https://github.com/QubesOS/qubes-manager/pull/421 (https://github.com/QubesOS/qubes-manager/pull/421/commits/2bed45c886640ae439096bc3a8673a68efaad776)
  • https://github.com/QubesOS/qubes-gui-agent-linux/pull/228 (https://github.com/QubesOS/qubes-gui-agent-linux/pull/228/commits/5947c790d8443fd4b2053519daeee8537be051af)
  • https://github.com/QubesOS/qubes-core-admin-client/pull/332 (https://github.com/QubesOS/qubes-core-admin-client/pull/332/commits/e22c531c6eb2e0b5404b58295e499ca758627009)
  • https://github.com/QubesOS/qubes-core-admin-client/pull/353 (https://github.com/QubesOS/qubes-core-admin-client/pull/353/commits/13cffb3b0c7fe4fd6b005153eab509db9c8ab3a8)
  • https://github.com/QubesOS/qubes-core-admin/pull/656 (https://github.com/QubesOS/qubes-core-admin/pull/656/commits/96cc2acbd1f5f64c91f31860f9745f14c032bf0d)
  • https://github.com/QubesOS/qubes-desktop-linux-manager/pull/256 (https://github.com/QubesOS/qubes-desktop-linux-manager/pull/256/commits/fc00fde4c57ae3bf5093836e8bc35983058f0320)
  • https://github.com/QubesOS/qubes-manager/pull/422 (https://github.com/QubesOS/qubes-manager/pull/422/commits/b682d5458d0e1a2c76b95c0e12eb4f3ce8cee9c6)

New failures, excluding unstable

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

  • system_tests_kde_gui_interactive

    • gui_keyboard_layout: wait_serial (wait serial expected) # wait_serial expected: "echo -e '[Layout]\nLayoutList=us,de' | sud...

    • gui_keyboard_layout: Failed (test died) # Test died: command 'test "$(cd ~user;ls e1*)" = "$(qvm-run -p wor...

  • system_tests_qwt_win10_seamless@hw13

    • windows_clipboard_and_filecopy: unnamed test (unknown)
    • windows_clipboard_and_filecopy: Failed (test died) # Test died: no candidate needle with tag(s) 'windows-Edge-address-...
  • system_tests_qwt_win11@hw13

    • windows_install: wait_serial (wait serial expected) # wait_serial expected: qr/dcWzE-\d+-/...

    • windows_install: Failed (test died + timed out) # Test died: command 'script -e -c 'bash -x /usr/bin/qvm-create-win...

  • system_tests_guivm_vnc_gui_interactive

    • guivm_manager: unnamed test (unknown)
    • guivm_manager: Failed (test died) # Test died: no candidate needle with tag(s) 'qubes-qube-manager' m...

Failed tests

8 failures
  • system_tests_kde_gui_interactive

    • gui_keyboard_layout: wait_serial (wait serial expected) # wait_serial expected: "echo -e '[Layout]\nLayoutList=us,de' | sud...

    • gui_keyboard_layout: Failed (test died) # Test died: command 'test "$(cd ~user;ls e1*)" = "$(qvm-run -p wor...

  • system_tests_qwt_win10_seamless@hw13

    • windows_clipboard_and_filecopy: unnamed test (unknown)
    • windows_clipboard_and_filecopy: Failed (test died) # Test died: no candidate needle with tag(s) 'windows-Edge-address-...
  • system_tests_qwt_win11@hw13

    • windows_install: wait_serial (wait serial expected) # wait_serial expected: qr/dcWzE-\d+-/...

    • windows_install: Failed (test died + timed out) # Test died: command 'script -e -c 'bash -x /usr/bin/qvm-create-win...

  • system_tests_guivm_vnc_gui_interactive

    • guivm_manager: unnamed test (unknown)
    • guivm_manager: Failed (test died) # Test died: no candidate needle with tag(s) 'qubes-qube-manager' m...

Fixed failures

Compared to: https://openqa.qubes-os.org/tests/132953#dependencies

14 fixed
  • system_tests_basic_vm_qrexec_gui

  • system_tests_kde_gui_interactive

    • clipboard_and_web: unnamed test (unknown)

    • clipboard_and_web: Failed (test died) # Test died: no candidate needle with tag(s) 'qubes-website' matche...

    • clipboard_and_web: wait_serial (wait serial expected) # wait_serial expected: "lspci; echo 2E8vz-\$?-"...

  • system_tests_audio

  • system_tests_whonix@hw7

    • whonixcheck: fail (unknown) Whonixcheck for sys-whonix failed...

    • whonixcheck: unnamed test (unknown)

  • system_tests_guivm_vnc_gui_interactive

    • gui_filecopy: unnamed test (unknown)
    • gui_filecopy: Failed (test died) # Test died: no candidate needle with tag(s) 'files-work' matched...
  • system_tests_whonix

    • whonixcheck: fail (unknown) Whonixcheck for sys-whonix failed...

    • whonixcheck: unnamed test (unknown)

  • system_tests_suspend

    • suspend: unnamed test (unknown)
    • suspend: Failed (test died) # Test died: no candidate needle with tag(s) 'SUSPEND-FAILED' match...
  • system_tests_qrexec

Unstable tests

Performance Tests

Performance degradation:

17 performance degradations
  • whonix-gateway-17_exec: 8.14 :small_red_triangle_up: ( previous job: 6.82, degradation: 119.34%)
  • whonix-gateway-17_socket-root: 8.81 :small_red_triangle_up: ( previous job: 7.65, degradation: 115.11%)
  • dom0_root_seq1m_q8t1_read 3:read_bandwidth_kb: 203535.00 :small_red_triangle_up: ( previous job: 446963.00, degradation: 45.54%)
  • dom0_root_seq1m_q1t1_read 3:read_bandwidth_kb: 186051.00 :small_red_triangle_up: ( previous job: 294295.00, degradation: 63.22%)
  • dom0_root_seq1m_q1t1_write 3:write_bandwidth_kb: 50470.00 :small_red_triangle_up: ( previous job: 95454.00, degradation: 52.87%)
  • dom0_root_rnd4k_q32t1_read 3:read_bandwidth_kb: 21904.00 :small_red_triangle_up: ( previous job: 79803.00, degradation: 27.45%)
  • dom0_root_rnd4k_q32t1_write 3:write_bandwidth_kb: 5522.00 :small_red_triangle_up: ( previous job: 6149.00, degradation: 89.80%)
  • dom0_root_rnd4k_q1t1_write 3:write_bandwidth_kb: 2034.00 :small_red_triangle_up: ( previous job: 4826.00, degradation: 42.15%)
  • dom0_varlibqubes_rnd4k_q1t1_write 3:write_bandwidth_kb: 3259.00 :small_red_triangle_up: ( previous job: 4903.00, degradation: 66.47%)
  • fedora-41-xfce_root_seq1m_q1t1_read 3:read_bandwidth_kb: 275216.00 :small_red_triangle_up: ( previous job: 318716.00, degradation: 86.35%)
  • fedora-41-xfce_root_rnd4k_q32t1_write 3:write_bandwidth_kb: 2369.00 :small_red_triangle_up: ( previous job: 3599.00, degradation: 65.82%)
  • fedora-41-xfce_root_rnd4k_q1t1_read 3:read_bandwidth_kb: 7532.00 :small_red_triangle_up: ( previous job: 8485.00, degradation: 88.77%)
  • fedora-41-xfce_private_seq1m_q1t1_write 3:write_bandwidth_kb: 52078.00 :small_red_triangle_up: ( previous job: 61534.00, degradation: 84.63%)
  • fedora-41-xfce_private_rnd4k_q32t1_write 3:write_bandwidth_kb: 1002.00 :small_red_triangle_up: ( previous job: 2215.00, degradation: 45.24%)
  • fedora-41-xfce_volatile_seq1m_q8t1_write 3:write_bandwidth_kb: 105015.00 :small_red_triangle_up: ( previous job: 179949.00, degradation: 58.36%)
  • fedora-41-xfce_volatile_seq1m_q1t1_write 3:write_bandwidth_kb: 15085.00 :small_red_triangle_up: ( previous job: 17567.00, degradation: 85.87%)
  • fedora-41-xfce_volatile_rnd4k_q32t1_read 3:read_bandwidth_kb: 69001.00 :small_red_triangle_up: ( previous job: 79021.00, degradation: 87.32%)

Remaining performance tests:

55 tests
  • debian-12-xfce_exec: 7.12 :small_red_triangle_up: ( previous job: 7.12, degradation: 100.02%)
  • debian-12-xfce_exec-root: 28.76 :small_red_triangle_up: ( previous job: 28.65, degradation: 100.37%)
  • debian-12-xfce_socket: 8.95 :small_red_triangle_up: ( previous job: 8.60, degradation: 104.03%)
  • debian-12-xfce_socket-root: 8.29 :green_circle: ( previous job: 8.52, improvement: 97.25%)
  • debian-12-xfce_exec-data-simplex: 68.15 :green_circle: ( previous job: 71.62, improvement: 95.16%)
  • debian-12-xfce_exec-data-duplex: 69.68 :green_circle: ( previous job: 70.34, improvement: 99.06%)
  • debian-12-xfce_exec-data-duplex-root: 74.95 :green_circle: ( previous job: 82.72, improvement: 90.61%)
  • debian-12-xfce_socket-data-duplex: 167.51 :small_red_triangle_up: ( previous job: 156.96, degradation: 106.72%)
  • fedora-41-xfce_exec: 8.92 :green_circle: ( previous job: 9.27, improvement: 96.23%)
  • fedora-41-xfce_exec-root: 62.28 :small_red_triangle_up: ( previous job: 61.51, degradation: 101.26%)
  • fedora-41-xfce_socket: 7.91 :green_circle: ( previous job: 8.63, improvement: 91.70%)
  • fedora-41-xfce_socket-root: 8.63 :green_circle: ( previous job: 8.71, improvement: 99.16%)
  • fedora-41-xfce_exec-data-simplex: 74.04 :green_circle: ( previous job: 75.53, improvement: 98.02%)
  • fedora-41-xfce_exec-data-duplex: 64.32 :green_circle: ( previous job: 71.56, improvement: 89.88%)
  • fedora-41-xfce_exec-data-duplex-root: 107.58 :green_circle: ( previous job: 109.13, improvement: 98.58%)
  • fedora-41-xfce_socket-data-duplex: 140.61 :green_circle: ( previous job: 150.61, improvement: 93.36%)
  • whonix-gateway-17_exec-root: 39.45 :green_circle: ( previous job: 40.43, improvement: 97.56%)
  • whonix-gateway-17_socket: 6.96 :green_circle: ( previous job: 7.24, improvement: 96.25%)
  • whonix-gateway-17_exec-data-simplex: 78.60 :small_red_triangle_up: ( previous job: 78.32, degradation: 100.36%)
  • whonix-gateway-17_exec-data-duplex: 64.73 :green_circle: ( previous job: 76.65, improvement: 84.44%)
  • whonix-gateway-17_exec-data-duplex-root: 92.47 :small_red_triangle_up: ( previous job: 88.52, degradation: 104.46%)
  • whonix-gateway-17_socket-data-duplex: 167.95 :green_circle: ( previous job: 171.76, improvement: 97.78%)
  • whonix-workstation-17_exec: 8.08 :small_red_triangle_up: ( previous job: 7.67, degradation: 105.33%)
  • whonix-workstation-17_exec-root: 53.54 :green_circle: ( previous job: 58.26, improvement: 91.90%)
  • whonix-workstation-17_socket: 8.15 :green_circle: ( previous job: 8.19, improvement: 99.40%)
  • whonix-workstation-17_socket-root: 7.78 :green_circle: ( previous job: 8.13, improvement: 95.66%)
  • whonix-workstation-17_exec-data-simplex: 70.09 :green_circle: ( previous job: 74.99, improvement: 93.45%)
  • whonix-workstation-17_exec-data-duplex: 76.01 :small_red_triangle_up: ( previous job: 72.71, degradation: 104.54%)
  • whonix-workstation-17_exec-data-duplex-root: 98.99 :green_circle: ( previous job: 99.82, improvement: 99.17%)
  • whonix-workstation-17_socket-data-duplex: 151.81 :green_circle: ( previous job: 169.50, improvement: 89.56%)
  • dom0_root_seq1m_q8t1_write 3:write_bandwidth_kb: 236378.00 :green_circle: ( previous job: 129298.00, improvement: 182.82%)
  • dom0_root_rnd4k_q1t1_read 3:read_bandwidth_kb: 11964.00 :green_circle: ( previous job: 10795.00, improvement: 110.83%)
  • dom0_varlibqubes_seq1m_q8t1_read 3:read_bandwidth_kb: 393757.00 :green_circle: ( previous job: 382273.00, improvement: 103.00%)
  • dom0_varlibqubes_seq1m_q8t1_write 3:write_bandwidth_kb: 320763.00 :green_circle: ( previous job: 250795.00, improvement: 127.90%)
  • dom0_varlibqubes_seq1m_q1t1_read 3:read_bandwidth_kb: 436906.00 :small_red_triangle_up: ( previous job: 437636.00, degradation: 99.83%)
  • dom0_varlibqubes_seq1m_q1t1_write 3:write_bandwidth_kb: 172233.00 :small_red_triangle_up: ( previous job: 184752.00, degradation: 93.22%)
  • dom0_varlibqubes_rnd4k_q32t1_read 3:read_bandwidth_kb: 106175.00 :green_circle: ( previous job: 62195.00, improvement: 170.71%)
  • dom0_varlibqubes_rnd4k_q32t1_write 3:write_bandwidth_kb: 7386.00 :green_circle: ( previous job: 6479.00, improvement: 114.00%)
  • dom0_varlibqubes_rnd4k_q1t1_read 3:read_bandwidth_kb: 7704.00 :green_circle: ( previous job: 7669.00, improvement: 100.46%)
  • fedora-41-xfce_root_seq1m_q8t1_read 3:read_bandwidth_kb: 377185.00 :green_circle: ( previous job: 368309.00, improvement: 102.41%)
  • fedora-41-xfce_root_seq1m_q8t1_write 3:write_bandwidth_kb: 162140.00 :green_circle: ( previous job: 162081.00, improvement: 100.04%)
  • fedora-41-xfce_root_seq1m_q1t1_write 3:write_bandwidth_kb: 96619.00 :green_circle: ( previous job: 87940.00, improvement: 109.87%)
  • fedora-41-xfce_root_rnd4k_q32t1_read 3:read_bandwidth_kb: 84529.00 :green_circle: ( previous job: 82694.00, improvement: 102.22%)
  • fedora-41-xfce_root_rnd4k_q1t1_write 3:write_bandwidth_kb: 1691.00 :green_circle: ( previous job: 542.00, improvement: 311.99%)
  • fedora-41-xfce_private_seq1m_q8t1_read 3:read_bandwidth_kb: 344359.00 :small_red_triangle_up: ( previous job: 373957.00, degradation: 92.09%)
  • fedora-41-xfce_private_seq1m_q8t1_write 3:write_bandwidth_kb: 191848.00 :green_circle: ( previous job: 170062.00, improvement: 112.81%)
  • fedora-41-xfce_private_seq1m_q1t1_read 3:read_bandwidth_kb: 322242.00 :small_red_triangle_up: ( previous job: 334687.00, degradation: 96.28%)
  • fedora-41-xfce_private_rnd4k_q32t1_read 3:read_bandwidth_kb: 94503.00 :green_circle: ( previous job: 80283.00, improvement: 117.71%)
  • fedora-41-xfce_private_rnd4k_q1t1_read 3:read_bandwidth_kb: 8764.00 :green_circle: ( previous job: 7540.00, improvement: 116.23%)
  • fedora-41-xfce_private_rnd4k_q1t1_write 3:write_bandwidth_kb: 1200.00 :green_circle: ( previous job: 1130.00, improvement: 106.19%)
  • fedora-41-xfce_volatile_seq1m_q8t1_read 3:read_bandwidth_kb: 345380.00 :small_red_triangle_up: ( previous job: 369868.00, degradation: 93.38%)
  • fedora-41-xfce_volatile_seq1m_q1t1_read 3:read_bandwidth_kb: 321846.00 :small_red_triangle_up: ( previous job: 324737.00, degradation: 99.11%)
  • fedora-41-xfce_volatile_rnd4k_q32t1_write 3:write_bandwidth_kb: 6121.00 :green_circle: ( previous job: 5672.00, improvement: 107.92%)
  • fedora-41-xfce_volatile_rnd4k_q1t1_read 3:read_bandwidth_kb: 7611.00 :small_red_triangle_up: ( previous job: 7867.00, degradation: 96.75%)
  • fedora-41-xfce_volatile_rnd4k_q1t1_write 3:write_bandwidth_kb: 1951.00 :small_red_triangle_up: ( previous job: 1953.00, degradation: 99.90%)

qubesos-bot avatar Apr 16 '25 21:04 qubesos-bot

Something went wrong, multi-line notes don't work - looks like newline char is replaced with _. When you fix it, a test for this case would be useful too.

It appears that qubes_pure_string_safe_for_display counts space as a safe character for display but does not count \n\t as safe characters.

alimirjamali avatar Apr 17 '25 06:04 alimirjamali