Skip to content

feat: add containers list command#66

Merged
yaseenisolated merged 4 commits intomainfrom
yaseen/containers-list-command
May 5, 2026
Merged

feat: add containers list command#66
yaseenisolated merged 4 commits intomainfrom
yaseen/containers-list-command

Conversation

@yaseenisolated
Copy link
Copy Markdown
Contributor

Summary

  • New cerebrium containers list APP_NAME command. Lists recent containers for an app along with their state.
  • Reuses the dashboard's v2 endpoint (/v2/projects/{p}/apps/{a}/containers), so terminating pods are surfaced too — they show up as TERMINATING.
  • containers is registered as a command group so we can hang future subcommands off it.

Test plan

  • cerebrium containers list my-app against a project with running containers
  • Confirm a pod that is being torn down renders as TERMINATING
  • cerebrium containers list missing-app returns the empty-state message cleanly
  • cerebrium containers --help and cerebrium containers list --help render correctly

🤖 Generated with Claude Code

yaseenisolated and others added 3 commits May 5, 2026 17:25
Adds a `cerebrium containers list APP_NAME` command that hits the same
v2 endpoint the dashboard uses, surfacing recent containers along with
their state. Pods being torn down are shown as TERMINATING.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Re-runs `make generate` so the auto-generated mocks file matches what
mockery would produce, instead of being hand-edited.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The v2 endpoint already returns the build id for each container — add a
BUILD column so users can tell which deploy a container came from.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@yaseenisolated yaseenisolated requested a review from wesrobin May 5, 2026 15:37
Comment thread internal/api/mock/client_gen.go
Comment thread internal/api/types.go Outdated
Copy link
Copy Markdown
Contributor

@jonoirwinrsa jonoirwinrsa left a comment

Choose a reason for hiding this comment

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

smol comment about comments, otherwise LGTM

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@yaseenisolated yaseenisolated enabled auto-merge May 5, 2026 15:50
@yaseenisolated yaseenisolated added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit 9848fa6 May 5, 2026
4 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