Skip to content

Add Jolokia-based healthcheck for ActiveMQ containerFeature/activemq jolokia healthcheck #1645

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.

Sign up for GitHub

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

Jump to bottom
Open
pradeep85841 wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add Jolokia-based healthcheck for ActiveMQ containerFeature/activemq jolokia healthcheck #1645

pradeep85841 wants to merge 3 commits into from
+100 −1

Conversation

@pradeep85841
Copy link
Contributor

@pradeep85841 pradeep85841 commented Jan 31, 2026

This PR adds a Docker HEALTHCHECK for the ActiveMQ container using Jolokia.

Key points:

  • Uses authenticated Jolokia search to verify Broker MBean is available.
  • Ensures broker is fully initialized before container is marked healthy.
  • Compatible with any ActiveMQ 6.x brokerName.
  • Avoids fragile attribute checks like 'Started'.
  • Origin header included to satisfy Jolokia security.
  • Works with CI pipelines and deterministic Docker health status.

Reviewer notes:

  • ENTRYPOINT and CMD unchanged.
  • No changes to shutdown/logging behaviour.
  • Only adds healthcheck to improve container observability.

Pradeep Kunchala added 2 commits January 26, 2026 18:14
Add test for VM transport behavior on broker restart
77eb289
Add Jolokia-based healthcheck for ActiveMQ container
dbc3ece 
- HEALTHCHECK uses Jolokia search on Broker MBean
- Ensures broker is running and accessible
- Auth and Origin headers included for Jolokia security
- Docker CI-ready and deterministic
@jbonofre
Copy link
Member

jbonofre commented Feb 1, 2026

If the healthcheck in Dockerfile is interesting, I'm not sure Jolokia is the best option (or maybe just temporary).

@jbonofre jbonofre self-requested a review February 1, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jbonofre jbonofre Awaiting requested review from jbonofre

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pradeep85841 @jbonofre