Skip to content
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

Add Artemis Service Connection for Docker Compose and Testcontainers #39311

Conversation

eddumelendez
Copy link
Contributor

@eddumelendez eddumelendez commented Jan 25, 2024

  • Update Testcontainers version to 1.19.4
  • Add service connection for Testcontainers Artemis (available in version 1.19.4)
  • Add service connection for Docker Compose Artemis

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 25, 2024
@eddumelendez
Copy link
Contributor Author

It fails because of the change in commons-compress. See diff apache/commons-compress@rel/commons-compress-1.24.0...rel/commons-compress-1.25.0#diff-19b93108b5b4d5b1f4826b7dce61b416beb5a02d928ae8868412d971601ab201R568. Current common-compress version in spring-boot-parent is 1.23.0. Testcontainers 1.19.3 uses commons-compress 1.23.0 and Testcontainers 1.19.4 uses commons-compress version 1.25.0. I noticed the change when submitting this PR.

darkmastermindz added a commit to darkmastermindz/spring-boot that referenced this pull request Jan 26, 2024
darkmastermindz added a commit to darkmastermindz/spring-boot that referenced this pull request Jan 26, 2024
mhalbritter pushed a commit to mhalbritter/spring-boot that referenced this pull request Jan 30, 2024
mhalbritter added a commit to mhalbritter/spring-boot that referenced this pull request Jan 30, 2024
@mhalbritter
Copy link
Contributor

I have rebased the PR on top of main and have some polishing changes in this branch: https://github.com/mhalbritter/spring-boot/tree/pr/39311

This fixes the docker compose integration, too.

I'd like to do the Testcontainers version update in a separate issue, however.

@mhalbritter mhalbritter added type: enhancement A general enhancement for: merge-with-amendments Needs some changes when we merge and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 30, 2024
@mhalbritter mhalbritter added this to the 3.3.x milestone Jan 30, 2024
@mhalbritter
Copy link
Contributor

Thanks a lot, Eddú!

@mhalbritter mhalbritter modified the milestones: 3.3.x, 3.3.0-M2 Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for: merge-with-amendments Needs some changes when we merge type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants