Skip to content

feat: Support selecting compose files with -f#323

Merged
muchzill4 merged 3 commits into
arm:mainfrom
muchzill4:pick-compose-file
Jul 7, 2026
Merged

feat: Support selecting compose files with -f#323
muchzill4 merged 3 commits into
arm:mainfrom
muchzill4:pick-compose-file

Conversation

@muchzill4

Copy link
Copy Markdown
Contributor

Changes

  • Add -f/--file compose file selection to topo deploy, configure, ps, and stop.
  • Centralise compose file lookup and validation in internal/compose.
  • Preserve default lookup order of compose.yaml, then compose.yml, with a warning when both exist (similar to how docker compose behaves).

muchzill4 added 2 commits July 7, 2026 13:30
Signed-off-by: Bartek Mucha <bartosz.mucha@arm.com>
Signed-off-by: Bartek Mucha <bartosz.mucha@arm.com>
@muchzill4 muchzill4 requested a review from a team as a code owner July 7, 2026 13:14
Signed-off-by: Bartek Mucha <bartosz.mucha@arm.com>

@Luke-Parkin Luke-Parkin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thanks :)

Comment thread cmd/topo/root.go
panic(fmt.Sprintf("internal error: compose file flag not registered: %s", composeFileFlag))
}

if flag.Changed {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TIL.

flag.Changed is a bool; where if it's set (to anything other than it's default) it's true?

@muchzill4 muchzill4 merged commit 4668e2d into arm:main Jul 7, 2026
5 checks passed
@muchzill4 muchzill4 deleted the pick-compose-file branch July 7, 2026 14:32
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