Skip to content

Add JMX connection status events #524

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

Merged
merged 6 commits into from
Jun 21, 2021
Merged

Add JMX connection status events #524

merged 6 commits into from
Jun 21, 2021

Conversation

hareetd
Copy link
Contributor

@hareetd hareetd commented Jun 18, 2021

Related #461

Add custom JFR events for monitoring JMX connection status (opened, closed and failed).

Example output:

Screenshot from 2021-06-18 13-34-14

@hareetd hareetd requested review from andrewazores and removed request for andrewazores June 18, 2021 18:08
@hareetd hareetd marked this pull request as ready for review June 18, 2021 18:25
@hareetd hareetd requested a review from andrewazores June 18, 2021 18:25
…k and future-proof JMX service url field names
@andrewazores andrewazores merged commit a314ee0 into cryostatio:main Jun 21, 2021
@hareetd hareetd deleted the add-custom-jmx-events branch June 21, 2021 18:25
jan-law pushed a commit to jan-law/cryostat that referenced this pull request Jun 21, 2021
* Add custom JFR events for JMX connection status

* Fix spotless formatting errors

* Change JMXConnectionOpened event to fit into a try-catch-finally block and future-proof JMX service url field names

* Modify JMXConnectionClosed event into a try-catch-finally block and update the events template to reflect past changes

* Change connection closed event to throw RuntimeException instead

* Remove unnecessary declaration of RuntimeException
jan-law pushed a commit to jan-law/cryostat that referenced this pull request Jun 21, 2021
andrewazores pushed a commit to andrewazores/cryostat that referenced this pull request Jun 24, 2021
* Add custom JFR events for JMX connection status

* Fix spotless formatting errors

* Change JMXConnectionOpened event to fit into a try-catch-finally block and future-proof JMX service url field names

* Modify JMXConnectionClosed event into a try-catch-finally block and update the events template to reflect past changes

* Change connection closed event to throw RuntimeException instead

* Remove unnecessary declaration of RuntimeException
aali309 pushed a commit to aali309/cryostat-legacy that referenced this pull request Jul 22, 2024
…rc/main/docker (cryostatio#524)

build(deps): bump ubi8/openjdk-17-runtime in /src/main/docker

Bumps ubi8/openjdk-17-runtime from 1.20-2 to 1.20-3.

---
updated-dependencies:
- dependency-name: ubi8/openjdk-17-runtime
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

2 participants