Skip to content

fix(lapis): set a low timeout on calling SILO info so that the health endpoint doesn't block if SILO doesn't respond#1593

Open
fengelniederhammer wants to merge 1 commit intomainfrom
infoTimeout
Open

fix(lapis): set a low timeout on calling SILO info so that the health endpoint doesn't block if SILO doesn't respond#1593
fengelniederhammer wants to merge 1 commit intomainfrom
infoTimeout

Conversation

@fengelniederhammer
Copy link
Contributor

@fengelniederhammer fengelniederhammer commented Mar 12, 2026

I tested it locally by start SILO locally with a "sleep one minute" in the info endpoint.

PR Checklist

- [ ] All necessary documentation has been adapted.
- [ ] All necessary changes are explained in the llms.txt.
- [ ] The implemented feature is covered by an appropriate test.

… endpoint doesn't block if SILO doesn't respond
@fengelniederhammer fengelniederhammer self-assigned this Mar 12, 2026
@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lapis Ready Ready Preview, Comment Mar 12, 2026 8:37am

Request Review

@fengelniederhammer fengelniederhammer marked this pull request as ready for review March 12, 2026 10:47
Copy link
Contributor

@fhennig fhennig left a comment

Choose a reason for hiding this comment

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

sounds good to me!

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