Skip to content

Commit

Permalink
[PLAT-15370][YBA CLI]backup list table output contains base_backup_uu…
Browse files Browse the repository at this point in the history
…id and backup_uuid

Summary: Since only full backups are displayed in the `./yba backup list` command, remove base backup uuid from table output format

Test Plan: Manually tested the code

Reviewers: skurapati

Reviewed By: skurapati

Subscribers: yugaware

Differential Revision: https://phorge.dev.yugabyte.com/D38198
  • Loading branch information
Deepti-yb committed Sep 19, 2024
1 parent 29c56ed commit 784160c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion managed/yba-cli/internal/formatter/backup/backup.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import (
)

const (
defaultBackupListing = "table {{.BackupUUID}}\t{{.BaseBackupUUID}}\t{{.Universe}}" +
defaultBackupListing = "table {{.BackupUUID}}\t{{.Universe}}" +
"\t{{.StorageConfig}}\t{{.StorageConfigType}}\t{{.BackupType}}" +
"\t{{.State}}\t{{.CompletionTime}}"

Expand Down
3 changes: 1 addition & 2 deletions managed/yba-cli/internal/formatter/backup/backup_full.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,7 @@ import (
const (
// Backup details
defaultFullBackupGeneral = "table {{.BackupUUID}}\t{{.BackupType}}\t{{.State}}"
backupDetails1 = "table {{.Universe}}\t{{.ScheduleName}}" +
"\t{{.BaseBackupUUID}}\t{{.HasIncrementalBackups}}"
backupDetails1 = "table {{.Universe}}\t{{.ScheduleName}}\t{{.HasIncrementalBackups}}"
backupDetails2 = "table {{.StorageConfig}}\t{{.StorageConfigType}}"
backupDetails3 = "table {{.CreateTime}}\t{{.CompletionTime}}\t{{.ExpiryTime}}"
)
Expand Down

0 comments on commit 784160c

Please sign in to comment.