Skip to content

Expose Solr port for dev#928

Merged
jp-tosca merged 1 commit into
developfrom
solr-expose-port
Apr 13, 2026
Merged

Expose Solr port for dev#928
jp-tosca merged 1 commit into
developfrom
solr-expose-port

Conversation

@pdurbin
Copy link
Copy Markdown
Member

@pdurbin pdurbin commented Mar 4, 2026

What this PR does / why we need it:

This useful when trying to reload Solr after adding a metadata block and running update-fields.sh.

See https://guides.dataverse.org/en/6.9/container/running/demo.html#additional-metadata-blocks

My workaround is to exec into the Solr container first:

pdurbin@beamish dataverse % docker exec -it dev_solr bash
solr@solr:/opt/solr-9.8.0$ curl "http://localhost:8983/solr/admin/cores?action=RELOAD&core=collection1"
{
  "responseHeader":{
    "status":0,
    "QTime":479
  }
}solr@solr:/opt/solr-9.8.0$ 

This useful when trying to reload Solr after adding a metadata block and running update-fields.sh.
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 97.362% (+0.02%) from 97.347%
when pulling b6a1efa on solr-expose-port
into 94aecb3 on develop.

Copy link
Copy Markdown
Contributor

@jp-tosca jp-tosca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jp-tosca jp-tosca merged commit a71dff2 into develop Apr 13, 2026
16 of 19 checks passed
@pdurbin
Copy link
Copy Markdown
Member Author

pdurbin commented Apr 30, 2026

Whoops, sorry this PR broke things! 😅

I just created a new PR with a fix (while still preserving the original intent of exposing the Solr port):

jp-tosca pushed a commit that referenced this pull request Apr 30, 2026
…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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants