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

Add restic stats command #259

Merged
merged 2 commits into from
Sep 5, 2019
Merged

Add restic stats command #259

merged 2 commits into from
Sep 5, 2019

Conversation

DeepikaDixit
Copy link
Contributor

Change Overview

Add Restic stats command to query stats given a snapshot ID

Pull request type

Please check the type of change your PR introduces:

  • Work in Progress
  • Refactoring (no functional changes, no api changes)
  • Trival/Minor
  • Bugfix
  • Feature
  • Documentation

Issues

Test Plan

  • Manual
  • Unit test
  • E2E

Restic Command

$ restic stats 7e17e764
repository c6746ff5 opened successfully, password is correct
found 2 old cache directories in /Users/deepikadixit/Library/Caches/restic, pass --cleanup-cache to remove them
scanning...
Stats for 7e17e764 in restore-size mode:
  Total File Count:   9
        Total Size:   10.322 KiB

@SupriyaKasten
Copy link
Contributor

Looks good to me @DeepikaDixit, are we planning to add unit test in restic_test.go in k10 ?

@DeepikaDixit
Copy link
Contributor Author

Looks good to me @DeepikaDixit, are we planning to add unit test in restic_test.go in k10 ?

Yes @SupriyaKasten. We have some integration tests there, I will update those

@DeepikaDixit DeepikaDixit merged commit bac09fd into master Sep 5, 2019
@DeepikaDixit DeepikaDixit deleted the restic-stats branch September 5, 2019 17:37
@DeepikaDixit DeepikaDixit mentioned this pull request Sep 5, 2019
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants