-
Notifications
You must be signed in to change notification settings - Fork 340
Updated SearchQnA to use nginx like ChatQnA #1769
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 11 commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
fd27ee6
Updating SearchQnA xeon with nginx component
edlee123 f366570
Update xeon README to use nginx, and instruction to access the UI.
edlee123 aa96ffe
Reverting back changes
edlee123 7c8ae4a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b7f098f
Merge branch 'main' into searchqna_fix
edlee123 7d18b24
Merging README changes
edlee123 c8bdc04
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 4e13de2
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 9720561
Update SearchQnA/docker_image_build/build.yaml file to include nginx …
edlee123 5f5075e
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 e16d9d4
Merge branch 'main' into searchqna_fix
edlee123 a8daf7f
Fix unneeded breaks in code blocks, and markdown title formatting
edlee123 dd9c32e
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 f52c3db
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 57548b7
Add nginx to SearchQnA hpu compose.yaml
edlee123 5703ceb
Add nginx to SearchQnA amd compose.yaml
edlee123 20d0e5a
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 f9bc776
Add nginx to build_docker_images in unit tests
edlee123 5c34ed2
Add nginx to SearchQnA for amd rocm compose_vllm.yaml
edlee123 738cdca
Change playwright config so that unit testis accessing the app via ng…
edlee123 5fc9901
Add nginx to vllm rocm unit test
edlee123 8eea721
Modified SearchQnA xeon README.md to be more explicit about best prac…
edlee123 d69e65c
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 5984157
Updated diagram to include nginx, and use names like the container names
edlee123 03d7502
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 ea03294
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 758bc8c
Format the microservice chart
edlee123 ef8e53a
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 11437ad
Generalize the SearchQnA diagram so its not xeon specific
edlee123 12c42cf
Merge branch 'main' into searchqna_fix
edlee123 50b44cb
Merge branch 'main' into searchqna_fix
edlee123 b3119be
Update the SearchQnA diagram to use nginx and container labels
edlee123 3fae551
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] cbbe4eb
Merge branch 'opea-project:main' into searchqna_fix
edlee123 001be5c
Merge branch 'main' into searchqna_fix
edlee123 7d281a5
Merge branch 'main' into searchqna_fix
yinghu5 d2d7793
Merge branch 'main' into searchqna_fix
yinghu5 3f6b7d6
Merge branch 'main' into searchqna_fix
xiguiw f68509c
Updated the flow diagram to include Nginx, and with container labeling
edlee123 62f9bf5
Revert back to original xeon README.md
edlee123 74428a4
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 152c36f
Reverting back to xeon README.md
edlee123 d0d017d
Merge branch 'searchqna_fix' of github.com:edlee123/GenAIExamples int…
edlee123 b1c0850
Explicitly link to README.md of hardware implementations rather than …
edlee123 618fd75
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 11a19ba
Merge branch 'main' into searchqna_fix
edlee123 187456b
Merge branch 'main' into searchqna_fix
yinghu5 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| BACKEND_BASE_URL = 'http://backend_address:3008/v1/searchqna' | ||
| BACKEND_BASE_URL = '/v1/searchqna' |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.