Adapt latest changes for data-prep #712
Conversation
|
We need dataprep related changes get merged into GenAIExamples first. Otherwise, there will be no image in CI |
| initContainers: | ||
| - name: wait-for-db |
There was a problem hiding this comment.
Wouldn't helm wait for the Chart be enough instead?
Or is this intended for cases where Helm is used just to generate manifest file?
There was a problem hiding this comment.
helm wait is not enough. The dataprep code itself will create a redis/milvus client to connect to DB when it starts, if that fails, it will never retry to create the db client and hence fails all the incoming user requests later. This is actually a cloud native non-friendly bug in the dataprep/retriever source code itself. But we don't have time to fix that in this release cycle. Already created an issue in GenAIComps.
There was a problem hiding this comment.
Ok. And I guess helm template manifest generation is valid use-case too...
lianhao
force-pushed
the
data-prep
branch
2 times, most recently
from
cee6420 to
7ed1a69
Compare
| @@ -38,21 +44,23 @@ Then run the command `kubectl port-forward svc/data-prep 6007:6007` to expose th | |||
| Open another terminal and run the following command to verify the service if working: | |||
|
|
|||
| ```console | |||
| curl http://localhost:6007/v1/dataprep \ | |||
| curl http://localhost:6007/v1/dataprep/ingest \ | |||
There was a problem hiding this comment.
The dataprep api has been changed for file uploading?
There was a problem hiding this comment.
Unfortunately, yes
lianhao
force-pushed
the
data-prep
branch
2 times, most recently
from
fb7a1da to
a1c4492
Compare
|
Need to wait for opea-project/GenAIExamples#1408 to get merged in first. |
Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>
Description
Adapt latest changes for data-prep.
Issues
Fixes #730
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.