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

Refine default Solr config settings post external Solr transition #2653

Merged
merged 1 commit into from Mar 14, 2022

Conversation

mark-cooper
Copy link
Member

Deprecated irrelevant :solr_home_directory.
Updated solr_url comment to be more appropriate.
Set no. of backups to 0 by default. Since external transition the
snapshot feature wouldn't work without cfg updates anyway and it's
now more difficult to assume there's a standard deployment config
for Solr backups that will be generally applicable.
Set :solr_index_directory to a reasonable looking "default" should
backups be enabled.
Abort the snapshot if the :solr_index_directory is not found.

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have read the CONTRIBUTING document.
  • I have authority to submit this code.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Deprecated irrelevant :solr_home_directory.
Updated solr_url comment to be more appropriate.
Set no. of backups to 0 by default. Since external transition the
snapshot feature wouldn't work without cfg updates anyway and it's
now more difficult to assume there's a standard deployment config
for Solr backups that will be generally applicable.
Set :solr_index_directory to a reasonable looking "default" should
backups be enabled.
Abort the snapshot if the :solr_index_directory is not found.
Remove embedded Solr checkindex scripts.
@mark-cooper mark-cooper marked this pull request as ready for review March 11, 2022 01:06
@quoideneuf quoideneuf merged commit 657d255 into archivesspace:master Mar 14, 2022
@cdibella cdibella added this to the 3.2.1 milestone Apr 27, 2022
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.

None yet

3 participants