Skip to content

feat(gpbackup): add display-report subcommand - #46

Merged
adityatoshniwal merged 3 commits into
mainfrom
display-report
Aug 24, 2026
Merged

feat(gpbackup): add display-report subcommand#46
adityatoshniwal merged 3 commits into
mainfrom
display-report

Conversation

@adityatoshniwal

Copy link
Copy Markdown
Contributor

Prints a stored backup's report by timestamp, pulling it from plugin storage when the local copy is gone. --format=json returns structured fields plus a nested object_counts.

@adityatoshniwal
adityatoshniwal requested review from a team, adam8157 and haolinw August 19, 2026 13:15
Prints a stored backup's report by timestamp, pulling it from plugin
storage when the local copy is gone. --format=json returns structured
fields plus a nested object_counts.
Comment thread backup/display_report.go
Comment thread backup/display_report.go Outdated
Comment thread backup/display_report.go Outdated
…g config file

backup_error was read from the backup's config.yaml, which display-report
would fatal-fetch (or fatal outright, for a local backup with no plugin)
if the file was missing locally, even though the report file itself
already has this text and text-mode never needed the config file at all.
Parse backup_error out of the report's own "backup error:" section
instead, dropping the config file dependency entirely.

@haolinw haolinw left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@adityatoshniwal
adityatoshniwal merged commit aeab503 into main Aug 24, 2026
4 checks passed
@adityatoshniwal
adityatoshniwal deleted the display-report branch August 24, 2026 07:22
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.

2 participants