DEVX-139: Fix clean_images URL-encoding for package names with slashes #218

Merged
emil merged 0 commits from DEVX-139-fix-clean-images-url-encoding into master 2026-07-14 12:34:37 +00:00
Owner

Package names with path prefixes (e.g. runner-images/ci-base) were not URL-encoded when calling the Gitea API, causing 404 errors for container packages. Also add docstring noting that filesystem blob GC is handled by the production VM cleanup script (cleanup_gitea.py).

Vikunja task: DEVX-139

Package names with path prefixes (e.g. runner-images/ci-base) were not URL-encoded when calling the Gitea API, causing 404 errors for container packages. Also add docstring noting that filesystem blob GC is handled by the production VM cleanup script (cleanup_gitea.py). Vikunja task: DEVX-139
emil changed title from DEVX-139: fix: URL-encode package names and versions in clean_images API calls to DEVX-139: Fix clean_images URL-encoding for package names with slashes 2026-07-14 12:21:05 +00:00
emil reviewed 2026-07-14 12:33:12 +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: OK
  • Tests: WARNING — source files changed but no test files updated
  • 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: OK - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-14 12:33:28 +00:00
Dismissed
kireto left a comment
Member

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil force-pushed DEVX-139-fix-clean-images-url-encoding from 1771ce1eb2 to 698d873310 2026-07-14 12:33:33 +00:00 Compare
emil reviewed 2026-07-14 12:34:19 +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: WARNING — source files changed but no test files updated
  • 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: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-14 12:34:33 +00:00
kireto left a comment
Member

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil merged commit db38453a54 into master 2026-07-14 12:34:37 +00:00
emil deleted branch DEVX-139-fix-clean-images-url-encoding 2026-07-14 12:34:37 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#218