feat(gateway): add support for connecting to other gateways#158
feat(gateway): add support for connecting to other gateways#158orlandohohmeier merged 1 commit intoalphafrom
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
orlandohohmeier
force-pushed
the
orlandohohmeier/fault-tollerant-gateway
branch
from
f628cd5 to
d97eeef
Compare
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Gateways can now connect to other gateways for network entry, similar to how workers and schedulers connect. The gateway attempts to connect to all provided gateway addresses during startup. By deploying more than one gateway and connecting them together, one can improve the systems reliability. Co-Authored-By: Claude <noreply@anthropic.com>
orlandohohmeier
force-pushed
the
orlandohohmeier/fault-tollerant-gateway
branch
from
d97eeef to
63bf2ad
Compare
|
I tested the change and it partially worked. I was able to start two gateway. One with a public IP and one within a private environment. The data node was connected to the first gateway and all workers to the second. The gateways connected as expected. However, I ran into a relayed connection issue which finally resulted in an error on the parameter server. I might have just missconfigured something. |
l45k
left a comment
l45k
left a comment
There was a problem hiding this comment.
With #176 merged the error still occures, but the retry is successful and the process continues. My understanding, we first use a relayed connection and once this is closed we are able to create a direct connection. Interessting is that this can take somehow longer. I see two errors in the logs. Thus, the first attempt isn't succesful.
|
🎉 This PR is included in version 1.0.0-alpha.22 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
2 participants
Gateways can now connect to other gateways for network entry, similar to how workers and schedulers connect. The gateway attempts to connect to all provided gateway addresses during startup. By deploying more than one gateway and connecting them together, one can improve the systems reliability.