Skip to content

Commit 5abf55d

Browse files
committed
adapted server service accounts list command to align to expectations
1 parent fbaf227 commit 5abf55d

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

  • internal/cmd/server/service-account/list

internal/cmd/server/service-account/list/list.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,6 @@ func NewCmd(params *types.CmdParams) *cobra.Command {
7878
return fmt.Errorf("list service accounts: %w", err)
7979
}
8080
serviceAccounts := *resp.Items
81-
if len(serviceAccounts) == 0 {
82-
params.Printer.Info("No service accounts found for server %s\n", serverName)
83-
return nil
84-
}
8581

8682
if model.Limit != nil && len(serviceAccounts) > int(*model.Limit) {
8783
serviceAccounts = serviceAccounts[:int(*model.Limit)]
@@ -133,14 +129,18 @@ func buildRequest(ctx context.Context, model *inputModel, apiClient *iaas.APICli
133129

134130
func outputResult(p *print.Printer, outputFormat, serverId, serverName string, serviceAccounts []string) error {
135131
return p.OutputResult(outputFormat, serviceAccounts, func() error {
132+
if len(serviceAccounts) == 0 {
133+
p.Info("No service accounts found for server %s\n", serverName)
134+
return nil
135+
}
136136
table := tables.NewTable()
137137
table.SetHeader("SERVER ID", "SERVER NAME", "SERVICE ACCOUNT")
138138
for i := range serviceAccounts {
139139
table.AddRow(serverId, serverName, serviceAccounts[i])
140140
}
141141
err := table.Display(p)
142142
if err != nil {
143-
return fmt.Errorf("rednder table: %w", err)
143+
return fmt.Errorf("render table: %w", err)
144144
}
145145
return nil
146146
})

0 commit comments

Comments
 (0)