Skip to content

update secrets token name for ProductivitySuite, RerankFinetuning, SearchQnA and Translation#2038

Merged
ZePan110 merged 1 commit intomainfrom
ze-token/P-T
Jun 6, 2025
Merged

update secrets token name for ProductivitySuite, RerankFinetuning, SearchQnA and Translation#2038
ZePan110 merged 1 commit intomainfrom
ze-token/P-T

Conversation

@ZePan110
Copy link
Copy Markdown
Collaborator

@ZePan110 ZePan110 commented Jun 5, 2025

Description

update secrets token name for ProductivitySuite, RerankFinetuning, SearchQnA and Translation
Fix shellcheck issue

Issues

#1954
#1999

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)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

…archQnA and Translation

Fix shellcheck issue

Signed-off-by: ZePan110 <ze.pan@intel.com>
Copilot AI review requested due to automatic review settings June 5, 2025 07:41
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 5, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

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 updates the environment variable references to use HF_TOKEN instead of HUGGINGFACEHUB_API_TOKEN for various services (Translation, SearchQnA, RerankFinetuning, and ProductivitySuite) and fixes a related shellcheck issue.

  • Replaces all occurrences of HUGGINGFACEHUB_API_TOKEN with HF_TOKEN across multiple docker-compose and environment files.
  • Adds missing shebang lines to test scripts.
  • Updates documentation to reflect the new token name.

Reviewed Changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Translation/docker_compose/intel/set_env.sh Replaces HUGGINGFACEHUB_API_TOKEN with HF_TOKEN.
Translation/docker_compose/intel/hpu/gaudi/compose.yaml Updates token environment variable reference for container configuration.
Translation/docker_compose/intel/cpu/xeon/compose.yaml Updates token environment variable used in compose file.
Translation/docker_compose/amd/gpu/rocm/set_env_vllm.sh Updates token reference for VLLM environment setup.
Translation/docker_compose/amd/gpu/rocm/set_env.sh Updates token reference in the main environment script.
SearchQnA/tests/README.md Updates documentation to use HF_TOKEN.
SearchQnA/docker_compose/intel/set_env.sh Replaces token variable as per the new naming convention.
SearchQnA/docker_compose/intel/hpu/gaudi/compose.yaml Updates token variable reference in docker-compose configuration.
SearchQnA/docker_compose/intel/cpu/xeon/compose.yaml Updates token reference in the compose file.
SearchQnA/docker_compose/intel/cpu/xeon/README.md Updates documentation, including instructions to use HF_TOKEN.
SearchQnA/docker_compose/amd/gpu/rocm/set_env_vllm.sh Updates token reference for GPU environment configuration.
SearchQnA/docker_compose/amd/gpu/rocm/set_env.sh Updates token reference in associated environment file.
SearchQnA/docker_compose/amd/gpu/rocm/README.md Updates documentation to use HF_TOKEN.
RerankFinetuning/tests/test_compose_on_xeon.sh Adds a shebang line for proper test script execution.
RerankFinetuning/tests/test_compose_on_gaudi.sh Adds a shebang line for proper test script execution.
ProductivitySuite/tests/README.md Updates documentation to use HF_TOKEN.
ProductivitySuite/docker_compose/intel/cpu/xeon/set_env.sh Updates token reference in environment script.
ProductivitySuite/docker_compose/intel/cpu/xeon/compose.yaml Updates token reference in the compose file.
ProductivitySuite/docker_compose/intel/cpu/xeon/README.md Updates documentation to use HF_TOKEN and provides instructions on setting the environment variable.
Comments suppressed due to low confidence (1)

ProductivitySuite/docker_compose/intel/cpu/xeon/README.md:113

  • There is a spelling mistake: 'tyour' should be corrected to 'your'.
> Change the Your_Huggingface_API_Token below with tyour actual Huggingface API Token value

@ZePan110 ZePan110 mentioned this pull request Jun 5, 2025
4 tasks
@ZePan110 ZePan110 merged commit 80502a1 into main Jun 6, 2025
44 of 45 checks passed
@ZePan110 ZePan110 deleted the ze-token/P-T branch June 6, 2025 08:37
edlee123 pushed a commit to edlee123/GenAIExamples that referenced this pull request Jun 24, 2025
…archQnA and Translation (opea-project#2038)

update secrets token name for ProductivitySuite, RerankFinetuning, SearchQnA and Translation
Fix shellcheck issue

Signed-off-by: ZePan110 <ze.pan@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
…archQnA and Translation (opea-project#2038)

update secrets token name for ProductivitySuite, RerankFinetuning, SearchQnA and Translation
Fix shellcheck issue

Signed-off-by: ZePan110 <ze.pan@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
…archQnA and Translation (opea-project#2038)

update secrets token name for ProductivitySuite, RerankFinetuning, SearchQnA and Translation
Fix shellcheck issue

Signed-off-by: ZePan110 <ze.pan@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.

4 participants