-
Notifications
You must be signed in to change notification settings - Fork 4
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
README tweaks #24
Open
duglin
wants to merge
1
commit into
Peripli:master
Choose a base branch
from
duglin:tweaks
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
README tweaks #24
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,12 +38,13 @@ Additionally, this specification defines the following terms: | |
|
||
- *Service Manager*: A component that acts as a platform as per OSB API and exposes a platform API. | ||
It allows the management and registration of service brokers and platform instances. | ||
Also acts as a central service broker. | ||
Also acts as a central service broker via the Service Broker Proxy. | ||
|
||
- *Service Broker Proxy*: A component that is associated with a platform that is registered at the Service Manager | ||
and is a service broker as per the OSB API specification. | ||
The proxy replicates the changes in the Service Manager into the platform instance in which the proxy resides. | ||
Service Brokers Proxies are in charge of registering and deregistering themselves at the platform it is responsible for. | ||
- *Service Broker Proxy*: | ||
The entity that is registered with the platform instance as an OSB API | ||
compliant service broker. The proxy works with the service manager to | ||
manage the OSB API communications between the platform and the actual | ||
service brokers. | ||
|
||
## Motivation | ||
|
||
|
@@ -68,13 +69,25 @@ The main part is the core component. | |
It is the central registry for service broker and platform registration, as well as for tracking of all service instances. | ||
This core component communicates with the registered brokers and acts as a platform per Open Service Broker specification for them. | ||
|
||
In each Platform Instance resides a component called the Service Broker Proxy. | ||
It is the substitute for all brokers registered at the Service Manager | ||
in order to replicate broker registration and access visibility changes in the corresponding Platform Instance. It also delegates lifecycle operations to create/delete/bind/unbind service instances from the corresponding Platform Instance to the Service Manager and the services registered there. | ||
|
||
When a broker is registered or deregistered at the Service Manager, | ||
the Service Broker Proxy registers or deregisters itself in the name of this service broker. | ||
From a Platform Instance point of view, the broker proxy is indistinguishable from the real broker because both implement the OSB API. | ||
For each Platform Instance there is an OSB API compliant Service Broker | ||
registered called the Service Broker Proxy, which is the second part of the | ||
Service Manager. | ||
The proxy is the substitute for all brokers registered at the Service Manager. | ||
It works with the Service Manager to manage the Platform Instance's view of | ||
the services and instances available to the Platform Instance. | ||
It also delegates lifecycle operations to create/delete/bind/unbind service | ||
instances from the corresponding Platform Instance to the Service Manager and | ||
the services registered there. | ||
|
||
As brokers are (de)registered at the Service Manager, the Platform Instance's | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Again, we may decouple the both roles:
|
||
view of the list of services will change. In some cases new Service Broker | ||
Proxies will be added to Platform Instances (so there is a 1:1 relationship | ||
between "real brokers" and "proxy brokers"). In other cases there will be | ||
a single Service Broker Proxy registered to Platform Instances and it will | ||
expose all services from all "real brokers". This specification allows for | ||
both types of models. In either case, from a Platform Instance point of view, | ||
the broker proxy is indistinguishable from the real broker because both | ||
implement the OSB API. | ||
|
||
When the Platform Instance makes a call to the service broker, for example to provision a service instance, | ||
the broker proxy accepts the call, forwards it to the Service Manager, which in turn forwards it to the real broker. | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
We miss here that SBProxy is needed to register a platform in Service Manager.
This current case this is an implementation detail, but we need to describe another role "SM Platform agent" that describe this.
In old description we assume that SBProxy accommodate this role too.