chore(build): clean uv cache and avoid build artifacts#2369
Conversation
ssbarnea
commented
Dec 9, 2025
ssbarnea
commented
- reduce size of uv cache which slowed down CI (especially wsl)
- disable artifact and log generation in navigator
- isolated navigator from user setu0p
There was a problem hiding this comment.
Pull request overview
This PR optimizes the CI build process by reducing cache sizes and preventing unnecessary build artifacts. The changes focus on cleaning UV and pip caches, disabling ansible-navigator artifact generation, and isolating the navigator configuration from user settings.
Key changes:
- Added UV and pip cache pruning in the
finishtask to reduce cache sizes - Configured ansible-navigator to disable logging and artifact generation for testing
- Modified cache strategy to only cache pip wheels instead of the entire pip cache
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Configures distutils to output egg-info to the out directory |
| examples/ansible-navigator.yml | Adds isolated navigator config that disables logging and artifacts |
| Taskfile.yml | Adds cache pruning commands to the finish task for cleanup |
| .github/workflows/ci.yaml | Integrates task finish into CI workflows for cache cleanup |
| .github/actions/setup/action.yml | Narrows cache path to only pip wheels for better cache management |
| .env | Sets environment variables to use the isolated navigator config |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,8 @@ | |||
| # Used for testing purposes to isolated from user configured one | |||
There was a problem hiding this comment.
Typo in comment: "setu0p" should be "setup".
| UV_PROJECT_ENVIRONMENT="${HOME}/.local/share/virtualenvs/vsa" | ||
| # Do not add NODE_OPTIONS here because this is not allowed by GHA. | ||
|
|
||
| # Isolated from user level config, disable logging/artifacts |
There was a problem hiding this comment.
Grammar issue: "isolated from" should be "Isolate from" to maintain consistency with the action-oriented comment style.
| # Isolated from user level config, disable logging/artifacts | |
| # Isolate from user level config, disable logging/artifacts |
|
|
||
| # Isolated from user level config, disable logging/artifacts | ||
| ANSIBLE_NAVIGATOR_CONFIG=examples/ansible-navigator.yml | ||
| # mcp tests do not use ^ but the benefit from next line |
There was a problem hiding this comment.
Typo in comment: "benefit" should be "benefits" to be grammatically correct ("tests...benefit" should be "tests...benefits").
| # mcp tests do not use ^ but the benefit from next line | |
| # mcp tests do not use ^ but benefit from next line |
- reduce size of uv cache which slowed down CI (especially wsl) - disable artifact and log generation in navigator - isolated navigator from user setu0p
ssbarnea
force-pushed
the
chore/build3
branch
from
0178d12 to
e409bfc
Compare
