Skip to content

Conversation

@jaydeluca
Copy link
Member

@jaydeluca jaydeluca commented Nov 23, 2025

Related to #15429

The BackgroundPreinitializingApplicationListener ignore is a common change amongst many of these modules, so that change is also present in #15433

Related to #14906

@jaydeluca jaydeluca requested a review from a team as a code owner November 23, 2025 14:50
new GenericContainer<>("quay.io/artemiscloud/activemq-artemis-broker:artemis.2.27.0")
.withEnv("AMQ_USER", "test")
.withEnv("AMQ_PASSWORD", "test")
new GenericContainer<>("apache/activemq-artemis:2.44.0")
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i had issues running the other container, but this one worked well. If we'd prefer to keep it as is, I can revert

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is much better to use the official image 👍

new GenericContainer<>("quay.io/artemiscloud/activemq-artemis-broker:artemis.2.27.0")
.withEnv("AMQ_USER", "test")
.withEnv("AMQ_PASSWORD", "test")
new GenericContainer<>("apache/activemq-artemis:2.44.0")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@trask trask merged commit 7f1160c into open-telemetry:main Nov 24, 2025
88 checks passed
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.

3 participants