disinto-admin/disinto
1
0
Fork 0
Code Issues 15 Pull requests 1 Projects Releases Packages Wiki Activity Actions

fix: restore write_result on pre-docker error paths in _launch_runner_docker
All checks were successful
ci/woodpecker/push/ci Pipeline was successful
Details
ci/woodpecker/pr/ci Pipeline was successful
Details

Browse source 
Prevents infinite retry loops when secret resolution or mount alias
validation fails before the docker run is attempted.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Claude 2026-04-16 00:33:55 +00:00
parent ef40433fff
commit 4487d1512c
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
docker/edge/dispatcher.sh
View file

@ -427,6 +427,7 @@ _launch_runner_docker() {
secret_val=$(load_secret "$secret") || true secret_val=$(load_secret "$secret") || true
if [ -z "$secret_val" ]; then if [ -z "$secret_val" ]; then
log "ERROR: Secret '${secret}' could not be resolved for action ${action_id}" log "ERROR: Secret '${secret}' could not be resolved for action ${action_id}"
write_result "$action_id" 1 "Secret not found: ${secret}"
return 1 return 1
fi fi
cmd+=(-e "${secret}=${secret_val}") cmd+=(-e "${secret}=${secret_val}")
@ -451,6 +452,7 @@ _launch_runner_docker() {
;; ;;
*) *)
log "ERROR: Unknown mount alias '${mount_alias}' for action ${action_id}" log "ERROR: Unknown mount alias '${mount_alias}' for action ${action_id}"
write_result "$action_id" 1 "Unknown mount alias: ${mount_alias}"
return 1 return 1
;; ;;
esac esac