Fix waring. (#504)

Signed-off-by: zepan <ze.pan@intel.com>

tests/test_agent_langchain.sh

@@ -13,7 +13,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $WORKPATH
     docker build --no-cache -t opea/comps-agent-langchain:comps -f comps/agent/langchain/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/comps-agent-langchain built fail"
         exit 1
     else

tests/test_asr.sh

@@ -11,14 +11,14 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/whisper:comps -f comps/asr/whisper/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/whisper built fail"
         exit 1
     else
         echo "opea/whisper built successful"
     fi
     docker build --no-cache -t opea/asr:comps -f comps/asr/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/asr built fail"
         exit 1
     else

tests/test_chathistory_mongo.sh

@@ -18,7 +18,7 @@ function build_docker_images() {
     docker run -d -p 27017:27017 --name=test-comps-mongo mongo:latest
 
     docker build --no-cache -t opea/chathistory-mongo-server:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/chathistory/mongo/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/chathistory-mongo-server built fail"
         exit 1
     else

tests/test_dataprep_milvus.sh

@@ -13,15 +13,15 @@ function build_docker_images() {
     echo $(pwd)
     # langchain mosec embedding image
     docker build --no-cache -t opea/langchain-mosec:comps --build-arg http_proxy=$http_proxy --build-arg https_proxy=$https_proxy -f comps/embeddings/langchain-mosec/mosec-docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/langchain-mosec built fail"
         exit 1
     else
         echo "opea/langchain-mosec built successful"
     fi
     # dataprep milvus image
     docker build --no-cache -t opea/dataprep-milvus:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/dataprep/milvus/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-milvus built fail"
         exit 1
     else

tests/test_dataprep_pgvector.sh

@@ -17,7 +17,7 @@ function build_docker_images() {
 
     # build dataprep image for pgvector
     docker build --no-cache -t opea/dataprep-pgvector:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f $WORKPATH/comps/dataprep/pgvector/langchain/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-pgvector built fail"
         exit 1
     else

tests/test_dataprep_pinecone.sh

@@ -11,7 +11,7 @@ function build_docker_images() {
 
     # build dataprep image for pinecone
     docker build --no-cache -t opea/dataprep-pinecone:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f $WORKPATH/comps/dataprep/pinecone/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-pinecone built fail"
         exit 1
     else

tests/test_dataprep_qdrant_langchain.sh

@@ -13,7 +13,7 @@ function build_docker_images() {
 
     # dataprep qdrant image
     docker build --no-cache -t opea/dataprep-qdrant:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/dataprep/qdrant/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-qdrant built fail"
         exit 1
     else

tests/test_dataprep_redis_langchain.sh

@@ -12,7 +12,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/dataprep-redis:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/dataprep/redis/langchain/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-redis built fail"
         exit 1
     else

tests/test_dataprep_redis_langchain_ray.sh

@@ -12,7 +12,7 @@ function build_docker_images() {
     echo "Building the docker images"
     cd $WORKPATH
     docker build --no-cache -t opea/dataprep-on-ray-redis:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/dataprep/redis/langchain_ray/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-on-ray-redis built fail"
         exit 1
     else

tests/test_dataprep_redis_llama_index.sh

@@ -12,7 +12,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/dataprep-redis-llama-index:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/dataprep/redis/llama_index/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/dataprep-redis-llama-index built fail"
         exit 1
     else

tests/test_embeddings_langchain-mosec.sh

@@ -11,7 +11,7 @@ function build_mosec_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --build-arg http_proxy=$http_proxy --build-arg https_proxy=$https_proxy --no-cache -t opea/embedding-langchain-mosec-endpoint:comps -f comps/embeddings/langchain-mosec/mosec-docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/embedding-langchain-mosec-endpoint built fail"
         exit 1
     else
@@ -23,7 +23,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --build-arg http_proxy=$http_proxy --build-arg https_proxy=$https_proxy --no-cache -t opea/embedding-langchain-mosec:comps -f comps/embeddings/langchain-mosec/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/embedding-langchain-mosec built fail"
         exit 1
     else

tests/test_embeddings_langchain.sh

@@ -11,7 +11,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/embedding-tei:comps -f comps/embeddings/langchain/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/embedding-tei built fail"
         exit 1
     else

tests/test_embeddings_llama_index.sh

@@ -12,7 +12,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/embedding-tei-llamaindex:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/embeddings/llama_index/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/embedding-tei-llamaindex built fail"
         exit 1
     else

tests/test_guardrails_llama_guard.sh

@@ -12,7 +12,7 @@ function build_docker_images() {
     cd $WORKPATH
     docker pull ghcr.io/huggingface/tgi-gaudi:2.0.1
     docker build --no-cache -t opea/guardrails-tgi:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/guardrails/llama_guard/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/guardrails-tgi built fail"
         exit 1
     else

tests/test_guardrails_pii_detection.sh

@@ -11,7 +11,7 @@ function build_docker_images() {
     echo "Start building docker images for microservice"
     cd $WORKPATH
     docker build --no-cache -t opea/guardrails-pii-detection:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/guardrails/pii_detection/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/guardrails-pii-detection built fail"
         exit 1
     else

tests/test_llms_faq-generation_tgi.sh

@@ -11,7 +11,7 @@ LOG_PATH="$WORKPATH/tests"
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/llm-faqgen-tgi:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/llms/faq-generation/tgi/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llm-faqgen-tgi built fail"
         exit 1
     else

tests/test_llms_summarization_tgi.sh

@@ -11,7 +11,7 @@ LOG_PATH="$WORKPATH/tests"
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/llm-tgi:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/llms/summarization/tgi/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llm-tgi built fail"
         exit 1
     else

tests/test_llms_text-generation_native.sh

@@ -14,7 +14,7 @@ function build_docker_images() {
         --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy \
         -t opea/llm-native:comps \
         -f comps/llms/text-generation/native/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llm-native built fail"
         exit 1
     else

tests/test_llms_text-generation_tgi.sh

@@ -10,7 +10,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -t opea/llm-tgi:comps -f comps/llms/text-generation/tgi/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llm-tgi built fail"
         exit 1
     else

tests/test_llms_text-generation_vllm-openvino.sh

@@ -21,7 +21,7 @@ function build_container() {
       . \
       --build-arg https_proxy=$https_proxy \
       --build-arg http_proxy=$http_proxy
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "vllm-openvino built fail"
         exit 1
     else

tests/test_llms_text-generation_vllm-ray.sh

@@ -13,7 +13,7 @@ function build_docker_images() {
     docker build \
         -f comps/llms/text-generation/vllm-ray/docker/Dockerfile.vllmray  \
         --no-cache -t opea/vllm_ray-habana:comps --network=host .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/vllm_ray-habana built fail"
         exit 1
     else
@@ -25,7 +25,7 @@ function build_docker_images() {
     docker build \
         --no-cache -t opea/llm-vllm-ray:comps \
         -f comps/llms/text-generation/vllm-ray/docker/Dockerfile.microservice .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llm-vllm-ray built fail"
         exit 1
     else

tests/test_llms_text-generation_vllm.sh

@@ -14,7 +14,7 @@ function build_docker_images() {
         -f docker/Dockerfile.hpu \
         --no-cache -t opea/vllm-hpu:comps \
         --shm-size=128g .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/vllm-hpu built fail"
         exit 1
     else
@@ -26,7 +26,7 @@ function build_docker_images() {
     docker build  \
         --no-cache -t opea/llm-vllm:comps \
         -f comps/llms/text-generation/vllm/docker/Dockerfile.microservice .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llm-vllm built fail"
         exit 1
     else

tests/test_lvms_llava.sh

@@ -11,14 +11,14 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/llava:comps -f comps/lvms/llava/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llava built fail"
         exit 1
     else
         echo "opea/llava built successful"
     fi
     docker build --no-cache -t opea/lvm:comps -f comps/lvms/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/lvm built fail"
         exit 1
     else

tests/test_lvms_tgi_llava_next.sh

@@ -12,7 +12,7 @@ function build_docker_images() {
     echo $(pwd)
     git clone https://github.com/yuanwu2017/tgi-gaudi.git && cd tgi-gaudi && git checkout v2.0.4
     docker build --no-cache -t opea/llava-tgi:comps .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/llava-tgi built fail"
         exit 1
     else
@@ -21,7 +21,7 @@ function build_docker_images() {
 
     cd ..
     docker build --no-cache -t opea/lvm-tgi:comps -f comps/lvms/Dockerfile_tgi .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/lvm-tgi built fail"
         exit 1
     else

tests/test_prompt_registry_mongo.sh

@@ -18,7 +18,7 @@ function build_docker_images() {
     docker run -d -p 27017:27017 --name=test-comps-mongo mongo:latest
 
     docker build --no-cache -t opea/promptregistry-mongo-server:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/prompt_registry/mongo/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/promptregistry-mongo-server built fail"
         exit 1
     else

tests/test_reranks_fastrag.sh

@@ -9,7 +9,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -t opea/reranking-fastrag:comps -f comps/reranks/fastrag/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/reranking-fastrag built fail"
         exit 1
     else

tests/test_reranks_langchain-mosec.sh

@@ -11,7 +11,7 @@ function build_mosec_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --build-arg http_proxy=$http_proxy --build-arg https_proxy=$https_proxy --no-cache -t opea/reranking-langchain-mosec-endpoint:comps -f comps/reranks/langchain-mosec/mosec-docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/reranking-langchain-mosec-endpoint built fail"
         exit 1
     else
@@ -23,7 +23,7 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --build-arg http_proxy=$http_proxy --build-arg https_proxy=$https_proxy --no-cache -t opea/reranking-langchain-mosec:comps -f comps/reranks/langchain-mosec/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/reranking-langchain-mosec built fail"
         exit 1
     else

tests/test_reranks_tei.sh

@@ -9,7 +9,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/reranking-tei:comps -f comps/reranks/tei/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/reranking-tei built fail"
         exit 1
     else

tests/test_retrievers_haystack_qdrant.sh

@@ -9,7 +9,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/retriever-qdrant:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/retrievers/haystack/qdrant/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/retriever-qdrant built fail"
         exit 1
     else

tests/test_retrievers_langchain_pgvector.sh

@@ -9,7 +9,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/retriever-pgvector:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/retrievers/langchain/pgvector/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/retriever-pgvector built fail"
         exit 1
     else

tests/test_retrievers_langchain_pinecone.sh

@@ -9,7 +9,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/retriever-pinecone:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/retrievers/langchain/pinecone/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/retriever-pinecone built fail"
         exit 1
     else

tests/test_retrievers_langchain_redis.sh

@@ -11,7 +11,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/retriever-redis:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/retrievers/langchain/redis/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/retriever-redis built fail"
         exit 1
     else

tests/test_tts.sh

@@ -11,14 +11,14 @@ function build_docker_images() {
     cd $WORKPATH
     echo $(pwd)
     docker build --no-cache -t opea/speecht5:comps -f comps/tts/speecht5/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/speecht5 built fail"
         exit 1
     else
         echo "opea/speecht5 built successful"
     fi
     docker build --no-cache -t opea/tts:comps -f comps/tts/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/tts built fail"
         exit 1
     else

tests/test_web_retrievers_langchain_chroma.sh

@@ -9,7 +9,7 @@ ip_address=$(hostname -I | awk '{print $1}')
 function build_docker_images() {
     cd $WORKPATH
     docker build --no-cache -t opea/web-retriever-chroma:comps --build-arg https_proxy=$https_proxy --build-arg http_proxy=$http_proxy -f comps/web_retrievers/langchain/chroma/docker/Dockerfile .
-    if $? ; then
+    if [ $? -ne 0 ]; then
         echo "opea/web-retriever-chroma built fail"
         exit 1
     else