Skip to content

Show command on failure#1600

Open
xverges wants to merge 1 commit intodapr:masterfrom
xverges:xv-show-failing-command
Open

Show command on failure#1600
xverges wants to merge 1 commit intodapr:masterfrom
xverges:xv-show-failing-command

Conversation

@xverges
Copy link

@xverges xverges commented Mar 1, 2026

Description

  • Make failures on subcommands more informative by displaying the command that triggered them and the exit code
  • Prevent potential deadlock by using exec.cmd.Output rather than sequentially reading stdout and stderr
./dapr init --container-runtime podman
⌛  Making the jump to hyperspace...
ℹ️  Container images will be pulled from Docker Hub
ℹ️  Installing runtime version 1.17.0
❌  Downloading binaries and setting up components...
❌  error running "podman run --name dapr_placement --restart always -d --entrypoint ./placement -p 50005:50005 -p 58080:8080 -p 59090:9090 docker.io/daprio/dapr:1.17.0" (exit code 126): Error: something went wrong with the request: "proxy already running\n"

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1594

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Xavier Vergés <[email protected]>
@xverges xverges requested review from a team as code owners March 1, 2026 10:21
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.

dapr cli could be more helpful reporting docker/podman errors

1 participant