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

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 !

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.

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