Skip to content

Fix TGI image HF_TOKEN environment variable rename#2059

Merged
yinghu5 merged 1 commit into
opea-project:mainfrom
xiguiw:token
Jun 11, 2025
Merged

Fix TGI image HF_TOKEN environment variable rename#2059
yinghu5 merged 1 commit into
opea-project:mainfrom
xiguiw:token

Conversation

@xiguiw
Copy link
Copy Markdown
Collaborator

@xiguiw xiguiw commented Jun 10, 2025

Fix #1991

Description

The summary of the proposed changes as long as the relevant motivation and context.

Issues

Fix #1991

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

No

Tests

Local test passed.

Fix opea-project#1991

Signed-off-by: Wang, Xigui <xigui.wang@intel.com>
Copilot AI review requested due to automatic review settings June 10, 2025 10:08
@xiguiw xiguiw requested a review from letonghan as a code owner June 10, 2025 10:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the environment variable naming issue for the TGI image by renaming the variable from HUGGINGFACEHUB_API_TOKEN to HF_TOKEN.

  • Updated the docker-compose YAML file to use HF_TOKEN instead of HUGGINGFACEHUB_API_TOKEN.
  • Addresses bug fix #1991 by ensuring the environment variable name is consistent.

@github-actions
Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

@xiguiw xiguiw requested review from ZePan110 and chensuyue June 10, 2025 10:08
Copy link
Copy Markdown
Collaborator

@yinghu5 yinghu5 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@yinghu5 yinghu5 merged commit b4ad636 into opea-project:main Jun 11, 2025
14 checks passed
edlee123 pushed a commit to edlee123/GenAIExamples that referenced this pull request Jun 24, 2025
Signed-off-by: Wang, Xigui <xigui.wang@intel.com>
Signed-off-by: Ed Lee <16417837+edlee123@users.noreply.github.com>
alexsin368 pushed a commit to alexsin368/GenAIExamples that referenced this pull request Aug 13, 2025
Signed-off-by: Wang, Xigui <xigui.wang@intel.com>
Signed-off-by: alexsin368 <alex.sin@intel.com>
cogniware-devops pushed a commit to Cogniware-Inc/GenAIExamples that referenced this pull request Dec 19, 2025
Signed-off-by: Wang, Xigui <xigui.wang@intel.com>
Signed-off-by: cogniware-devops <ambarish.desai@cogniware.ai>
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.

[Bug] DocSum tgi UT test fail

4 participants