Skip to content

Conversation

@lianhao
Copy link
Collaborator

@lianhao lianhao commented Jun 27, 2025

Description

tei: upgrade to version cpu-1.7

Issues

Fixes #1133.

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

@lianhao lianhao requested a review from yongfengdu as a code owner June 27, 2025 08:57
@lianhao lianhao requested a review from poussa June 27, 2025 08:57
Copy link
Collaborator

@eero-t eero-t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TEI image is the only thing with cpu-1.5 tag, so it's easy to find instances where it's used. TEI values file is not the only place:

$ git grep -l cpu-1.5
helm-charts/chatqna/faqgen-rocm-tgi-values.yaml
helm-charts/chatqna/faqgen-rocm-values.yaml
helm-charts/chatqna/rocm-tgi-values.yaml
helm-charts/chatqna/rocm-values.yaml
helm-charts/common/tei/README.md
helm-charts/common/tei/values.yaml
helm-charts/common/teirerank/README.md
helm-charts/common/teirerank/values.yaml
microservices-connector/config/manifests/data-prep_milvus.yaml
microservices-connector/config/manifests/retriever-usvc_milvus.yaml
microservices-connector/config/manifests/tei.yaml
microservices-connector/config/manifests/tei_gaudi.yaml
microservices-connector/config/manifests/teirerank.yaml
microservices-connector/config/manifests/teirerank_gaudi.yaml
microservices-connector/config/samples/ChatQnA/use_cases.md

@eero-t
Copy link
Collaborator

eero-t commented Jun 27, 2025

Why ChatQnA ROCM value files (#949) added duplicated TEI default CPU values?

I think tei sections can be just removed from them...

Upgrade tei/teirerank default version to cpu-1.7.

Remove duplicated teirerank settings for chatqna rocm environment.

Signed-off-by: Lianhao Lu <[email protected]>
@lianhao lianhao closed this Jun 30, 2025
@lianhao lianhao reopened this Jun 30, 2025
@lianhao
Copy link
Collaborator Author

lianhao commented Jun 30, 2025

Why ChatQnA ROCM value files (#949) added duplicated TEI default CPU values?

I think tei sections can be just removed from them...

Done

@lianhao
Copy link
Collaborator Author

lianhao commented Jun 30, 2025

TEI image is the only thing with cpu-1.5 tag, so it's easy to find instances where it's used. TEI values file is not the only place:

$ git grep -l cpu-1.5
helm-charts/chatqna/faqgen-rocm-tgi-values.yaml
helm-charts/chatqna/faqgen-rocm-values.yaml
helm-charts/chatqna/rocm-tgi-values.yaml
helm-charts/chatqna/rocm-values.yaml
helm-charts/common/tei/README.md
helm-charts/common/tei/values.yaml
helm-charts/common/teirerank/README.md
helm-charts/common/teirerank/values.yaml
microservices-connector/config/manifests/data-prep_milvus.yaml
microservices-connector/config/manifests/retriever-usvc_milvus.yaml
microservices-connector/config/manifests/tei.yaml
microservices-connector/config/manifests/tei_gaudi.yaml
microservices-connector/config/manifests/teirerank.yaml
microservices-connector/config/manifests/teirerank_gaudi.yaml
microservices-connector/config/samples/ChatQnA/use_cases.md

Also updated teirerank.

GMC will not be covered in this PR.

@lianhao
Copy link
Collaborator Author

lianhao commented Jun 30, 2025

weird github action error. Will close this PR and recreate new one #1135

@lianhao lianhao closed this Jun 30, 2025
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

Successfully merging this pull request may close these issues.

[ci-auto] upgrade tei version to cpu-1.7

2 participants