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

Add a roadmap and vision doc for cdevents #9

Merged
merged 2 commits into from
Feb 1, 2022

Conversation

afrittoli
Copy link
Contributor

Add a roadmap doc for cdevents that include the mission and vision
for the project too.

Signed-off-by: Andrea Frittoli [email protected]

Add a roadmap doc for cdevents that include the mission and vision
for the project too.

Signed-off-by: Andrea Frittoli <[email protected]>
Copy link

@fdegir fdegir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few comments inline.

roadmap.md Outdated

The mission of CDEvents project is:

> Provide interoperability in the continuous delivery ecosystem through a common events format
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it make sense to call this a protocol rather than a format?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I steered away from the term protocol on purpose because I didn't want to give the impression that we're re-inventing CloudEvents or MQTT which is a common question I've got on cdevents.
That said I do like the term protocol so I'm open to suggestion about how to use the word and avoid confusion.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the word specification better to use?

Copy link

@fdegir fdegir Jan 19, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't want to give the impression that we're re-inventing CloudEvents or MQTT which is a common question I've got on cdevents

couldn't this protocol be seen as an extended/enriched/ version of CloudEvents assuming CD events will stay compliant with CloudEvents so calling this a protocol still makes sense to me.

But I also understand your concern so I'm fine either way.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I agree the term "protocol" is appropriate, and that's what I used to call it in fact - the only reason I changed term is because I noticed it triggered some confusion.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I switched back to "protocol"


- Capture our key use cases and design decisions in the [CDEvents primer](primer.md) document
- Develop the spec to fully support our [key use cases](#use-cases)
- Validate and demonstrate the spec through proofs-of-concept
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could this be called "reference implementation" so whoever wants to adhere to the protocol can use what is implemented by CD Events as reference?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Our proofs of concept would not be reference implementations of the spec, that will be in the SDK instead.
They could be seen as reference architectures - but I'd like to make it clear that we plan to go beyond diagram and slides.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would agree with @afrittoli that the POC is not a reference implementation.

Copy link

@fdegir fdegir Jan 19, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should have said "would you think of having reference implementations in addition to PoCs?".

The reason why I ask this is that I may be using certain pipeline technologies A and B which may not have adapted CD Events but if CD Events could have a reference implementation (or deployments which I should probably have used originally) for the technologies C and D, I could look at them to get A and B on par with C and D so what I'm after is more than and addition to SDKs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I see what you mean.

Well, ideally we'd like to help projects produce cdevents natively, but the reality is that for the PoCs we'll probably need some "adapters" - like we had for the initial PoC which would translate Keptn messages into cdevents and back.

Maintaining these kind of adapters as general purpose reference implementations is bit beyond our capacity as a community right now, but we could definitely host them in the project if we had the contributors for them.

I will add a note about this to the roadmap.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Andrea!

roadmap.md Outdated
- Capture our key use cases and design decisions in the [CDEvents primer](primer.md) document
- Develop the spec to fully support our [key use cases](primer.md#use-cases)
- Validate and demonstrate the spec through proofs-of-concept
- Specify and rework the CloudEvents binding, re-create the SDK for the `go` language
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be overkill to also aim for having a Python SDK this year? I would be willing to contribute to it, and I have some colleagues who should be able to spend some time on it as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added it to the roadmap

Include the work on reference implementations, or "adapters" that
will be required to produce / consume cdevents in various platforms.

Relocate the use cases to the primer.md document.

Signed-off-by: Andrea Frittoli <[email protected]>
Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the update!

@afrittoli
Copy link
Contributor Author

Thanks, as discussed in the WG, I'm going to merge this.

@afrittoli afrittoli merged commit 30f0b10 into cdevents:main Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants