DEVX-88: Fix clean_images delete URL and error handling #140

Merged
emil merged 0 commits from DEVX-88-fix-clean-images into master 2026-06-27 21:40:50 +00:00
Owner

Problem

The cleanup step in build-images.yml was failing silently — all 285 delete attempts failed over 5 minutes but the step passed with exit code 0.

Root Cause

  1. Wrong delete URL: Code used /packages/{owner}/{name}/{version} but Gitea API requires /packages/{owner}/{type}/{name}/{version} — missing the container type segment
  2. Silent failure: Script exited 0 even when all deletes failed
  3. No retry: Transient 5xx/timeout failures were not retried

Fix

  • Add container type to delete URL
  • Exit non-zero when any deletes fail
  • Add retry with exponential backoff for 5xx and network errors
  • Treat 404 as success (already deleted)

Closes DEVX-88

## Problem The cleanup step in build-images.yml was failing silently — all 285 delete attempts failed over 5 minutes but the step passed with exit code 0. ## Root Cause 1. **Wrong delete URL**: Code used `/packages/{owner}/{name}/{version}` but Gitea API requires `/packages/{owner}/{type}/{name}/{version}` — missing the `container` type segment 2. **Silent failure**: Script exited 0 even when all deletes failed 3. **No retry**: Transient 5xx/timeout failures were not retried ## Fix - Add `container` type to delete URL - Exit non-zero when any deletes fail - Add retry with exponential backoff for 5xx and network errors - Treat 404 as success (already deleted) Closes DEVX-88
emil reviewed 2026-06-27 21:39:59 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil merged commit 598238e4d6 into master 2026-06-27 21:40:50 +00:00
emil deleted branch DEVX-88-fix-clean-images 2026-06-27 21:40:50 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#140