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

Add solr image with cloud mode for tugboat. #82

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

apotek
Copy link

@apotek apotek commented Sep 27, 2024

Description

This sets up a docker container that enables solrcloud mode for use in Tugboat.

To use this be sure to either:

Run solr zk cp file:/var/solr/data/security.json zk:/security.json -z localhost:9983 in the init phase of the service, or
Run /opt/solr/docker/scripts/zkauth.sh

Motivation / Context

The default TugboatQA docker image does not support solrcloud mode, and, once the container is running, it's very hard to configure it for solrcloud mode while still keeping intact the service monitoring and service invocations that are the preferred way for Tugboat to work.

Testing Instructions / How This Has Been Tested

cd services/solr && docker build -t solrcloud .
docker run solrcloud
docker exec -it <imageid> bash
/opt/solr/docker/scripts/zkauth.sh

Screenshots

Documentation

@apotek apotek self-assigned this Sep 27, 2024
@apotek apotek requested a review from mattdaniel September 27, 2024 20:12
@agarzola agarzola marked this pull request as draft September 30, 2024 14:08
@agarzola
Copy link
Member

I made this branch a draft, since the work from which this effort originated is being put on hold.

FROM tugboatqa/solr:${SOLR_VERSION}
ENV COMPOSER_DISCARD_CHANGES=true
ENV COMPOSER_NO_INTERACTION=true
ENV COMPOSER_ALLOW_SUPERUSER=true
Copy link
Author

Choose a reason for hiding this comment

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

We don't need these COMPOSER env vars since there is no code checkout on this service.

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