opea-project / GenAIComps

GenAI components at micro-service level; GenAI service composer to create mega-service
Apache License 2.0
78 stars 140 forks source link

PR: OpenSearch integration components for OPEA #908

Open cameronmorin opened 1 week ago

cameronmorin commented 1 week ago

Description

These proposed changes create the OPEA components necessary to integrate OpenSearch with GenAIExamples, specifically tested & validated with ChatQnA

Issues

n/a

Type of change

List the type of change like below. Please delete options that are not relevant.

Dependencies

opensearch-py

Tests

I tested the components independently, as well as end-to-end by creating my own ChatQnA cluster replacing the redis microservices (dataprep, retriever, and vectorstore) with the OpenSearch components I've created here.

lvliang-intel commented 1 week ago

@cameronmorin, Thanks for your contribution. Please check the pre-commit CI issues, looks like the yaml file has format issue. image

letonghan commented 1 week ago

Hi @cameronmorin , thanks for your contribution. Please check out the comments above, and remember to add CI test scripts like this for each of the new components. Please name the test scripts following the schema of test/${component}/test_${component}_${storage}_${framework}.sh. Thank you.

cameronmorin commented 1 week ago

Hi @letonghan ! Thank you so much for reviewing my PR and leaving your helpful comments. I've addressed the simple ones of removing unnecessary comments & files, and I'm in the process of adding the test scripts you mentioned that are required.

joshuayao commented 6 days ago

Hi @cameronmorin

Thanks for your PR. Could you please check the failures in the CI test? Thanks.