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

Update Actions to use dockerized solr and remove solr_wrapper from rake CI task #1061

Merged
merged 2 commits into from Aug 27, 2021

Conversation

eliotjordan
Copy link
Member

@eliotjordan eliotjordan commented Aug 26, 2021

Replaces solr_wrapper with custom solr docker image in GitHub Actions.
Closes #1059

Docker image defined here:
https://github.com/geoblacklight/geoblacklight-docker/tree/master/solr

On DockerHub:
https://hub.docker.com/repository/docker/geoblacklight/solr

@eliotjordan eliotjordan marked this pull request as draft August 26, 2021 20:29
@eliotjordan eliotjordan force-pushed the 1059-solr-docker branch 2 times, most recently from f034f93 to 3b65a68 Compare August 26, 2021 20:59
@eliotjordan eliotjordan force-pushed the 1059-solr-docker branch 2 times, most recently from 36ba999 to fd15087 Compare August 26, 2021 21:39
@eliotjordan eliotjordan marked this pull request as ready for review August 26, 2021 21:40
Copy link
Contributor

@tampakis tampakis left a comment

Choose a reason for hiding this comment

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

CI is passing! Noticing that the developer documentation is going to need to be updated to add Docker/Lando as a dependency eventually, even though with this PR that is only true for the rake ci task

@tampakis tampakis merged commit 4c3965d into main Aug 27, 2021
@tampakis tampakis deleted the 1059-solr-docker branch August 27, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use Dockerized Solr in CI rather than solr_wrapper
3 participants