Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix SolrContainer start parameters for version >= 9.7.0 #9926

Merged
merged 2 commits into from
Feb 13, 2025

Conversation

mkr
Copy link
Contributor

@mkr mkr commented Jan 31, 2025

Fixes #9601

@mkr
Copy link
Contributor Author

mkr commented Feb 12, 2025

@eddumelendez Does this fix makes sense from your perspective?

@eddumelendez
Copy link
Member

Hi @mkr, sorry for the delay. Can you please add a test? Checking the version is 👍🏽

@eddumelendez eddumelendez added this to the next milestone Feb 12, 2025
@mkr
Copy link
Contributor Author

mkr commented Feb 13, 2025

Hi @eddumelendez, thank you. I parameterized the SolrContainerTest and let it run for latest 8.x and 9.x versions.

@eddumelendez eddumelendez changed the title Adjust Solr start parameters for Solr >= 9.7.0 Fix SolrContainer start parameters for version >= 9.7.0 Feb 13, 2025
@eddumelendez eddumelendez merged commit df40cd8 into testcontainers:main Feb 13, 2025
108 checks passed
@eddumelendez
Copy link
Member

Thanks for your contribution, @mkr !

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

Successfully merging this pull request may close these issues.

[Bug]: Default start parameters not working for Solr 9.7.0
2 participants