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

Update Camel-K examples for use with Camel-K version 1.0.0-M1 #179

Closed
wants to merge 2 commits into from

Conversation

lubuxnl
Copy link

@lubuxnl lubuxnl commented Sep 14, 2019

The tutorial uses Knative v0.7.1. According to this Camel-K issue the compatible Camel-K version for this Knative version is 1.0.0-M1. The instructions for the Camel-K examples need some editing too, but the sources for these web pages doesn't seem to be in a Git repo?!

@kameshsampath
Copy link
Contributor

@lubuxnl - thanks for the PR the docs for the camel-k is https://github.com/redhat-developer-demos/knative-tutorial-module-camelk

@kameshsampath
Copy link
Contributor

I will take review soon

@lubuxnl
Copy link
Author

lubuxnl commented Sep 17, 2019

Thanks for the links to the documentation @kameshsampath
Actually I am still in doubt whether it's a good idea to switch to Camel-K 1.0.0-M1, because of an issue I found with this version. I reported this on the Camel-K repository, but unfortunately it hasn't been solved yet. I did find a workaround however (using the integration kit for one example as input for another example), but that would introduce a dependency between the examples, especially between the cartoon message mover and cartoon message downloader. Is this a problem? An alternative solution would be to 'downgrade' the Knative version in the tutorial to the recommended version for Camel-K 0.3.3. What's your opinion on this?

@kameshsampath
Copy link
Contributor

@lubuxnl do you still want to do this? would you mind sending a new PR to https://github.com/redhat-developer-demos/knative-tutorial-module-camelk ? I will be adding a 1.0.0-M1 branch to camel-k soon .

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

Successfully merging this pull request may close these issues.

None yet

2 participants