-
Notifications
You must be signed in to change notification settings - Fork 219
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
Reduce amount of services #1339
Comments
I think that's a great idea. We could also look into improving the HTTP ingress/load balancer to detect the things cluster node where eg a thing with ID x is running to directly route the Http request to the correct node. |
Having slept one night over it I fear that we cannot get rid of "gateway" so easily.
Those 2 endpoints - similar to "connectivity" - directly process Ditto Protocol messages for all entities (things + policies + search commands). So FMPOV we could get of "concierge", but would still require the edges "gateway" and "connectivity". |
You're right. I forgot about websockets. One solution could be to move those "combined" api to connectivity, so connectivity becomes the only edge service. |
…as architecture simplification in Ditto 3.0 Signed-off-by: Thomas Jaeckle <thomas.jaeckle@bosch.io>
…s architecture simplification in Ditto 3.0 Signed-off-by: Thomas Jaeckle <thomas.jaeckle@bosch.io>
[#1339] adding DADR-0007: Removal of Concierge service as architecture simplification in Ditto 3.0
I think it would be a good idea to eliminate the two services:
The plan that came to my mind is to move the HTTP API and also command authorization to the respective service the command is related to. So each of the remaining services will have an HTTP API and handles the authorization inside their service instead of concierge.
Remaining services would be:
Of course this should not be reimplemented for all services but instead we could introduce a library for common HTTP / Authorization topics and only implement the specific parts in the respective service.
Benefits:
Downsides:
Please feel free to add arguments for or against this change. (Of course this would require a Ditto 3.0 release)
The text was updated successfully, but these errors were encountered: