Skip to content

fix(Run Commands Service): print valid JSON/YAML output for list cmds#1378

Merged
rubenhoenle merged 2 commits intostackitcloud:mainfrom
j1n-o9r:fix/STACKITCLI-385-run-command-json-list-output
Apr 27, 2026
Merged

fix(Run Commands Service): print valid JSON/YAML output for list cmds#1378
rubenhoenle merged 2 commits intostackitcloud:mainfrom
j1n-o9r:fix/STACKITCLI-385-run-command-json-list-output

Conversation

@j1n-o9r
Copy link
Copy Markdown
Contributor

@j1n-o9r j1n-o9r commented Apr 22, 2026

Description

relates to STACKITCLI-385 / #893

Testing Instructions

Create a new server with name yyy and id zzz

  • stackit server command list --server-id zzz -> Expected: No commands found for server yyy
  • stackit server command list --server-id zzz --output-format json -> Should output valid JSON
  • stackit server command list --server-id zzz --output-format yaml -> Should output valid YAML

Here the response needs to be mocked:

  • stackit server command template list -> Expected: No templates found
  • stackit server command template list --output-format json -> Should output valid JSON
  • stackit server command template list--output-format yaml -> Should output valid YAML

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see e.g. here)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@j1n-o9r j1n-o9r changed the title Fix/stackitcli 385 run command json list output fix(Run Commands Service): print valid JSON/YAML output for list cmds Apr 22, 2026
@j1n-o9r j1n-o9r marked this pull request as ready for review April 27, 2026 09:13
@j1n-o9r j1n-o9r requested a review from a team as a code owner April 27, 2026 09:13
@rubenhoenle rubenhoenle merged commit e17c7ff into stackitcloud:main Apr 27, 2026
3 checks passed
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