Skip to content

feat: add caching of Go modules to persistent storage#145

Merged
vbelouso merged 1 commit into
RHEcosystemAppEng:rh-aiq-mainfrom
vbelouso:feat-go-cache
Nov 13, 2025
Merged

feat: add caching of Go modules to persistent storage#145
vbelouso merged 1 commit into
RHEcosystemAppEng:rh-aiq-mainfrom
vbelouso:feat-go-cache

Conversation

@vbelouso

@vbelouso vbelouso commented Nov 11, 2025

Copy link
Copy Markdown
Collaborator

Currently, when downloading Go modules, they are stored by default in ~/go/pkg/mod
This cache is ephemeral, meaning it gets lost every time the container or pod restarts.

This PR introduces a persistent cache mechanism to retain these modules between pod restarts.

Signed-off-by: Vladimir Belousov <vbelouso@redhat.com>
@vbelouso vbelouso requested a review from zvigrinberg November 11, 2025 10:43
@vbelouso vbelouso self-assigned this Nov 11, 2025
@vbelouso vbelouso added the chore label Nov 11, 2025

@zvigrinberg zvigrinberg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vbelouso Very good job.
LGTM Approved.
Next i'd be glad if you'll do it also for Python and in the future for JAVA also ( which is currently still WIP, the PR is not ready yet).
Thank you.

@vbelouso

Copy link
Copy Markdown
Collaborator Author

@vbelouso Very good job. LGTM Approved. Next i'd be glad if you'll do it also for Python and in the future for JAVA also ( which is currently still WIP, the PR is not ready yet). Thank you.
@zvigrinberg Yeah I already have drafts for this. I just didn't include everything in one PR

@vbelouso vbelouso merged commit a6a63e3 into RHEcosystemAppEng:rh-aiq-main Nov 13, 2025
heatherzh01 pushed a commit that referenced this pull request Mar 6, 2026
feat: add caching of Go modules to persistent storage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants