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

[Registry] New feature requirements - Announcements #829

Open
clemensv opened this issue Jun 10, 2021 · 7 comments
Open

[Registry] New feature requirements - Announcements #829

clemensv opened this issue Jun 10, 2021 · 7 comments

Comments

@clemensv
Copy link
Contributor

@deissnerk and myself (and several of our respective coworkers) have been working through a number of scenarios in the last few months. There will be some PRs based on the outcome of this work, but I'd like to capture the resulting requirements for the group before we come around with solutions for them.

In Discovery, we have so far only really considered the side of the discovering party and how we can tell a "discoverer" what services (subscription managers) are available and what events can be subscribed from them.

As we as trying to enable event flow between two rather large platform ecosystems, we find that we als need to consider the side of the publishers and how they can "announce" to a middleware infrastructure that they have services and can raise events.

An exemplary use case is that there is an event publisher somewhere in a solution inside the SAP platform and there is a prospective event subscriber like an Azure Function inside an Azure platform subscription. We need to make it possible for that subscriber to make it known to the SAP event bus that it has a certain set of evets that it can raise and the SAP event bus must then be able to make it known to Azure Event Grid that such events may be sent.

Why do we need that information? At a minimum because there ought to be a drop-down box in the Azure portal where the developer can select that source and the kinds of events they want to subscribe to.

The present pull-style discovery model is insufficient for this. We need to be able to push the required discovery metadata along the future event flow path and get it near the prospective subscriber.

The fundamental assumption we are making here is that we establish trust at the platform level (that is a separate discussion that I will also file an issue for) and are flowing consolidated event streams between SAP and Azure in either direction. The (de-)multiplexing model we have allows for those event streams to be spliced up into per-tenant streams. That creates a flow path from a tenant scope inside of the SAP platform to a tenant scope inside the Azure platform.

It should then be possible for a custom application inside that tenant scope of the SAP platform to announce that it makes a certain set of events available for consumption. It basically announces the "service object" that we already have in the discovery API. That announcement should then be able to follow the established flow path and become available in the discovery API endpoint for the respective Azure tenant scope.

The announcement is an event. The "target" of such announcements is new endpoint on the discovery service that accepts such events and then decides whether to make them available to its callers. The relationship between the publisher and the discovery service may be direct, meaning that there is no subscription setup required to send the event. An implementation may also choose to subscribe for announcements at the publisher or the publisher's middleware. An announcement may end up being broadcast to several discovery services.

An announcement can take two forms: The publisher can announce on such a consolidated event path that there will be events of that sort coming and may then start sending them right away without requiring a subscription. For dispatching events out of that consolidated stream into particular endpoints such as a serverless function, there would still be a subscription manager that manages access to the consolidated stream. That approach has the downside of sending events through that consolidated channel that have no active subscribers, but the upside of avoiding the complex bookkeeping of the second approach:

The publisher may announce and the announcement will, as in the current service object, declare a subscription manager endpoint. When that announcement is then accepted by a discovery service, it might quite well be that while the discovery service can communicate with the publisher's subscription manager endpoint, prospective subscribers might not because of access permissions or network segmentation. In that case, the discovery service might host a subscription manager proxy. The subscription manager proxy is reachable by prospective subscribers "on the inside". The service objects that are returned by the discovery service then have an overridden subscription manager endpoint that points to the proxy. Each (un-)subscription made on the proxy then translates to the same operation on the upstream publisher. That model can then be chained. What makes that approach complex and potentially brittle is that intermediaries can't trust their subscribers to clean up subscriptions properly, which means that they have to do their own bookkeeping.

@duglin duglin changed the title Discovery: New feature requirements - Announcements [Registry] New feature requirements - Announcements Feb 23, 2023
@duglin
Copy link
Collaborator

duglin commented Apr 18, 2023

@clemensv @deissnerk do we still need this issue?

@duglin
Copy link
Collaborator

duglin commented Sep 21, 2023

@clemensv @deissnerk ping!

@deissnerk
Copy link
Contributor

Yes, I think it is still needed, although it takes us a bit longer to get there. I think the announcements could be realized by events about an xRegistry. In fact, what @clemensv described above, is a complex target scenario that might need several enhancements of our current concepts. Therefore, I think it makes sense to keep it here as a reminder.

@duglin
Copy link
Collaborator

duglin commented Sep 21, 2023

Is this a CE issue or an xReg issue?

@deissnerk
Copy link
Contributor

Good question. It's a bit of both. It may be more about xRegistry. As the described scenario is about CloudEvents, we can as well leave it here.

@github-actions
Copy link

This issue is stale because it has been open for 30 days with no
activity. Mark as fresh by updating e.g., adding the comment /remove-lifecycle stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants