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 Testcontainers Solace module when solace and testcontainers are selected #1262

Closed

Conversation

eddumelendez
Copy link
Contributor

Testcontainers offers a solace module that can be used when
both technologies are selected.

…elected

Testcontainers offers a solace module that can be used when
both technologies are selected.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 31, 2023
@snicoll
Copy link
Contributor

snicoll commented Jul 31, 2023

Thanks @eddumelendez but I'd like to add new features like this consistently with what we have already. I wonder if we could generate the @ServiceConnection for the test app automatically. I also wonder about generating the docker-compose bits when Docker Compose is selected. I can help with part (of all of this). Let me know what you'd prefer.

cc @Mrc0113

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Jul 31, 2023
@snicoll snicoll changed the title Add Testcontainers Solace module when solace and testcontainers are selected Add Testcontainers service connection support Jul 31, 2023
@snicoll
Copy link
Contributor

snicoll commented Jul 31, 2023

Ahem. I've just realized the auto-configuration on the other side needs to handle @ServiceConnection. So we're a bit stuck, aren't we? I'll create an issue to see if there's interest.

@snicoll snicoll changed the title Add Testcontainers service connection support Add Testcontainers Solace module when solace and testcontainers are selected Jul 31, 2023
@snicoll snicoll added type: enhancement and removed status: waiting-for-feedback We need additional information before we can continue status: waiting-for-triage An issue we've not yet triaged labels Jul 31, 2023
@snicoll snicoll self-assigned this Jul 31, 2023
@snicoll
Copy link
Contributor

snicoll commented Jul 31, 2023

I've created SolaceProducts/solace-spring-boot#106

snicoll pushed a commit that referenced this pull request Jul 31, 2023
This commit adds the solace module for Testcontainers if Testcontainers
and Solace are selected. Note that it does not add support for
connection details as Solace does not implement them at the moment.

See gh-1262
@snicoll snicoll closed this in e125f0d Jul 31, 2023
@eddumelendez
Copy link
Contributor Author

Looking forward to see the ServiceConnection support in Solace Spring Boot so it can be autogenerated in start.spring.io. Thanks for raising the issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants