update search_api_solr to 3.8

[elizoller]

What does this Pull Request do?

Updates search_api_solr to version 3.8

What's new?

• Updates search_api_solr to version 3.8
• Per the install.md - adds a patch for Drupal Core
• Per the install.md - uses drush command to generate the default solr config files
• Reorders the playbook.yml so that solr.yml runs after webserver.yml because the solr config setup requires the output of the drush command

How should this be tested?

• check out this PR, vagrant up
• check to see that you got a running solr instance with a core called ISLANDORA
• check to see that you have a default solr server in the drupal search api config (http://localhost:8000/admin/config/search/search-api/)
• check that http://localhost:8000/admin/config/search/search-api/server/default_solr_server says the schema is set to drupal-8.3.8-solr-7.x
• check that you can create an object and that it is indexed in solr and searchable

Additional Notes:

I was concerned about conflating the solr host and the webserver host so that is why I implemented the "Unarchive solr config" as a request to localhost instead of directly from a file path like /var/www/...

Interested parties

@Islandora-Devops/committers

[seth-shaw-unlv]

Results in a warning: "Solarium 5.0.2 is too old and has known issues. "

To alleviate this we can update the composer task to be:

- name: Add a patch for symfony-event-dispatcher per the INSTALL.md docs on search_api_solr:3.8
  composer:
    command: require
    arguments: symfony/event-dispatcher:'4.3.4 as 3.4.99' solarium/solarium:^5.1.4 drupal/search_api_solr
    working_dir: "{{ drupal_composer_install_dir }}"

I'm building a new instance with this revised task right now.

[elizoller]

awesome, thanks @seth-shaw-unlv

[seth-shaw-unlv]

So, this works!

HOWEVER, this screen shot is taken before we import the islandora_search configs. Doing the feature import breaks the Solr Server config. With the updated search_api_solr module they appear to assume solr will be accessible @ '/solr/' so the linked config line turns http://127.0.0.1:8983/solr/#/ISLANDORA into http://127.0.0.1:8983/solr/solr/#/ISLANDORA. I don't know why they did that, but there it is.

So, @elizoller , if you can add that solarium bit to the composer command in this PR we can merge this one. I'll create a new PR on islandora_defaults to purge the path config.

[seth-shaw-unlv]

Formalizing the requested change....

[seth-shaw-unlv]

Spinning up another instance to double check before merge.

[elizoller]

This build completed successfully for me and no more solarium warning

[seth-shaw-unlv]

I forgot to pull in the Grok playbook fix for CentOS so my build died. 🤦‍♂️ It is building again and should be done in a little while.