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 to 8.3, solr 7, make it stand alone #123

Closed
wants to merge 1 commit into from

Conversation

Natkeeran
Copy link
Contributor

@Natkeeran Natkeeran commented Jul 4, 2019

GitHub Issue: (link)
Islandora/documentation#1163
Islandora/documentation#1197

What does this Pull Request do?

  • Installs and configures search_api_solr 8.3
  • Supports installing solr on its own server.

How should this be tested?

  • Get the PR
  • vagrant up
  • Create content
  • Verify that it gets indexed and returned

In addition, testing the following is helpful.

  • Get the PR
  • Modify or create the following playbook:
---

- include: bootstrap.yml
  tags:
    - bootstrap
- include: solr.yml
  • deploy and check that solr has been correctly installed

Additional Notes:

Although the above works, " You are using outdated Solr configuration set. Please follow the instructions described in the INSTALL.md file for setting up Solr. " is being reported here: http://localhost:8000/admin/config/search/search-api/server/default_solr_server

Not sure what may be cause of error.

Interested parties

@Islandora-CLAW/committers

@dannylamb would be helpful if you can test and get this one in.

@dannylamb
Copy link
Member

@Natkeeran Testing now

@dannylamb
Copy link
Member

@Natkeeran The box provisions correctly, but after ingesting content, it seems to get indexed, but I can't search by title. Default results of searching an empty text field return the result, but not if i actually using the title text.

@dannylamb
Copy link
Member

@Natkeeran title is included in the solr indexing config 🤷‍♂️ Weird.

@Natkeeran
Copy link
Contributor Author

Part of this PR has been merged; #126. Updating to 8.3 needs further review and would need a different PR. Thus closing this one.

@Natkeeran Natkeeran closed this Aug 27, 2019
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