RFR: 8284: Fix Jolokia discovery issues [v2]

duke duke at openjdk.org
Thu Nov 7 07:56:49 UTC 2024


On Mon, 4 Nov 2024 12:22:49 GMT, Martin Skarsaune <duke at openjdk.org> wrote:

>> This fixes two bugs with Jolokia discovery:
>> 1. serious : repeated discovery runs crash. Runing repeatedly in the background is the whole point
>> 2. more of an annoyance: the PR disables JMC agent by omitting JVM info in the descriptor (as presence is interpreted as a local JVM)
>> Both tested OK.
>
> Martin Skarsaune has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - Merge remote-tracking branch 'origin/master' into jolokia_discovery_issues
>  - reverted file that was committed by error
>  - Remove JVM info on discovered Jolokia VMs in order to disable JMC Agent functionality
>  - Fix bug where repeated runs of discovery crashes (kind of a showstopper)

@skarsaune 
Your change (at version 21515ce8d81255f908c1a3ceadc1cdb4f715316e) is now ready to be sponsored by a Committer.

-------------

PR Comment: https://git.openjdk.org/jmc/pull/597#issuecomment-2461547142


More information about the jmc-dev mailing list