Skip to content

Replaced hard coded ports with dynamically assigned values from set_env file#17

Merged
okhleif-10 merged 5 commits intommqna-image-queryfrom
omar/ports
Dec 30, 2024
Merged

Replaced hard coded ports with dynamically assigned values from set_env file#17
okhleif-10 merged 5 commits intommqna-image-queryfrom
omar/ports

Conversation

@okhleif-10
Copy link
Copy Markdown
Collaborator

@okhleif-10 okhleif-10 commented Dec 20, 2024

Description

Removed all instances of hard coded ports in MMQnA and replaced them with values set from the set_env file in GenAIExamples.
Linked with This GenAIExamples PR

Issues

N/A

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

N/A

Tests

Tested by setting env vars, building, composing, and verifying logs.

Signed-off-by: okhleif-IL <omar.khleif@intel.com>
Signed-off-by: okhleif-IL <omar.khleif@intel.com>
Signed-off-by: okhleif-IL <omar.khleif@intel.com>
@dmsuehir
Copy link
Copy Markdown
Collaborator

Signed-off-by: okhleif-IL <omar.khleif@intel.com>
Signed-off-by: okhleif-IL <omar.khleif@intel.com>
Copy link
Copy Markdown
Owner

@mhbuehler mhbuehler left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@dmsuehir dmsuehir left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@okhleif-10 okhleif-10 merged commit 20a79e1 into mmqna-image-query Dec 30, 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

Successfully merging this pull request may close these issues.

3 participants