-
Notifications
You must be signed in to change notification settings - Fork 20
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
🐛 [Bug]: panic: ./docs/swagger.yaml file does not exist #209
Comments
Local testing does not surface the same error. |
This appears to also be in issue with |
Hm, good catch 👀 I'm wondering if this is related to migratation from hertz to fiber |
Seems like Fiber swagger integration lacks a way to embed the swagger file, so they have a PR to bring that: |
@mkrueger12 I have it: #211 Had to switch to another official swagger ext and serve swagger spec via Glide API but it should work now 😌 |
…g swagger.yaml file (#211) This change fixes panics like "./docs/swagger.yaml is not found"
Final major improvements to streaming chat workflow. Fixed issues with Cohere streaming chat. Expanded and revisited Cohere params in config. ## Changelog ### Added - 🔧 #195 #196: Set router ctx in stream chunks & handle end of stream in case of some errors (@roma-glushko) - 🐛🔧 #197: Handle max_tokens & content_filtered finish reasons across OpenAI, Azure and Cohere (@roma-glushko) ### Changed - 🔧 💥 #198: Expose more Cohere params & fixing validation of provider params in config (breaking change) (@roma-glushko) - 🔧 #186: Rendering Durations in a human-friendly way (@roma-glushko) ### Fixed - 🐛 #209: Embed Swagger specs into binary to fix panics caused by missing swagger.yaml file (@roma-glushko) - 🐛 #200: Implemented a custom json per line stream reader to read Cohere chat streams correctly (@roma-glushko)
## Summary ✨ Bringing support for streaming chat in Glide (integrated with OpenAI, Azure OpenAI and Cohere) ✨ Started handling 401 errors to mark models as premaritally unavailable (e.g. when API key was not correct) 🐛 Fixing the panic related to swagger.yaml file 🐛 Fixing Anthropic chat workflow by passing API key correctly 🔧 Improved Cohere param config and validation ## Changelog ### Added - ✨Streaming Chat Workflow #149 #163 #161 (@roma-glushko) - ✨Streaming Support for Azure OpenAI #173 (@mkrueger12) - ✨Cohere Streaming Chat Support #171 (@mkrueger12) - ✨Start counting token usage in Anthropic Chat #183 (@roma-glushko) - ✨Handle unauthorized error in health tracker #170 (@roma-glushko) - 🔧 #195 #196: Set router ctx in stream chunks & handle end of stream in case of some errors (@roma-glushko) - 🐛🔧 #197: Handle max_tokens & content_filtered finish reasons across OpenAI, Azure and Cohere (@roma-glushko) ## Changed - 🔧 💥 #198: Expose more Cohere params & fixing validation of provider params in config (breaking change) (@roma-glushko) - 🔧 #186: Rendering Durations in a human-friendly way (@roma-glushko) ### Fixed - 🐛 Fix Anthropic API key header #183 (@roma-glushko) - 🐛 #209: Embed Swagger specs into binary to fix panics caused by missing swagger.yaml file (@roma-glushko) - 🐛 #200: Implemented a custom json per line stream reader to read Cohere chat streams correctly (@roma-glushko) ### Security - 🔓 Update crypto lib, golang, fiber #148 (@roma-glushko) ### Miscellaneous - 🐛 Update README.md to fix helm chart location #167 (@arjunnair22) - 🔧 Updated .go-version (@roma-glushko) - ✅ Covered the telemetry by tests #146 (@roma-glushko) - 📝 Separate and list all supported capabilities per provider #190 (@roma-glushko)
Great. I'll be able to take a look tomorrow. Thanks. |
@roma-glushko Seeing the same issue with swagger.json |
This is interesting. I will try to use dockercompose to pull the image and reproduce the issue that way |
@mkrueger12 This is what I see trying to run the image in docker: I'm little bit surprised to see swagger.json in there. In v0.0.3 I have used go:embed directive to pull its content into the binary. So the binary itself doesn't depend on any files anymore. Could you please check if 0.0.3 was really pulled there? |
And there is some issues with build info, so I've filled a task for look into it: #217 |
@roma-glushko I also could not reproduce it locally. I'll check the logs/build |
Looks like the build was correct. |
Issue appears resolved. It appears to be related to specifying "latest-alpine" in the chart instead of "0.0.3-alpine" where "latest-alpine" throws the error. |
@mkrueger12 I think I have got it. |
I am deploying the latest release via helm.
values.yaml
specification:image: repository: ghcr.io/einstack/glide tag: "0.0.3-rc.1-alpine" pullPolicy: IfNotPresent pullSecrets: []
On startup I receive the following error:
This is the first time I've experienced this error. Had success deploying it previously.
The text was updated successfully, but these errors were encountered: