Skip to content

fixes mem0 with valkey/redis issue#1

Closed
MatthiasHowellYopp wants to merge 2 commits into
mainfrom
mh/fix/valkey
Closed

fixes mem0 with valkey/redis issue#1
MatthiasHowellYopp wants to merge 2 commits into
mainfrom
mh/fix/valkey

Conversation

@MatthiasHowellYopp
Copy link
Copy Markdown
Owner

Fixes issue with using mem0 as external memory with valkey/redis as vector store

Copy link
Copy Markdown
Collaborator

@Jonathan-Improving Jonathan-Improving left a comment

Choose a reason for hiding this comment

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

I see the test_mem0_storage.py uses mocked service objects, were you able to integration test this at all?

@MatthiasHowellYopp
Copy link
Copy Markdown
Owner Author

the integration tests, such as there are, work -mostly by using recordings of LLM interactions. However testing it in the use case I'm working on - crewai with Mem0 and Valkey - works.

@MatthiasHowellYopp MatthiasHowellYopp force-pushed the mh/fix/valkey branch 8 times, most recently from 8bc007d to fe7a87e Compare February 6, 2026 21:57
@MatthiasHowellYopp MatthiasHowellYopp force-pushed the mh/fix/valkey branch 5 times, most recently from 1568393 to 61a8f22 Compare February 12, 2026 18:57
@MatthiasHowellYopp
Copy link
Copy Markdown
Owner Author

crewai re-architected their memory structure.

@MatthiasHowellYopp MatthiasHowellYopp deleted the mh/fix/valkey branch March 3, 2026 18:08
MatthiasHowellYopp pushed a commit that referenced this pull request Mar 5, 2026
…ulnerability (crewAIInc#4740)

GitPython ==3.1.38 is affected by a high-severity path traversal
vulnerability (dependabot alert #1). Bump to >=3.1.41,<4 which
includes the fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants