Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update sample instructions for ChatQnA to use published Docker images #412

Closed
arun-gupta opened this issue Jul 17, 2024 · 7 comments
Closed
Assignees

Comments

@arun-gupta
Copy link
Contributor

Docker images for different microservices are published at #361. All the sample instructions should be updated to use those images. "Building the Docker image" instructions are still useful for somebody who wants to tinker with the code and should be move to a different docuemnt.

@xiguiw
Copy link
Collaborator

xiguiw commented Jul 18, 2024

@arun-gupta
After the images are published, we will update the instructions for ChatQnA to use published Docker images.

@xiguiw xiguiw self-assigned this Jul 18, 2024
@xiguiw
Copy link
Collaborator

xiguiw commented Jul 18, 2024

@arun-gupta
It seems the Docker images are published.
After I got the formal messages and verified it, I'll update the instructions.

Thank you!

@arun-gupta
Copy link
Contributor Author

@xiguiw
Copy link
Collaborator

xiguiw commented Aug 29, 2024

@xiguiw any updates on this? https://github.com/opea-project/GenAIExamples/tree/main/ChatQnA/docker/xeon#-build-docker-images is redundant
@arun-gupta Will update it next week.

When you mention it is redundant, do you mean pull docker image instead build it?
The docker image is specific or weekly build version, for daily update, build image from source code is needed.

@arun-gupta
Copy link
Contributor Author

@xiguiw exactly, these instructions can be moved elsewhere and keep the primary set of instructions really simple by pulling a docker image. The instructions can use the latest tag. Other versions can be pulled by looking at different tags on Docker Hub.

@xiguiw xiguiw mentioned this issue Sep 6, 2024
1 task
@xiguiw
Copy link
Collaborator

xiguiw commented Sep 6, 2024

@arun-gupta

Please help to review PR#762 for this issue.

There are platforms other than Xeon (Gauid,APIC) to be updated. And other GenAIExamples to be update.

@xiguiw
Copy link
Collaborator

xiguiw commented Oct 24, 2024

Document updated.
All docker images could be download by run
docker compose up -d

@xiguiw xiguiw closed this as completed Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants