-
Notifications
You must be signed in to change notification settings - Fork 350
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
Camel K seems to fail with Knative 0.23 #2343
Comments
Yes, we have to do a round of upgrade to the latest Knative API versions, and drop the ones that have been deprecated / removed from recent Knative versions. I've been holding it back because of #2195. We need to make sure Knative test suite runs on newest versions. |
@astefanutti Is there anything we should consider for OpenShift 3 support when upgrading Knative like Ingress #2084? Or can we simply jump into the latest version if the test suite passes? |
@tadayosi we should not concern with OpenShift 3. As soon as the test suite passes with the latest version, it's good to go. |
Due to apache#2195 and apache#2343, eventing is still required to stay lower than v0.18.0.
Upgrading to Knative Eventing 0.23 drops support for the following API versions: * messaging.knative.dev/v1beta1 * sources.knative.dev/v1alpha1 * sources.knative.dev/v1alpha2
Upgrading to Knative Eventing 0.23 drops support for the following API versions: * messaging.knative.dev/v1beta1 * sources.knative.dev/v1alpha1 * sources.knative.dev/v1alpha2
Fixed with #2424. Thanks! |
Probably it's because for example
messaging.knative.dev/v1beta1
is dropped and now Knative supports onlymessaging.knative.dev/v1
:https://knative.dev/docs/reference/api/eventing/
All the examples that still use
messaging.knative.dev/v1beta1
InMemoryChannel
are not deployable with Knative 0.23.The text was updated successfully, but these errors were encountered: