Skip to content
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

Allow multiple controllers #29

Open
michaelsproul opened this issue Oct 4, 2023 · 3 comments
Open

Allow multiple controllers #29

michaelsproul opened this issue Oct 4, 2023 · 3 comments

Comments

@michaelsproul
Copy link
Member

michaelsproul commented Oct 4, 2023

It could be useful to allow multiple CLs to connect to the /canonical endpoint.

Currently this is possible, but it won't work particularly well if the CLs send conflicting updates. It's basically the same as if the controlling CLs were directly connected the EL.

Instead, Eleel could act as a mediator, detecting conflicts and preventing them from reaching the EL. The hard part would be tracking which CL to follow in the event of a conflict. This requires some state beyond just "which CL sent me an update first", which would have the EL re-orging all over the place.

@Savid
Copy link
Contributor

Savid commented Oct 20, 2023

would the primary use case for this being able to restart one of the "canonical" CL gracefully and not impacting other non canonical CLs attached? If so this would be an awesome feature!

@michaelsproul
Copy link
Member Author

michaelsproul commented Oct 20, 2023

@Savid Yeah, exactly. It would provide redundancy + resilience for all the (canonical) CLs sharing the EL.

I'm not sure if I'll have time to work on this soon, but could help mentor a PR if you wanted to give it a shot

@Savid
Copy link
Contributor

Savid commented Oct 20, 2023

I'm not sure if I'll have time to work on this soon, but could help mentor a PR if you wanted to give it a shot

Yep sounds good, can have a crack in the next week or two

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

No branches or pull requests

2 participants