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

README.md states components are beta, but the channel is still in alpha? #623

Open
brigitops opened this issue Nov 21, 2024 · 2 comments
Open

Comments

@brigitops
Copy link

brigitops commented Nov 21, 2024

README.md states that components are in BETA.

However, the channel is still under the v1alpha1 package and the main knative documentation states only that there is NATSS (the old NATS streaming channel) in alpha status, here: https://github.com/knative/docs/blob/main/docs/eventing/channels/channels.yaml

Which is correct?

If I have misunderstood, and the README is referring to something other than the channel as being in BETA, I anyway think the README should be explicit about the status of the channel as this is going to be of most interest to users of the knative extension.

I'm happy to raise PRs but it's unclear what the change should be.

Once we've cleared this up, I think it would also be helpful to raise a PR against knative docs to replace references to NATSS with Nats Jetstream, at the appropriate maturity level, as that's also confusing. However, I don't want to do that until I am sure the README.md here is accurate and the maturity level is well understood.

@astelmashenko
Copy link
Member

Good day, I recently removed nats-streaming, which was beta, I tried to cleanup all places but certainly missed few.
Starting 1.16 there is only JetSteam based channel implementation, which is v1alpha1 in codebase, but actually we are using it in production for around 18 months. It is pretty stable.
The reason it is alpha is probably it is based on JetStream push consumers, which are deprecated now, and I need to migrate them to pull-consumers.

@brigitops
Copy link
Author

Hi! Thanks for your reply!

I suggest we update the README to say that the components are in alpha, and to call out that JetStream is supported?

We can update the README again once the component is officially decided to be in beta, and follow it up with a PR on knative website docs?

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

No branches or pull requests

2 participants