Skip to content

fix: configure VDB persistent cache directory#112

Merged
vbelouso merged 1 commit into
RHEcosystemAppEng:rh-aiq-mainfrom
vbelouso:fix-vdb-cache-dir
Sep 9, 2025
Merged

fix: configure VDB persistent cache directory#112
vbelouso merged 1 commit into
RHEcosystemAppEng:rh-aiq-mainfrom
vbelouso:fix-vdb-cache-dir

Conversation

@vbelouso

@vbelouso vbelouso commented Sep 9, 2025

Copy link
Copy Markdown
Collaborator

The code indexer was creating a new DocumentEmbedding instance with default relative paths, ignoring the cache directories specified in the configuration.

This change ensures the correct paths from the config are passed during initialization, forcing the indexer to use the intended cache directory.

@vbelouso vbelouso requested a review from zvigrinberg September 9, 2025 12:30
@vbelouso vbelouso added the bug Something isn't working label Sep 9, 2025
@vbelouso vbelouso marked this pull request as ready for review September 9, 2025 12:30
@vbelouso vbelouso marked this pull request as draft September 9, 2025 12:31
Signed-off-by: Vladimir Belousov <vbelouso@redhat.com>
@vbelouso vbelouso marked this pull request as ready for review September 9, 2025 12:37

@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.

LGTM

@vbelouso vbelouso merged commit 5ebd1b0 into RHEcosystemAppEng:rh-aiq-main Sep 9, 2025
@vbelouso vbelouso deleted the fix-vdb-cache-dir branch September 9, 2025 12:41
heatherzh01 pushed a commit that referenced this pull request Mar 6, 2026
fix: configure VDB persistent cache directory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants