Skip to content

Add missing env vars for MMQnA UI data prep endpoints#62

Merged
dmsuehir merged 3 commits into
mmqna-phase3from
dina/gaudi-fixes
Mar 18, 2025
Merged

Add missing env vars for MMQnA UI data prep endpoints#62
dmsuehir merged 3 commits into
mmqna-phase3from
dina/gaudi-fixes

Conversation

@dmsuehir
Copy link
Copy Markdown
Collaborator

@dmsuehir dmsuehir commented Mar 14, 2025

Description

This PR updates the compose.yaml files for both Gaudi and AMD ROCm to set the data prep get files and delete file endpoints in the MMQnA UI container.

Issues

The buttons to list and delete files in the vector store do not work on Gaudi (and ROCm) without this update.

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

None

Tests

Manually tested UI on Gaudi.

Signed-off-by: dmsuehir <dina.s.jones@intel.com>
@dmsuehir dmsuehir requested a review from mhbuehler as a code owner March 14, 2025 21:41
Signed-off-by: dmsuehir <dina.s.jones@intel.com>
Copy link
Copy Markdown
Collaborator

@HarshaRamayanam HarshaRamayanam left a comment

Choose a reason for hiding this comment

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

LGTM

@dmsuehir dmsuehir merged commit 8d5690e into mmqna-phase3 Mar 18, 2025
@dmsuehir dmsuehir deleted the dina/gaudi-fixes branch March 18, 2025 21:16
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.

2 participants