You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/10/01 05:45:49 UTC
[GitHub] [camel-k] tadayosi opened a new pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
tadayosi opened a new pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665
<!-- Description -->
<!--
Enter your extended release note in the below block. If the PR requires
additional action from users switching to the new release, include the string
"action required". If no release note is required, write "NONE".
You can (optionally) mark this PR with labels "kind/bug" or "kind/feature" to make sure
the text is added to the right section of the release notes.
-->
**Release Note**
```release-note
Jolokia is upgraded to 1.7.1
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-935682474
Would you be aware of any changes introduced between Jolokia 1.7.0 and 1.7.1 that could relate to that exception?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-935538177
Does anyone have any ideas about the exception? Currently I find it hard to detect where it comes from.
> Exception in thread "jolokia-0" java.lang.NoSuchMethodError: 'org.jolokia.shaded.org.json.simple.JSONObject org.jolokia.http.HttpRequestHandler.handleThrowable(java.lang.Throwable)'
This exception doesn't occur when you attach the jolokia agent to a Java app locally, and it doesn't even in a container image with `adoptopenjdk/openjdk11:slim`. So it appears to be related to the way Camel K assembles the classpath for an integration app.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-935635576
And indeed if I copy the entire app & dependencies from a deployed pod on k8s and run it locally, it reproduces the exception.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-932024368
@tadayosi it seems there is an exception thrown for some reasons during the _Jolokia_ trait e2e test execution:
```
Exception in thread "jolokia-0" java.lang.NoSuchMethodError: 'org.jolokia.shaded.org.json.simple.JSONObject org.jolokia.http.HttpRequestHandler.handleThrowable(java.lang.Throwable)'
at org.jolokia.jvmagent.handler.JolokiaHttpHandler.doHandle(JolokiaHttpHandler.java:245)
at org.jolokia.jvmagent.handler.JolokiaHttpHandler.handle(JolokiaHttpHandler.java:180)
at jdk.httpserver/com.sun.net.httpserver.Filter$Chain.doFilter(Filter.java:77)
at jdk.httpserver/sun.net.httpserver.AuthFilter.doFilter(AuthFilter.java:82)
at jdk.httpserver/com.sun.net.httpserver.Filter$Chain.doFilter(Filter.java:80)
at jdk.httpserver/sun.net.httpserver.ServerImpl$Exchange$LinkHandler.handle(ServerImpl.java:717)
at jdk.httpserver/com.sun.net.httpserver.Filter$Chain.doFilter(Filter.java:77)
at jdk.httpserver/sun.net.httpserver.ServerImpl$Exchange.run(ServerImpl.java:689)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
at java.base/java.lang.Thread.run(Thread.java:829)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-937395664
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-937395664
Indeed Jolokia 1.7.1 pom brings `com.googlecode.json-simple:json-simple` and `org.jolokia:jolokia-core` to the classpath of the pod. Removing them manually makes it run fine locally. Let's see what's the best way to exclude them.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-932025017
Yes, I'm looking into it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-937625232
Created PR apache/camel-k-runtime#742 to include jolokia-jvm to camel-catalog so that the issue is fixed.
We need to wait for a new camel-k-runtime release to merge this one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
astefanutti commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-935741502
@tadayosi right, it's possible having the duplicated classes from the shading and the POM, causes the issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] tadayosi commented on pull request #2665: chore(jolokia): upgrade to Jolokia 1.7.1
Posted by GitBox <gi...@apache.org>.
tadayosi commented on pull request #2665:
URL: https://github.com/apache/camel-k/pull/2665#issuecomment-935730013
@astefanutti The main changes in 1.7.1 is this simple commit:
https://github.com/rhuss/jolokia/commit/bde11f8a1fcc3fac5edddc358da6672e467de2c1
but this seems to have introduced some unsolicited dependencies (i.e. `com.googlecode.json-simple:json-simple`) to its pom:
https://repo1.maven.org/maven2/org/jolokia/jolokia-jvm/1.7.1/jolokia-jvm-1.7.1.pom
I'll compare it with the dependencies set from the working Jolokia 1.7.0 deployment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org