Skip to content
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

ProductivitySuite: Fix CI failure with faqgen microservice #1271

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

hteeyeoh
Copy link
Collaborator

Description

Endpoint content-type change in faqgen microservice causing failure in CI.

Issues

#1270

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

n/a

Copy link

github-actions bot commented Dec 19, 2024

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

@hteeyeoh hteeyeoh force-pushed the fix_prod_ci branch 9 times, most recently from 5256a1c to cd492ed Compare December 19, 2024 09:03
Endpoint content-type change in faqgen microservice causing failure in
CI.

Github Issue: opea-project#1270

Signed-off-by: Yeoh, Hoong Tee <[email protected]>
@hteeyeoh
Copy link
Collaborator Author

Code changes able to pass on local machine but in ci test it failed with langserve error
| Traceback (most recent call last):
| File "/home/user/.local/lib/python3.11/site-packages/starlette/responses.py", line 255, in wrap
| await func()
| File "/home/user/.local/lib/python3.11/site-packages/starlette/responses.py", line 244, in stream_response
| async for chunk in self.body_iterator:
| File "/home/user/comps/llms/faq-generation/tgi/langchain/llm.py", line 77, in stream_generator
| from langserve.serialization import WellKnownLCSerializer
| File "/home/user/.local/lib/python3.11/site-packages/langserve/init.py", line 8, in
| from langserve.client import RemoteRunnable
| File "/home/user/.local/lib/python3.11/site-packages/langserve/client.py", line 24, in
| from httpx._types import AuthTypes, CertTypes, CookieTypes, HeaderTypes, VerifyTypes
| ImportError: cannot import name 'VerifyTypes' from 'httpx._types' (/home/user/.local/lib/python3.11/site-packages/httpx/_types.py)
+------------------------------------

  • '[' 200 -ne 200 ']'

@hteeyeoh
Copy link
Collaborator Author

This error is langserve upstream issue as mentioned in this issue as well opea-project/GenAIComps#969

@hteeyeoh hteeyeoh force-pushed the fix_prod_ci branch 3 times, most recently from e632aa9 to 5fd8950 Compare December 23, 2024 01:24
@hteeyeoh hteeyeoh force-pushed the fix_prod_ci branch 2 times, most recently from 28bf790 to efcede6 Compare December 23, 2024 04:04
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.

1 participant