Replies: 13 comments 13 replies
-
|
@skarsaune Please join the discussion! |
Beta Was this translation helpful? Give feedback.
-
That's exactly the plan. Recently I brought back real JDK6 compatibility (ff4f020) while keeping all the tests working on JDK17. |
Beta Was this translation helpful? Give feedback.
-
|
I recently switched to springboot3 and unfortunately jolokia doesn't seem to work there anymore. All in all I find the library (jolokia) perfect to allow an admin view on the application |
Beta Was this translation helpful? Give feedback.
-
|
See also #568. |
Beta Was this translation helpful? Give feedback.
-
|
Hi. Do you have an idea when Jolokia 2 with support for Spring Boot 3 will be available? |
Beta Was this translation helpful? Give feedback.
-
|
We're in the process of preparing a release, which should happen within few next weeks. Spring Boot 3 related stuff may be added/checked before or after the release though. So please give us some time. |
Beta Was this translation helpful? Give feedback.
-
|
Ok, very good. Looking forward to it. |
Beta Was this translation helpful? Give feedback.
-
|
@perspilling In the meantime, we've already released a milestone 2.0.0-M4. You can try it early and see if it works as you expect. |
Beta Was this translation helpful? Give feedback.
-
|
I can try that. What Maven dependency should I use in the client application? |
Beta Was this translation helpful? Give feedback.
-
|
In the SB 2 app I use jolokia-core |
Beta Was this translation helpful? Give feedback.
-
Yes, because artifacts in Jolokia 2.0 were reorganized back in 2016 ;) Part of Jolokia 2 documentation will be a listing of all available Maven artifacts. Please for now see the description of #557. |
Beta Was this translation helpful? Give feedback.
-
|
Please start with 3 jars:
|
Beta Was this translation helpful? Give feedback.
-
|
I added this to a vanilla hello-world SB 3 app with spring-boot-admin-starter-client 3.1.5 (generated by https://start.spring.io). It didn't work. It seems that the JmxServiceActivator is not called. |
Beta Was this translation helpful? Give feedback.
-
Can you share your project set up via github? |
Beta Was this translation helpful? Give feedback.
-
|
Yes, here: https://github.com/perspilling/sba-example |
Beta Was this translation helpful? Give feedback.
-
|
Thanks, let me check. |
Beta Was this translation helpful? Give feedback.
-
|
Please see perspilling/sba-example#1 where I've added some changes to your sba-example/sba-server project and it should work. |
Beta Was this translation helpful? Give feedback.
-
|
Hi, I am looking to use Jolokia in my projects but ran into a problem. What I am most interested in is using Jolokia to do JFR recordings from JMC. I did the setup using a Java Agent approach and installed the plugin from @skarsaune Sorry for cross posting this but really wanted to know is there a way to get a JFR recording snapshot using Jolokia? I thought using jmc-cloud-extensions would do the trick but I get an exception when attempting to run it. Any thoughts or suggestions would be helpful. Also I did this following the recommendations from the docs: 
There was a recording that @skarsaune did which shows doing the JFR recording but not sure of all the right versions of things to make this possible. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for the report. There must be some problem if it doesn't work with 1.7.3. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for the quick reply, I am not sure if it's an issue with 1.7.3 as I did try the 2.0 version and it failed as well with the same exception. I am just not familiar enough to know if the error is on the jolokia side sending some data which the client isn't expecting or something with the plugin which was developed. It might not be a terrible idea to contact @skarsaune to see if the plugin should be part of the Jolokia project as I imagine there could be some compatibility issues there as new versions of Jolokia are published. JFR is awesome it's the ultimate profiler for Java and being able to adhoc take snapshots over http is very interesting. |
Beta Was this translation helpful? Give feedback.
-
|
@grgrzybek I am going to attempt to do this Jolokia 1.7.0 that seems to be the same version of the Jolokia that the plugin by @skarsaune uses. |
Beta Was this translation helpful? Give feedback.
-
|
Still seems to be the same issue with 1.7.0 :( |
Beta Was this translation helpful? Give feedback.
-
|
I can try to reproduce. When it comes to reverse mapping of types, support
may have to be gradually added in some cases, to support new use cases.
Will try to look at both permanent solutions in jolokia, and whether there
are any workarounds (manually overriding types from the client side)
…On Wed, Nov 8, 2023 at 7:56 PM Michael Bazos ***@***.***> wrote:
Still seems to be the same issue with 1.7.0 :(
Caused by: java.lang.ClassCastException: class org.json.simple.JSONArray cannot be cast to class [Ljavax.management.openmbean.CompositeData; (org.json.simple.JSONArray is in unnamed module of loader org.eclipse.osgi.internal.loader.EquinoxClassLoader @57b52b8a; [Ljavax.management.openmbean.CompositeData; is in module java.management of loader 'bootstrap')
at org.openjdk.jmc.rjmx.services.jfr.internal.FlightRecorderServiceV2.getAvailableRecordings(FlightRecorderServiceV2.java:297)
at org.openjdk.jmc.flightrecorder.controlpanel.ui.wizards.RecordingWizardModel.getDescriptors(RecordingWizardModel.java:320)
at org.openjdk.jmc.flightrecorder.controlpanel.ui.wizards.RecordingWizardModel.findUniqueName(RecordingWizardModel.java:287)
at org.openjdk.jmc.flightrecorder.controlpanel.ui.wizards.RecordingWizardModel.<init>(RecordingWizardModel.java:164)
at org.openjdk.jmc.flightrecorder.controlpanel.ui.actions.StartRecordingAction.doCreateWizard(StartRecordingAction.java:68)
at org.openjdk.jmc.ui.wizards.AbstractWizardUserAction.createWizard(AbstractWizardUserAction.java:65)
at org.openjdk.jmc.ui.wizards.WizardAdapterFactory.getAdapter(WizardAdapterFactory.java:45)
... 76 more
—
Reply to this email directly, view it on GitHub
<#555 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABQB2GQDUMEOLQ2OJPYT4PDYDPIVBAVCNFSM6AAAAAAZ5FADX6VHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM3TKMJUGMZDM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
Thanks @skarsaune I haven't directly worked on the jolokia library but if there is anything I can do to help please let me know. Appreciate the prompt reply on this. |
Beta Was this translation helpful? Give feedback.
-
|
@grgrzybek Just wanted to report back. So I did get JFR recording working with Jolokia and a custom branch of JMC that @skarsaune made. Please see here if you want to replicate it skarsaune/jmc-cloud-extensions#3 (comment) I didn't review the changes @skarsaune made to JMC to get it to work but the version of JMC is pretty out of date and as mentioned would be great if this could work with the latest off the shelf JMC instead of a custom build.
|
Beta Was this translation helpful? Give feedback.
-
|
Absolutely agree with all your points @mbazos . I aim to both update the plugins and first and foremost work on the PR to get it merged to jmc proper. With regards to the specific issue accessing fligth recordings over jolokia, I created a separate issue to track progress on troubleshooting of that: #613 |
Beta Was this translation helpful? Give feedback.
-
We will shortly have an update from @rhuss on the transfer of Jolokia project from his own repo to organisation. It is to keep Jolokia up to date by the community.
One of the major goals of the community-based Jolokia development would be releasing Jolokia 2.0. We internally discussed the ideas about Jolokia 2.0 in advance, and here are the summary of the discussion:
(@rhuss @grgrzybek @apupier Please correct me if I'm misunderstanding it.)
Here I opened a GitHub discussion so that this topic is shared with the community and can be discussed further with wider audience. Please share your thoughts. Thank you!
Beta Was this translation helpful? Give feedback.
All reactions