Skip to content

Commit c5177c5

Browse files
Fix DocIndexRetriever CI error on Xeon (opea-project#1167)
Signed-off-by: minmin-intel <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 006c61b commit c5177c5

File tree

3 files changed

+36
-14
lines changed

3 files changed

+36
-14
lines changed

DocIndexRetriever/tests/test.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
import requests
77

88

9-
def search_knowledge_base(query: str, url: str, request_type="chat_completion") -> str:
9+
def search_knowledge_base(query: str, url: str, request_type: str) -> str:
1010
"""Search the knowledge base for a specific query."""
1111
print(url)
1212
proxies = {"http": ""}
@@ -18,12 +18,13 @@ def search_knowledge_base(query: str, url: str, request_type="chat_completion")
1818
"top_n": 2,
1919
}
2020
else:
21-
print("Sending text request")
21+
print("Sending textdoc request")
2222
payload = {
2323
"text": query,
2424
}
2525
response = requests.post(url, json=payload, proxies=proxies)
2626
print(response)
27+
print(response.json().keys())
2728
if "documents" in response.json():
2829
docs = response.json()["documents"]
2930
context = ""
@@ -32,7 +33,6 @@ def search_knowledge_base(query: str, url: str, request_type="chat_completion")
3233
context = str(i) + ": " + doc
3334
else:
3435
context += "\n" + str(i) + ": " + doc
35-
# print(context)
3636
return context
3737
elif "text" in response.json():
3838
return response.json()["text"]
@@ -44,7 +44,6 @@ def search_knowledge_base(query: str, url: str, request_type="chat_completion")
4444
context = doc["text"]
4545
else:
4646
context += "\n" + doc["text"]
47-
# print(context)
4847
return context
4948
else:
5049
return "Error parsing response from the knowledge base."

DocIndexRetriever/tests/test_compose_on_gaudi.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ LOG_PATH="$WORKPATH/tests"
1515
ip_address=$(hostname -I | awk '{print $1}')
1616

1717
function build_docker_images() {
18+
echo "Building Docker Images...."
1819
cd $WORKPATH/docker_image_build
1920
if [ ! -d "GenAIComps" ] ; then
2021
git clone https://github.com/opea-project/GenAIComps.git && cd GenAIComps && git checkout "${opea_branch:-"main"}" && cd ../
@@ -26,9 +27,11 @@ function build_docker_images() {
2627
docker pull redis/redis-stack:7.2.0-v9
2728
docker pull ghcr.io/huggingface/tei-gaudi:1.5.0
2829
docker images && sleep 1s
30+
echo "Docker images built!"
2931
}
3032

3133
function start_services() {
34+
echo "Starting Docker Services...."
3235
cd $WORKPATH/docker_compose/intel/hpu/gaudi
3336
export EMBEDDING_MODEL_ID="BAAI/bge-base-en-v1.5"
3437
export RERANK_MODEL_ID="BAAI/bge-reranker-base"
@@ -47,6 +50,7 @@ function start_services() {
4750
# Start Docker Containers
4851
docker compose up -d
4952
sleep 20
53+
echo "Docker services started!"
5054
}
5155

5256
function validate() {

DocIndexRetriever/tests/test_compose_on_xeon.sh

Lines changed: 29 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,10 @@ LOG_PATH="$WORKPATH/tests"
1515
ip_address=$(hostname -I | awk '{print $1}')
1616

1717
function build_docker_images() {
18+
echo "Building Docker Images...."
1819
cd $WORKPATH/docker_image_build
1920
if [ ! -d "GenAIComps" ] ; then
21+
echo "Cloning GenAIComps repository"
2022
git clone https://github.com/opea-project/GenAIComps.git && cd GenAIComps && git checkout "${opea_branch:-"main"}" && cd ../
2123
fi
2224
service_list="dataprep-redis embedding-tei retriever-redis reranking-tei doc-index-retriever"
@@ -25,9 +27,12 @@ function build_docker_images() {
2527
docker pull ghcr.io/huggingface/text-embeddings-inference:cpu-1.5
2628
docker pull redis/redis-stack:7.2.0-v9
2729
docker images && sleep 1s
30+
31+
echo "Docker images built!"
2832
}
2933

3034
function start_services() {
35+
echo "Starting Docker Services...."
3136
cd $WORKPATH/docker_compose/intel/cpu/xeon
3237
export EMBEDDING_MODEL_ID="BAAI/bge-base-en-v1.5"
3338
export RERANK_MODEL_ID="BAAI/bge-reranker-base"
@@ -45,7 +50,8 @@ function start_services() {
4550

4651
# Start Docker Containers
4752
docker compose up -d
48-
sleep 20
53+
sleep 5m
54+
echo "Docker services started!"
4955
}
5056

5157
function validate() {
@@ -66,7 +72,7 @@ function validate_megaservice() {
6672
echo "===========Ingest data=================="
6773
local CONTENT=$(http_proxy="" curl -X POST "http://${ip_address}:6007/v1/dataprep" \
6874
-H "Content-Type: multipart/form-data" \
69-
-F 'link_list=["https://opea.dev"]')
75+
-F 'link_list=["https://opea.dev/"]')
7076
local EXIT_CODE=$(validate "$CONTENT" "Data preparation succeeded" "dataprep-redis-service-xeon")
7177
echo "$EXIT_CODE"
7278
local EXIT_CODE="${EXIT_CODE:0-1}"
@@ -77,19 +83,26 @@ function validate_megaservice() {
7783
fi
7884

7985
# Curl the Mega Service
80-
echo "================Testing retriever service: Default params================"
81-
82-
local CONTENT=$(curl http://${ip_address}:8889/v1/retrievaltool -X POST -H "Content-Type: application/json" -d '{
83-
"messages": "Explain the OPEA project?"
86+
echo "================Testing retriever service: Text Request ================"
87+
cd $WORKPATH/tests
88+
local CONTENT=$(http_proxy="" curl http://${ip_address}:8889/v1/retrievaltool -X POST -H "Content-Type: application/json" -d '{
89+
"text": "Explain the OPEA project?"
8490
}')
91+
# local CONTENT=$(python test.py --host_ip ${ip_address} --request_type text)
8592
local EXIT_CODE=$(validate "$CONTENT" "OPEA" "doc-index-retriever-service-xeon")
8693
echo "$EXIT_CODE"
8794
local EXIT_CODE="${EXIT_CODE:0-1}"
8895
echo "return value is $EXIT_CODE"
8996
if [ "$EXIT_CODE" == "1" ]; then
90-
docker logs tei-embedding-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
97+
echo "=============Embedding container log=================="
98+
docker logs embedding-tei-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
99+
echo "=============Retriever container log=================="
91100
docker logs retriever-redis-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
92-
docker logs reranking-tei-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
101+
echo "=============TEI Reranking log=================="
102+
docker logs tei-reranking-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
103+
echo "=============Reranking container log=================="
104+
docker logs reranking-tei-xeon-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
105+
echo "=============Doc-index-retriever container log=================="
93106
docker logs doc-index-retriever-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
94107
exit 1
95108
fi
@@ -102,9 +115,15 @@ function validate_megaservice() {
102115
local EXIT_CODE="${EXIT_CODE:0-1}"
103116
echo "return value is $EXIT_CODE"
104117
if [ "$EXIT_CODE" == "1" ]; then
105-
docker logs tei-embedding-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
118+
echo "=============Embedding container log=================="
119+
docker logs embedding-tei-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
120+
echo "=============Retriever container log=================="
106121
docker logs retriever-redis-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
107-
docker logs reranking-tei-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
122+
echo "=============TEI Reranking log=================="
123+
docker logs tei-reranking-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
124+
echo "=============Reranking container log=================="
125+
docker logs reranking-tei-xeon-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
126+
echo "=============Doc-index-retriever container log=================="
108127
docker logs doc-index-retriever-server | tee -a ${LOG_PATH}/doc-index-retriever-service-xeon.log
109128
exit 1
110129
fi

0 commit comments

Comments
 (0)