Skip to content

fix for Container dev_solr invalid start port '8983:8983': invalid syntax#978

Merged
jp-tosca merged 1 commit intodevelopfrom
928-expose-solr-port
Apr 30, 2026
Merged

fix for Container dev_solr invalid start port '8983:8983': invalid syntax#978
jp-tosca merged 1 commit intodevelopfrom
928-expose-solr-port

Conversation

@pdurbin
Copy link
Copy Markdown
Member

@pdurbin pdurbin commented Apr 30, 2026

This PR had a defect in it (sorry):

Now commands like this should work fine, which was the original intent:

curl http://localhost:8983/solr/collection1/schema/fields

See doc/sphinx-guides/source/container/running/demo.rst under "Additional Metadata Blocks" for related commands. Here: https://guides.dataverse.org/en/6.10.1/container/running/demo.html#additional-metadata-blocks

This change should make the "update Solr" steps in the following PR (where we add the "review" block) actually work:

…ntax

This PR had a defect in it:

- #928

Now commands like this should work fine, which was the original intent:

curl http://localhost:8983/solr/collection1/schema/fields

See doc/sphinx-guides/source/container/running/demo.rst under
"Additional Metadata Blocks" for related commands.
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 97.515% (-0.05%) from 97.565% — 928-expose-solr-port into develop

@jp-tosca jp-tosca self-assigned this Apr 30, 2026
@jp-tosca jp-tosca self-requested a review April 30, 2026 16:41
@github-project-automation github-project-automation Bot moved this to Ready for QA ⏩ in IQSS Dataverse Project Apr 30, 2026
@jp-tosca jp-tosca merged commit 7a3863b into develop Apr 30, 2026
14 of 19 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for QA ⏩ to Merged 🚀 in IQSS Dataverse Project Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged 🚀

Development

Successfully merging this pull request may close these issues.

3 participants