The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix-edge3-worker-admin-exit-codes has failed.
Run started by GitHub user dkranchii (triggered by dkranchii).

Head commit for run:
20d94d19005b6eb9c771682c11e622429c0141fb / Deepak Kumar <[email protected]>
edge3: Exit non-zero when worker admin CLI commands fail

The add-worker-queues, remove-worker-queues, and set-worker-concurrency
subcommands raised bare `SystemExit` after catching a TypeError from the
model layer, which yields exit status 0. Shell scripts wrapping these
commands treated failures as success and continued as if the queue or
concurrency change had been applied.

Pass the exception message to SystemExit so the process exits with
status 1 and writes the failure to stderr, matching the file's own
convention for validation-error paths (see the explicit
`raise SystemExit("Error: ...")` calls in the same subcommands) and the
`raise SystemExit(str(e))` pattern already used elsewhere in the edge3
CLI. The model layer already logs the error before raising, so the
redundant logger.error call is removed.

Report URL: https://github.com/apache/airflow/actions/runs/29710914785

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to