-
Notifications
You must be signed in to change notification settings - Fork 60
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 core reload/optimize #215
Conversation
@DonRichards right now I am comfortable with running Off the top of my head I am thinking I can manually add some images, then add some minimal metadata. Run some searches. Then run |
@ysuarez Adding anything at all (Islandora content) should be enough to indicate the core is not current. To see the Solr dashboard you'll need to "expose" it. Set it to true in the env file. Here's how to pull in changes to the |
@ysuarez Any luck testing this? |
I am currently testing this and sharing results with Don as I write this. I will provide updates when I have results. |
Here is how the testing went. I first ran I then switched to running I first visited http://islandora.traefik.me:8983/solr/#/~cores/ISLANDORA and I saw that the "current value" was set to red forbidden icon, i.e. not current. (regretfully I forgot to take a screenshot) Then I created 2 repository nodes, but there was a Drupal writing permission error, so I could not ingest images. I then added some additional metadata to the 2 nodes from earlier. I then visited http://islandora.traefik.me:8983/solr/#/~cores/ISLANDORA and the 'current' value had a green check mark(!). I may be misunderstanding the PR, but I expected a red icon at this point. I then finally ran As a test (without making any node changes) I ran I hope this helps. |
@ysuarez What was the output of running |
Don, below is the only output I got today after I ran
For the record, my test instance was built by running |
@ysuarez That brings up a good point. This PR should check if Solr is exposed. Otherwise, it won't work. |
@DonRichards I just ran http://islandora.traefik.me:8983/solr/#/ I will now try with BTW the "tag" value is: |
I got further when using Here is the error:
Here is line 160 in context: Lines 156 to 163 in b3349e9
I wonder if this is a macOS issue with the use of |
Never mind, I thought this was another PR that needed someone to merge. |
The
make solr-cores
command "Creates solr-cores according to the environment variables.". This new commandsolr-reload-cores
reloads the solr core with current live configurations and then optimizes it. This allows people to reload and optimize from the terminal quickly.To test
Before running
![Before](https://user-images.githubusercontent.com/2738244/151240490-0f11554b-d94a-447a-b80a-a51d683a8e31.png)
Then run
make solr-reload-cores
After running command
![After running](https://user-images.githubusercontent.com/2738244/151240550-50b52b52-80fb-434f-bcb9-be6981d17f64.png)