-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
feat(p2p): add support for configuration of edgevpn listen_maddrs, dht_announce_maddrs and bootstrap_peers #4200
feat(p2p): add support for configuration of edgevpn listen_maddrs, dht_announce_maddrs and bootstrap_peers #4200
Conversation
✅ Deploy Preview for localai ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
f63b40a
to
0a181d9
Compare
e4dfe2e
to
13a9fa7
Compare
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.
Thank you @mintyleaf ! was about to get to it myself at some point, but still stuck into other PRs/fixes I have laying around. Really appreciated!
@mudler keep in mind that edgevpn dependency needs to be updated to v0.28.4 for it to work I don't get how that fancy chore workflow system works and if i can bump deps by myself |
Signed-off-by: mudler <[email protected]>
Just pushed in the branch the bump. For future reference, the command that I had to run is: |
81e5eb9
to
181eaa3
Compare
in order to build this i needed to do
+ i'm aware of some fancy localai-bot deps update automatization, so i was confused and wasn't sure if it's ok to update them like this manually |
Description
Notes for Reviewers
Signed commits