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 progress miscalculated when compression is enabled #997

Open
marmarek opened this Issue May 13, 2015 · 4 comments

Comments

Projects
None yet
4 participants
@marmarek
Member

marmarek commented May 13, 2015

Backup progress shows progress as "size of output written"/"total size of selected VMs". When compression is enabled, this is obviously wrong, because at the end "size of output written" is not the same as original size.

This may be not easy to fix (while preserving frequent enough progress updates), because the first step applies the compression (tar -cz...) and we don't have direct progress reporting there. So the most obvious fix would mean that progress would be updated only when going to the next VM.

@marmarek marmarek added this to the Release 3.0 milestone May 14, 2015

@v6ak

This comment has been minimized.

Show comment
Hide comment
@v6ak

v6ak May 21, 2015

Maybe adding some process that counts all the bytes and stores the value somewhere would help. I mean something like this:

tar -c | watch-progress /tmp/progressfile | gzip

As far as I remember, dd can do this (it prints statistics when USR1 signal is received), but writing a custom simple proxy might be easier.

(I don't care about this issue much, so I won't send a patch. I just wanted to share an idea of a hopefully easy fix.)

v6ak commented May 21, 2015

Maybe adding some process that counts all the bytes and stores the value somewhere would help. I mean something like this:

tar -c | watch-progress /tmp/progressfile | gzip

As far as I remember, dd can do this (it prints statistics when USR1 signal is received), but writing a custom simple proxy might be easier.

(I don't care about this issue much, so I won't send a patch. I just wanted to share an idea of a hopefully easy fix.)

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek May 21, 2015

Member

Backup tool monitor all the process involved in backup creation, so when
changed from "tar -c --use-compression-program=gzip" to "tar | something
| gzip" it means two additional processes to watch. I'm not saying it
isn't possible, just more work than it may look as. And while this has
"minor" priority...

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Member

marmarek commented May 21, 2015

Backup tool monitor all the process involved in backup creation, so when
changed from "tar -c --use-compression-program=gzip" to "tar | something
| gzip" it means two additional processes to watch. I'm not saying it
isn't possible, just more work than it may look as. And while this has
"minor" priority...

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

@marmarek marmarek modified the milestones: Release 3.1, Release 3.0 May 27, 2015

@andrewdavidwong

This comment has been minimized.

Show comment
Hide comment
@andrewdavidwong

andrewdavidwong Jun 3, 2015

Member

This seems very minor, IMHO. I would be surprised if many people even notice this, and among those who do notice it, I would be surprised if any were bothered by it. All I observed is that my backups seem to finish sooner than expected, which is actually rather nice. :P

Member

andrewdavidwong commented Jun 3, 2015

This seems very minor, IMHO. I would be surprised if many people even notice this, and among those who do notice it, I would be surprised if any were bothered by it. All I observed is that my backups seem to finish sooner than expected, which is actually rather nice. :P

@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, but it is, as you say, minor in extreme

Member

unman commented Apr 14, 2017

@andrewdavidwong Confirmed issue still arises in 3.2 milestone, but it is, as you say, minor in extreme

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