-
Notifications
You must be signed in to change notification settings - Fork 0
docs(design): service provider #19
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
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
d82a9f9 to
ef1b151
Compare
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
|
Regarding
The shall not be used to
Currently we are violating rule (a). This is because there is no "release-channel"/"marketplace" API available that would provide this information for all service providers. However, this is where we want to go in the mid to long term. "release-channel"/"marketplace" API would be available for both the service providers and the consumers of the service provider api. |
maximiliantech
left a comment
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.
I love this document!! 🚀
I have some minor comments. What do you think?
On-behalf-of: @SAP [email protected] Signed-off-by: Christopher Junk <[email protected]>
maximiliantech
left a comment
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.
That's great! I like the changes. That definitely makes it more precise. 👍
jakobmoellerdev
left a comment
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.
Sorry the review took longer than expected. I have two major grypes with this one. I read through the doc but honestly it was very high level. I did not get a lot of actual design conclusions from it, but that may just be me
- As I understand theres gonna be an additional library on top of controller runtime. I would love if we can keep as much as possible a low level integration and we can orchestrate cluster access ourselves
- I would love to understand technical implications of the runtime abstraction and I need to understand how we will actually code a multi-runtime abstraction on top of our existing controllers
Hi @jakobmoellerdev thanks for the feedback. the doc is supposed to be very high level and imo should stay that way but things should get clearer as soon as we have the service provider template ready. Then we can iterate over all the technical details |
What this PR does / why we need it:
This document outlines the
ServiceProviderdomain and its technical considerations within the context of the openMCP project, providing a foundation for understanding its architecture and operational aspects.Which issue(s) this PR fixes:
Fixes openmcp-project/backlog#332
Special notes for your reviewer:
General RFC
Release note: