Skip to content

feat(container): add registry list command and public link on container create#614

Merged
fabienfleureau merged 2 commits intomainfrom
qovery-dev-skill
Mar 9, 2026
Merged

feat(container): add registry list command and public link on container create#614
fabienfleureau merged 2 commits intomainfrom
qovery-dev-skill

Conversation

@fabienfleureau
Copy link
Collaborator

@fabienfleureau fabienfleureau commented Mar 3, 2026

Summary

  • Add qovery container registry list subcommand — lists all container registries for an organization with --organization and --json flags
  • Add --json flag to qovery container create — JSON output includes id, name, and public_link (when a port is configured and a link is available after creation)
  • Replace hand-rolled fmt.Sprintf JSON string in container create with encoding/json + anonymous struct to avoid escaping issues on names containing special characters

Copilot AI review requested due to automatic review settings March 3, 2026 15:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds two new features to the Qovery CLI: a qovery container registry list subcommand that lists all container registries for an organization, and a --json flag to qovery container create that outputs structured JSON including the service id, name, and optionally public_link after creation. It also replaces hand-rolled JSON strings with encoding/json marshalling in the create command.

Changes:

  • Add qovery container registry list subcommand with --organization and --json flags for tabular and JSON output
  • Add --json flag to qovery container create, outputting id, name, and public_link using encoding/json instead of manual string formatting
  • Add qovery container registry parent command as the new subcommand group host

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
cmd/container_registry.go New parent registry subcommand under container, following the same pattern as container_domain.go and container_env.go
cmd/container_registry_list.go New list subcommand that fetches and displays container registries in tabular or JSON format
cmd/container_create.go New create subcommand for container services with --json output including a public_link field

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…er create

- Add `qovery container registry` subcommand with `list` subcommand
  supporting `--organization` and `--json` flags
- Add `--json` flag to `qovery container create`; JSON output now includes
  `public_link` when a port is configured and a link is available
- Use `encoding/json` for JSON output in container create (replaces
  hand-rolled fmt.Sprintf to avoid escaping issues)
- Guard registry.Name (a *string) before dereferencing in both the table
  loop and the JSON output function in container_registry_list.go
- Check json.Marshal error in container_create.go instead of silently
  discarding it, consistent with the rest of the codebase
@fabienfleureau fabienfleureau merged commit 4d6a2c3 into main Mar 9, 2026
6 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.

3 participants