[Bug] Inconsistency of file_path in dataprep APIs

[cheehook]

Priority

P2-High

OS type

Ubuntu

Hardware type

Xeon-GNR

Installation method

• Pull docker images from hub.docker.com
• Build docker images from source
• Other
• N/A

Deploy method

• Docker
• Docker Compose
• Kubernetes Helm Charts
• Other
• N/A

Running nodes

Single Node

What's the version?

6e30a85

Description

In dataprep comp, /get API return file name and id with "rag-redis_" appended on actual file name.
However, in /delete API, the returned id from /get cannot be directly used in "file_path" param as it will get appended with another "rag-redis_" prefix causing 404 error.

This inconsistency requires user to manually filter out the prefix of file name/id from /get API to use for /delete API "file_path" param.

Reproduce steps

following the steps as mentioned in https://github.com/opea-project/GenAIComps/blob/main/comps/dataprep/src/README_redis.md

Raw log

Attachments

[letonghan]

Hi @MSCetin37 , please take a look into this issue.
It's introduced in this PR. Please consider to unify the file name format of /ingest, /get, and /delete.
We'd better not to combine index_name into uploaded file name, because the index_name is already specified.

[MSCetin37]

it is addressed in this PR#1569

[cheehook]

The PR fix has unblocked E2E test in GenAIStudio. Thanks for the patch.