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