-
Notifications
You must be signed in to change notification settings - Fork 584
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
Add readme description and initial road map #4
Conversation
README.md
Outdated
|
||
Events are everywhere. However, event publishers tend to describe events differently. | ||
|
||
The lack of a common way of describing events means developers must constantly re-learn how to receive events. This also limits the potential for libraries, tooling and infrastructure to aide the delivery of event data across environments, like SDKs, event routers or tracing systems. The portability and productivity we can achieve from event data is hindered overall. |
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.
Can we wrap things at 80 columns? it makes the github diff tool work much better.
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.
Yes!
README.md
Outdated
Enter OpenEvents, a specification for describing event data in a common way. OpenEvents seeks to ease event declaration and delivery across services, platforms and beyond. | ||
|
||
OpenEvents is a new effort and it's still under active development. However, its working group has received a surprising amount of industry interest, ranging from major cloud providers to popular SaaS companies. Our end goal is to offer this specification to the [Cloud Native Computing Foundation](https://www.cncf.io/). | ||
|
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 think there's an AI to add a "usecases" doc. Would it make sense to add at least a placeholder doc and href into this PR so people can then incrementally add to the doc over time?
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.
The use-cases are in another PR. Let me know if you think we should still link to it.
README.md
Outdated
# spec | ||
# OpenEvents | ||
|
||
Events are everywhere. However, event publishers tend to describe events differently. |
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.
Do we need to be explicit about whether we're talking about events as they appear "on the wire" vs "as presented to the function" or both? This is probably related to my "usecases" comment below - meaning I think our target usecases should make it clear where the level of interop is that we're shooting for.
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.
Yes, but I don't think that distinction needs to be made in this line.
README.md
Outdated
|
||
*January* | ||
|
||
* Establish governance, contributing guidelines and initial stakeholders. |
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.
Might be good to include things like: gather/agree to usecases and design goals for v1
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.
This one a good point. On one hand, it is useful to list all the use cases where event-driven architecture applies. On the other hand, it will keep the spec focussed if we define which use cases we are targetting and which we are explicitly leaving out at this stage.
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've added "agree upon design goals" to the Road map. Unsure about whether to limit the use-cases. We should discuss.
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.
You are right. It may be too early to have a conversation about limiting the use cases. 👍 for "agree upon design goals"
@ac360 did you want to address the comments before today's call? |
reopening to kick travis |
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.
LGTM
README.md
Outdated
|
||
*February* | ||
|
||
* Iterate on version 0.1 of the specification. |
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.
redundant. Last bullet about finalization assumes this.
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.
👍
Addressed the comments in a few minor updates. Signed-off-by: Austen Collins <austenATserverless.com> |
LGTM |
README.md
Outdated
environments, like SDKs, event routers or tracing systems. The portability and | ||
productivity we can achieve from event data is hindered overall. | ||
|
||
Enter OpenEvents, a specification for describing event data in a common way. |
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.
s/OpenEvents/CloudEvents
README.md
Outdated
productivity we can achieve from event data is hindered overall. | ||
|
||
Enter OpenEvents, a specification for describing event data in a common way. | ||
OpenEvents seeks to ease event declaration and delivery across services, |
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.
CloudEvents
README.md
Outdated
OpenEvents seeks to ease event declaration and delivery across services, | ||
platforms and beyond. | ||
|
||
OpenEvents is a new effort and it's still under active development. However, |
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.
CloudEvents
README.md
Outdated
*February* | ||
|
||
* Draft educational materials on use-cases. | ||
* Collaborate on libraries and supporting tools to use OpenEvents and |
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.
CloudEvents
README.md
Outdated
|
||
*March* | ||
|
||
* Collaborate on libraries and supporting tools to use OpenEvents and integrate |
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.
CloudEvents
README.md
Outdated
|
||
*April* | ||
|
||
* Collaborate on libraries and supporting tools to use OpenEvents and integrate |
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.
CloudEvents
overall looks good to me - just a few old refs to OpenEvents and a rebase needed. |
DCO check failing too |
@ac360 I've updated references from OpenEventing to CloudEvents. If you'll allow my user |
README.md
Outdated
platforms and beyond. | ||
|
||
OpenEvents is a new effort and it's still under active development. However, | ||
its working group has received a surprising amount of industry interest, |
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 don't think it is surprising at all, given that a lot of Cloud providers have event platforms that are so similar that a 3rd party can create an interoperable platform even without a standard :)
README.md
Outdated
ranging from major cloud providers to popular SaaS companies. Our end goal is | ||
to offer this specification to the [Cloud Native Computing Foundation](https://www.cncf.io/). | ||
|
||
## Roadmap |
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 think It would be good to split the roadmap into a separate doc, then will be easier to manage PRs separately!
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.
+1. I'll do this.
roadmap.md
Outdated
* Collaborate on libraries and supporting tools to use CloudEvents and | ||
integrate it with the ecosystem. | ||
* Draft documentation and user guides. | ||
* Build out promotional materials (website, logos, etc.) |
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.
minor nit but I think the period goes outside the parens. Below too.
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.
lol. Period is for the abbreviation. It also needs a period after the sentence though.
612ec20
to
856f3fe
Compare
LGTM |
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.
LGTM
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.
LGTM
Signed-off-by: ac360 <austen@serverless.com>
856f3fe
to
5f0b5df
Compare
Approved on Feb 1st call |
Adds a basic overview of the effort and an initial road map.
Signed-off-by: Austen Collins <austenATserverless.com>