fix: use pre-installed yamllint from runner-tools #217

claude · 2026-02-22T14:31:56+01:00

claude commented

2026-02-22 14:31:56 +01:00

yamllint теперь предустановлен в runner-tools образе.
Убирает хрупкий pip install который молча ломался из-за PEP 668.

yamllint теперь предустановлен в runner-tools образе. Убирает хрупкий pip install который молча ломался из-за PEP 668.

claude added 1 commit 2026-02-22 14:31:56 +01:00

fix: use pre-installed yamllint from runner-tools image

AI Review / AI Code Review (pull_request) Successful in 2s

Details

PR Checks / Validate & Security Scan (pull_request) Successful in 12s

Details

3aca0fcea7

yamllint is now pre-installed in the runner-tools Docker image.
Removes fragile pip install step that was failing silently.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude commented

2026-02-22 14:32:08 +01:00

AI Code Review

Спасибо за PR. Это CI/CD изменение, а не K8s манифесты, поэтому мой стандартный чек-лист (PSS, NetworkPolicy, RBAC, ресурсы) здесь не применим.

Краткий review:

✅ Изменение корректно:

Удаляется хрупкий pip install --break-system-packages (обходит PEP 668, может молча сломаться)
yamllint уже в runner-tools образе → опираемся на готовый артефакт
Упрощает pipeline, снижает entropy

⚠️ Вопрос к уточнению (не блокирует, но рекомендую):

Проверить, что runner-tools образ всегда содержит yamllint нужной версии (версионирование образа?)
Если версия критична → документировать минимальную версию runner-tools в README или .gitea/workflows/

Вывод: Merge-ready. Хорошее решение для избежания хрупких зависимостей в CI.

Automated review by Claude Haiku | advisory, does not block merge

## AI Code Review Спасибо за PR. Это **CI/CD изменение**, а не K8s манифесты, поэтому мой стандартный чек-лист (PSS, NetworkPolicy, RBAC, ресурсы) здесь не применим. **Краткий review:** ✅ **Изменение корректно:** - Удаляется хрупкий `pip install --break-system-packages` (обходит PEP 668, может молча сломаться) - `yamllint` уже в `runner-tools` образе → опираемся на готовый артефакт - Упрощает pipeline, снижает entropy ⚠️ **Вопрос к уточнению** (не блокирует, но рекомендую): - Проверить, что `runner-tools` образ всегда содержит `yamllint` нужной версии (версионирование образа?) - Если версия критична → документировать минимальную версию `runner-tools` в README или `.gitea/workflows/` **Вывод:** Merge-ready. Хорошее решение для избежания хрупких зависимостей в CI. --- _Automated review by Claude Haiku | advisory, does not block merge_

claude merged commit 3f7f5cf49f into main

2026-02-22 14:34:14 +01:00

claude referenced this issue from a commit

2026-02-22 14:34:16 +01:00

Merge pull request 'fix: use pre-installed yamllint from runner-tools' (#217) from fix/use-preinstalled-yamllint into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/k8s-apps#217