From c6521cf10819d461b107a38063967c849935c67a Mon Sep 17 00:00:00 2001 From: Deepti-yb Date: Thu, 19 Sep 2024 05:05:00 +0000 Subject: [PATCH] [PLAT-15368][YBA CLI]Backup list print "No more backup present" after printing all the backups in -o pretty/json Summary: Don't print `No more backups` for output types json and pretty Test Plan: Manually test the workflow Reviewers: skurapati Reviewed By: skurapati Subscribers: yugaware Differential Revision: https://phorge.dev.yugabyte.com/D38195 --- managed/yba-cli/cmd/backup/list_backup.go | 4 +++- managed/yba-cli/cmd/backup/restore/list_restore.go | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/managed/yba-cli/cmd/backup/list_backup.go b/managed/yba-cli/cmd/backup/list_backup.go index 81cbffa1a797..48bb5d297131 100644 --- a/managed/yba-cli/cmd/backup/list_backup.go +++ b/managed/yba-cli/cmd/backup/list_backup.go @@ -107,7 +107,9 @@ var listBackupCmd = &cobra.Command{ // Check if there are more pages hasNext := r.GetHasNext() if !hasNext { - logrus.Infoln("No more backups present\n") + if util.IsOutputType(formatter.TableFormatKey) { + logrus.Infoln("No more backups present\n") + } break } diff --git a/managed/yba-cli/cmd/backup/restore/list_restore.go b/managed/yba-cli/cmd/backup/restore/list_restore.go index e73d03e7a780..cabcd0089371 100644 --- a/managed/yba-cli/cmd/backup/restore/list_restore.go +++ b/managed/yba-cli/cmd/backup/restore/list_restore.go @@ -96,7 +96,9 @@ var listRestoreCmd = &cobra.Command{ // Check if there are more pages hasNext := r.GetHasNext() if !hasNext { - logrus.Infoln("No more restores present\n") + if util.IsOutputType(formatter.TableFormatKey) { + logrus.Infoln("No more restores present\n") + } break }