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
Store snapshot statistics & print snapshot size #4705
Store snapshot statistics & print snapshot size #4705
Conversation
Does the |
@MichaelEischer Is there a reason why your implementation deviates from |
Differences to #693 (comment):
Although, the more precise description would be that the summary contains the fields from I thought about omitting
Yes, it even includes the full statistics information stored in the snapshot. |
If you name them like this you make the snapshot format incompatible to what rustic implemented some 2 years ago. I don't care too much if you omit the other fields, but same things should be named the same, IMO. |
The value describes how much data was added after compression. Previously, it was only available in the text output.
add8958
to
6813959
Compare
Just for the record this is only meaningful until you start deleting other snapshots which share data, right? |
No. It is as meaningful as it is saving the output of |
Right, that's what it seemed like. It's useful in the moment, but if you have e.g. auto cleanup of older backups it quickly becomes stale. Can still be useful of course, but not the same as e.g. TimeMachine's "calculatedrift" option (which would presumably be a lot more expensive to run). |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
I've renamed the attribute to In general, we're interested in a friendly coexistence with rustic. But it should also be clear that we won't always agree how things should be implemented. Nevertheless there's still value in trying to maintain compatibility where it makes sense. That judgement will still be up to us, but that doesn't mean the we do X just because rustic implemented Y.
Let me add just one thing here: the PR was intentionally still marked as draft as the naming issue was not yet resolved. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Thanks for changing this. I think this helps user who use both restic and rustic. And it's about our users what we are doing, right?
Good to hear. I am also very interested in a friendly coexistence. This is all free open source software! |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
created using this or a future restic version. For this, the `backup` command | ||
now stores the backup summary statistics in the snapshot. | ||
|
||
The text output of the `snapshots` command only shows the snapshot size. The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What exactly of all the discussed options is the “snapshot size” here?
Bytes added to the repo when this backup was created?
If so, fine with me, but I would still love to also see the “amount of data backed up” (what others call restore size) for a quick double-check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is total_bytes_processed
which IMO should be the restore size.
What does this PR change? What problem does it solve?
Store the backup statistics in a snapshot. Sample output:
In addition the
snapshots
command now also prints the snapshot size (based ontotal_bytes_processed
).Was the change previously discussed in an issue or on the forum?
Fixes #693
Checklist
changelog/unreleased/
that describes the changes for our users (see template).gofmt
on the code in all commits.