-
Notifications
You must be signed in to change notification settings - Fork 217
k8s deployment for neo4j #1254
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
k8s deployment for neo4j #1254
Conversation
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
for more information, see https://pre-commit.ci
Signed-off-by: jeanyu-habana <[email protected]>
get the latest
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
for more information, see https://pre-commit.ci
|
Hi @jeanyu-habana, thanks for your contribution. |
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
fixed. @letonghan |
|
Can I have at least one more approval from the reviewers? @XinyaoWa @lvliang-intel |
|
You are using the neo4j helm chart from upstream, and have customized values.yaml file. Include the dependency in your HybridRAG dependency: |
Yes, this component can be used in our HybridRAG, but we have not decided if we will use neo4j or NebulaGraph in HybriRAG. The intent of this PR is to provide NebulaGraph as a general purpose microservice. HybridRAG is a work in progress, and we will reference the comment above as part of the HybridRAG dependency. However, in the context of this PR, do you think we can keep the content as is? |
|
@jeanyu-habana The purpose of deployment/kubernetes directory is to put values file for testing our own helm charts (https://github.com/opea-project/GenAIInfra/tree/main/helm-charts). Please take care of the CI test and maintain when necessary.
|
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
|
@yongfengdu Thank you for meeting with me to discuss similar concerns over adding k8s components such as neo4j. Just to summarize, we went over the example in GenAIExamples and GenAIInfra (i.e., milvus), and I agreed when we implement our HybridRAG in GenAIExamples and GenAIInfra, we will follow the guidance. However, having neo4j k8s deployment in the GenAIComps repo still has value. The script here is tested and can be used to deploy neo4j as microservice in k8s. Please let me know if there is anything needed, and we will address them ASAP. |
This PR is NOT active for weeks. @jeanyu-habana |
Description
The purpose is to enable neo4j component to cover the installation and configuration in k8s env. This component will be a part of an end-to-end GenAI pipeline (named HybridRAG).
Issues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
n/a
Tests
Tested the commands and configuration file (values.yaml) in k8s environment. Neo4j was properly installed and configured. Database was generated and can be accessed and updated from within the k8s environment, from command-line, and from the web console.