-
Notifications
You must be signed in to change notification settings - Fork 183
Hyperloop Gin server and logs forwarding #1233
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
Conversation
|
|
||
| # ======== IP ADDRESSES ==================== | ||
|
|
||
| // todo: (2025-09-22): this can be removed when all orchestrator will be rolled with internal logs collector server |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
task for future IaC removal https://linear.app/e2b/issue/ENG-3029/remove-vector-public-load-balancer-from-iac
Thank you, good bot! |
spec/openapi-hyperloop.yml
Outdated
| schema: | ||
| $ref: "#/components/schemas/Me" | ||
|
|
||
| /: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you move it under /logs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then we need to send it somehow to envd. Now we are using address parameter in the init request, and with adding back compatibility, we would need to send /logs here too, to make it not re-usable for different things.
Maybe we can do /logs, update in newer envds and do / just for back compatibility in http server?
|
dd5cede is just wip for now. Moves url for sandbox logs to |
dd5cede to
569ed9f
Compare
LOGS_COLLECTOR_PUBLIC_IPin orchestrator, replaced with hyperloop