Skip to content
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

Backup size is severely overestimated #5699

Open
hexagonrecursion opened this issue Feb 29, 2020 · 5 comments
Open

Backup size is severely overestimated #5699

hexagonrecursion opened this issue Feb 29, 2020 · 5 comments
Labels
affects-4.1 This issue affects Qubes OS 4.1. affects-4.2 This issue affects Qubes OS 4.2. C: core C: storage needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@hexagonrecursion
Copy link

Qubes OS version
Qubes release 4.0 (R4.0)

Affected component(s) or functionality
"Backup Qubes" wizard

Brief summary
On the page where you select which qubes to include in the backup the size of each qube is overestimated.

To Reproduce

  1. Start the "Backup Qubes" wizard
  2. On the initial page note the total backup size
  3. Clink next 2 times
  4. Compare total size to the one reported on the summary page

Expected behavior
A reasonably accurate estimate of the contribution of each qube to the backup size and the total backup size should be displayed.

Actual behavior
The size appears to be severely overestimated. In particular:

  1. dom0 is reported to be tens of GiB in size
  2. Any qube that is running at the time is reported to have a much bigger size than it actually has

Additional context
In case it matters: my primary pool is a btrfs-backed file-reflink. I understand that it is impossible to predict the effectiveness of backup compression. This issue in not about compression, but specifically about two wizard pages disagreeing on the backup size.

Relevant documentation you've consulted
none

Related, non-duplicate issues
none

@hexagonrecursion hexagonrecursion added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Feb 29, 2020
@hexagonrecursion hexagonrecursion changed the title Backup size severely overestimated Backup size is severely overestimated Feb 29, 2020
@andrewdavidwong andrewdavidwong added this to the Release 4.0 updates milestone Mar 1, 2020
@marmarek
Copy link
Member

marmarek commented Mar 1, 2020

I guess it counts the size together with all previous and temporary snapshots, by summing them...

@rustybird
Copy link

rustybird commented Mar 2, 2020

Any downsides to changing get_disk_utilization() in qubesadmin.vm (and also the equivalent one in qubes.storage, which currently has no callers) to only sum up save_on_stop volumes?

Edit: On second thought, that would make it return 0 for all DisposableVMs, which is correct in backup VM selection but unhelpful in the VM tray widget and in qvm-ls.

@rustybird
Copy link

The way too large reported size of dom0 is because shutil.disk_usage() is called on the home directory, but this actually returns the usage of the whole mountpoint.

@marmarek marmarek self-assigned this Mar 5, 2020
@andrewdavidwong andrewdavidwong added the eol-4.0 Closed since Qubes 4.0 has been EOL for over one year label Aug 5, 2023
@github-actions
Copy link

github-actions bot commented Aug 5, 2023

This issue is being closed because:

If anyone believes that this issue should be reopened and reassigned to an active milestone, please leave a brief comment.
(For example, if a bug still affects Qubes OS 4.1, then the comment "Affects 4.1" will suffice.)

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 5, 2023
@rustybird
Copy link

Affects 4.2

@DemiMarie DemiMarie reopened this Aug 14, 2023
@DemiMarie DemiMarie added affects-4.1 This issue affects Qubes OS 4.1. affects-4.2 This issue affects Qubes OS 4.2. and removed eol-4.0 Closed since Qubes 4.0 has been EOL for over one year labels Aug 14, 2023
@andrewdavidwong andrewdavidwong removed this from the Release 4.0 updates milestone Aug 14, 2023
@andrewdavidwong andrewdavidwong added needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. C: core C: storage and removed C: manager/widget labels Aug 14, 2023
@marmarek marmarek removed their assignment Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.1 This issue affects Qubes OS 4.1. affects-4.2 This issue affects Qubes OS 4.2. C: core C: storage needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

No branches or pull requests

5 participants