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

Errors with log-sink and log-action Kamelet #1227

Closed
christophd opened this issue Dec 20, 2022 · 10 comments
Closed

Errors with log-sink and log-action Kamelet #1227

christophd opened this issue Dec 20, 2022 · 10 comments
Milestone

Comments

@christophd
Copy link
Contributor

I am seeing errors in YAKS tests while using log-action Kamelet:

ERROR [io.qua.run.Application] (main) Failed to start application (with profile prod): org.apache.camel.ResolveEndpointFailedException: Failed to resolve endpoint: log://log-action?level=INFO&logMask=false&multiline=false&showAllProperties=false&showBody=true&showBodyType=true&showCachedStreams=true&showExchangePattern=true&showHeaders=false&showProperties=false&showStreams=false due to: There are 1 parameters that couldn't be set on the endpoint. Check the uri if the parameters are spelt correctly and that they are properties of the endpoint. Unknown parameters=[{showCachedStreams=true}]

Do we need to remove showCachedStreams property form the Kamelet?

@christophd christophd changed the title Errors with log-action Kamelet Errors with log-sink and log-action Kamelet Dec 20, 2022
@christophd
Copy link
Contributor Author

same with log-sink Kamelet

@oscerd
Copy link
Contributor

oscerd commented Dec 20, 2022

ShowCachedStreams is in 3.20.0-SNAPSHOT. it should be released soon. I think this is the problem. But need to double check

@tadayosi
Copy link
Member

Yes, showCachedStreams is a new addition to 3.20.0 so it should be fixed once Camel is upgraded to 3.20.0.
https://camel.apache.org/components/next/log-component.html#_endpoint_options

@oscerd oscerd added this to the 3.20.0 milestone Jan 9, 2023
@oscerd
Copy link
Contributor

oscerd commented Jan 9, 2023

This is related to camel k runtime not aligned to 3.20.0, it should be something to report in camel k

@oscerd oscerd closed this as completed Jan 9, 2023
@christophd
Copy link
Contributor Author

Do we already have an issue on camel-k-runtime tracking this?

@squakez
Copy link
Contributor

squakez commented Jan 9, 2023

I think we already fixed that, making main Camel K to depend on the latest Kamelets tag aligned with the expected Camel version.

@christophd
Copy link
Contributor Author

@squakez I did rebase the PR on main branch but still the log-action Kamelet is not able to run properly with Camel K as shown in the tests https://github.com/apache/camel-kamelets/actions/runs/3872513218/jobs/6601438316#step:11:817

@squakez
Copy link
Contributor

squakez commented Jan 9, 2023

Camel K main is built against a stable relase of Kamelets 0.10.0: https://github.com/apache/camel-k/blob/main/script/Makefile#L88 - this is because the latest version of Camel K still uses Camel 3.19. If you need a Camel K operator (and runtime) available for Camel 3.20, I guess you'll need to wait for the related update (which is mainly driven by Camel Quarkus).

@christophd
Copy link
Contributor Author

Unfortunately this is not a good situation for the CI tests in this Kamelet catalog repository. They will keep failing until the Camel K runtime with Camel 3.20 is available.

I see an option to move from Camel K runtime to JBang in the tests in order to align with Camel 3.20. I really would like to keep Camel K runtime in Kamelets tests, too but waiting such a long time is not a good option IMO

@squakez
Copy link
Contributor

squakez commented Jan 10, 2023

Yes. As Kamelets are going to be completely decoupled from Camel K, it makes sense to use some other way of testing it against the latest Camel version.

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

4 participants