Skip to content

Integrating Solr10 in babel#220

Open
liseli wants to merge 2 commits into
mainfrom
ETT-1463_Migrate-ptsearch-solrSolr8-Solr10
Open

Integrating Solr10 in babel#220
liseli wants to merge 2 commits into
mainfrom
ETT-1463_Migrate-ptsearch-solrSolr8-Solr10

Conversation

@liseli

@liseli liseli commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

This PR is related to the pt-search Solr migration from Solr 8 to Solr 10. Find ptsearch-solr PR here. This changes are part of the Jira ticket ETT-1463

It involves some tidy updates I need to make to use Solr 10 in the Docker environment, along with resolving a dependency issue in stage-item. I used the stage-item application to stage documents in Solr used for testing pt search.

@liseli liseli requested a review from aelkiss June 4, 2026 20:42
Comment thread slip/Dockerfile Outdated
RUN cpanm --notest \
File::Pairtree

RUN cpanm --notest \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't know a lot about this part yet, but just a minor nitpick: I see a double space before \ (one space is enough). Feel free to ignore though as it's not important.

@liseli liseli Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks so much, @eumalin! I've combined both dependencies into a single command.

@aelkiss

aelkiss commented Jun 5, 2026

Copy link
Copy Markdown
Member

I'm working on getting this working locally for me.

@aelkiss

aelkiss commented Jun 5, 2026

Copy link
Copy Markdown
Member

I see a couple things locally in the apache environment:

apache-1  | [Fri Jun 05 17:14:02.386782 2026] [cgi:error] [pid 10:tid 10] [client 172.18.0.1:39880] AH01215: stderr from /htapps/babel/pt/cgi/search.choke: Config /htapps/babel/slip-lib/Config/run-20.conf references ENV[PTSEARCH_SOLR_BASIC_AUTH] which is unset at /htapps/babel/mdp-lib/MdpConfig.pm line 271., referer: http://localhost:8080/cgi/pt?id=test.pd_open&seq=1
apache-1  | 172.18.0.1 - - [05/Jun/2026:17:14:02 +0000] "GET /cgi/pt/search?id=test.pd_open;start=1;sz=25;q1=committee;sort=seq;format=json;page=search;seq=1;orient=0 HTTP/1.1" 302 296 "http://localhost:8080/cgi/pt?id=test.pd_open&seq=1" "Mozilla/5.0 (X11; Linux x86_64; rv:140.0) Gecko/20100101 Firefox/140.0"

I think the complaint about PTSEARCH_SOLR_BASIC_AUTH is a red herring -- from the solr log it looks like requests are getting through, and authentication isn't set up -- but I'm not sure why the /cgi/pt/search request itself is returning a 302 redirect. It looks like it's redirecting back to itself and then ultimately failing.

I'll try with an item with stage-item - maybe the test item won't work with ptsearch for some reason - but using stage_item should get everything set in the catalog, full-text search solr, etc.

@aelkiss

aelkiss commented Jun 5, 2026

Copy link
Copy Markdown
Member

After staging pur1.32754063106516 everything works fine for me including result highlighting. I suspect the issue with test.pd_open is that indexing is failing, probably because that item isn't in the catalog solr.

@liseli

liseli commented Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

Thank you for your tests, Aaron. You describe the same behavior I saw, and it is not related to Solr; it is more about missing data in the Catalog. The test.pd_open failed for me as well.

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