oblachno-oss/devx
Public Access
3
0
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects Releases 10 Wiki Activity

DEVX-19: Revert --ignore-installed, use retry approach for pip #33

Merged
emil merged 0 commits from DEVX-19-fix-pip-ignore-installed into master 2026-06-23 23:01:33 +00:00
Conversation 5 Commits - Files Changed -
emil commented 2026-06-23 22:40:59 +00:00
Owner
Copy Link
Copy Source

Summary

First attempt uses --break-system-packages only (preserves ansible-galaxy binary). If that fails (debian packages without RECORD files), retry with --ignore-installed. This fixes molecule test setup failures where ansible-galaxy wasn't found after --ignore-installed broke the install.

Closes DEVX-19

## Summary First attempt uses --break-system-packages only (preserves ansible-galaxy binary). If that fails (debian packages without RECORD files), retry with --ignore-installed. This fixes molecule test setup failures where ansible-galaxy wasn't found after --ignore-installed broke the install. Closes DEVX-19
emil reviewed 2026-06-23 22:41:32 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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 reviewed 2026-06-23 22:52:41 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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 reviewed 2026-06-23 22:57:27 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: retry logic is clean, shutil.which fallback is idiomatic. Security: no secrets exposed. Tests: 100% coverage with new tests. Docs: internal fix, no doc changes needed.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: retry logic is clean, shutil.which fallback is idiomatic. Security: no secrets exposed. Tests: 100% coverage with new tests. Docs: internal fix, no doc changes needed.
emil reviewed 2026-06-23 22:57:33 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: retry logic is clean, shutil.which fallback is idiomatic. Security: no secrets exposed. Tests: 100% coverage with new tests. Docs: internal fix, no doc changes needed.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: retry logic is clean, shutil.which fallback is idiomatic. Security: no secrets exposed. Tests: 100% coverage with new tests. Docs: internal fix, no doc changes needed.
emil added the ready-to-merge label 2026-06-23 22:57:37 +00:00
emil reviewed 2026-06-23 23:00:33 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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 034cbde2f7 into master 2026-06-23 23:01:33 +00:00
emil deleted branch DEVX-19-fix-pip-ignore-installed 2026-06-23 23:01:34 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
emil
Labels
Clear labels
bug
Something is not working
 ci-improvement
CI workflow improvements
 doc-improvement
Documentation improvements
 feedback
Issues from subagent or agent feedback
 ready-to-merge
tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels ready-to-merge
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) emo kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#33