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

qvm-backup doesn't check if there is enough space on dest if backing up to a VM #864

Closed
marmarek opened this Issue Mar 8, 2015 · 6 comments

Comments

Projects
None yet
5 participants
@marmarek
Member

marmarek commented Mar 8, 2015

Reported by joanna on 1 Jun 2014 09:51 UTC
None

Migrated-From: https://wiki.qubes-os.org/ticket/864

@marmarek marmarek added this to the Release 2 milestone Mar 8, 2015

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by marmarek on 1 Jun 2014 11:27 UTC
Yes, that's true. Any idea how it can check that, specially when custom store command used?

BTW When compression is enabled, backup will use about half or original data size. So it's not always a good idea to prevent starting backup when theoretically no enough space is available.

Member

marmarek commented Mar 8, 2015

Comment by marmarek on 1 Jun 2014 11:27 UTC
Yes, that's true. Any idea how it can check that, specially when custom store command used?

BTW When compression is enabled, backup will use about half or original data size. So it's not always a good idea to prevent starting backup when theoretically no enough space is available.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Modified by joanna on 30 Jun 2014 11:47 UTC

Member

marmarek commented Mar 8, 2015

Modified by joanna on 30 Jun 2014 11:47 UTC

@marmarek marmarek modified the milestones: Release 2.1 (post R2), Release 2 Mar 8, 2015

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Modified by woju on 30 Jun 2014 13:19 UTC

Member

marmarek commented Mar 8, 2015

Modified by woju on 30 Jun 2014 13:19 UTC

@nothingmuch

This comment has been minimized.

Show comment
Hide comment
@nothingmuch

nothingmuch Oct 18, 2016

I've had two issues that I think are related, as based on the comment about compression, I believe this is mostly about clearly reporting errors, not so much predicting them

  1. i tried backing up to FAT32 disk I had around, and at the 4G file limit the backup just failed, with no indication of what error occurred,
  2. Attempting a backup to ext4 failed immediately, again with no error reported, which chown -R user:user on the fresh ext4 partition rectified.

In fact, the second error has been plaguing me for a number of attempts over several weeks, and finally today when I accidentally tried a FAT32 disk and it partly worked, I only realized what was going on after i had partial success. I'm not sure why I assumed the backup would run as root in the target vm, but I do think there's room for UI improvements. I did not have time to debug the actual backup program and find out why errors were being ellided, unfortunately.

I've had two issues that I think are related, as based on the comment about compression, I believe this is mostly about clearly reporting errors, not so much predicting them

  1. i tried backing up to FAT32 disk I had around, and at the 4G file limit the backup just failed, with no indication of what error occurred,
  2. Attempting a backup to ext4 failed immediately, again with no error reported, which chown -R user:user on the fresh ext4 partition rectified.

In fact, the second error has been plaguing me for a number of attempts over several weeks, and finally today when I accidentally tried a FAT32 disk and it partly worked, I only realized what was going on after i had partial success. I'm not sure why I assumed the backup would run as root in the target vm, but I do think there's room for UI improvements. I did not have time to debug the actual backup program and find out why errors were being ellided, unfortunately.

@unman

This comment has been minimized.

Show comment
Hide comment
@unman

unman Apr 14, 2017

Member

@andrewdavidwong Confirmed issue still arises in 3.2 milestone

Member

unman commented Apr 14, 2017

@andrewdavidwong Confirmed issue still arises in 3.2 milestone

@woju

This comment has been minimized.

Show comment
Hide comment
@woju

woju Apr 18, 2018

Member

This issue is abandoned, so I'm going to close it. If that's a mistake, feel free to reopen.
@marmarek @andrewdavidwong

Member

woju commented Apr 18, 2018

This issue is abandoned, so I'm going to close it. If that's a mistake, feel free to reopen.
@marmarek @andrewdavidwong

@woju woju closed this Apr 18, 2018

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