Skip to content

ETT-1793 solr init script - #250

Merged
aelkiss merged 1 commit into
mainfrom
ETT-1793-solr-init-script
Sep 8, 2026
Merged

ETT-1793 solr init script#250
aelkiss merged 1 commit into
mainfrom
ETT-1793-solr-init-script

Conversation

@aelkiss

@aelkiss aelkiss commented Sep 3, 2026

Copy link
Copy Markdown
Member

Relevant commit: 49bdb0b

See hathitrust/lss_solr_configs#24

Needs to wait for #248 to be merged.

Updates health check.

Recommended testing procedure:

First make sure everything solr related is cleaned out:

docker compose --profile solr9 down
docker system prune
docker volume rm babel_zoo1_data babel_solr1_data
docker compose --profile solr9 up

You should (eventually) see solr come up & create the collection, and docker ps should show the container as healthy.

Then:

  • stop it with ctrl-c
  • restart with docker compose --profile solr9 up

You should (eventually) see a message from the solr container that the collection already exists, and docker ps should show the container as healthy.

@liseli liseli left a comment

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've tested locally and it works

@aelkiss

aelkiss commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

I'll rebase and then go ahead and merge this.

* update health check to use built-in one
* use init script from lss_solr_configs
@aelkiss
aelkiss force-pushed the ETT-1793-solr-init-script branch from 49bdb0b to 0ed9f78 Compare September 4, 2026 15:12
@aelkiss

aelkiss commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

Rebased & force pushed; once tests pass I will merge.

@aelkiss
aelkiss merged commit 02aae8a into main Sep 8, 2026
3 checks passed
@aelkiss
aelkiss deleted the ETT-1793-solr-init-script branch September 8, 2026 14:02
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.

2 participants