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

chore: Parse Kopia restore result #3011

Merged
merged 1 commit into from
Feb 14, 2025
Merged

chore: Parse Kopia restore result #3011

merged 1 commit into from
Feb 14, 2025

Conversation

e-sumin
Copy link
Contributor

@e-sumin e-sumin commented Aug 2, 2024

Change Overview

We already have parsing of Kopia progress lines, this PR introduces parsing of final stats of restore operation.

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test
  • 🏗️ Build

Issues

  • fixes #issue-number

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

@e-sumin e-sumin force-pushed the parse-kopia-result branch 2 times, most recently from fa1d77e to 63e05b3 Compare August 7, 2024 09:40
Copy link
Contributor

This PR is marked as stale due to inactivity. Add a new comment to reactivate it. CC @kanisterio/maintainers

Comment about only using last matching result

Signed-off-by: Daniil Fedotov <[email protected]>

Signed-off-by: Daniil Fedotov <[email protected]>
@hairyhum hairyhum added the kueue label Feb 14, 2025
@mergify mergify bot merged commit eed5816 into master Feb 14, 2025
24 checks passed
@mergify mergify bot deleted the parse-kopia-result branch February 14, 2025 21:12
aaron-kasten pushed a commit to aaron-kasten/kanister that referenced this pull request Feb 21, 2025
Signed-off-by: Daniil Fedotov <[email protected]>
Signed-off-by: Aaron Alpar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants