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

Allow for indexless installs #171

Merged
merged 3 commits into from
Feb 14, 2018
Merged

Conversation

scollazo
Copy link
Contributor

@scollazo scollazo commented Feb 6, 2018

This adds the ARCHIVEMATICA__SEARCH_ENABLED env var

@scollazo
Copy link
Contributor Author

scollazo commented Feb 6, 2018

This is needed by artefactual/deploy-pub#52

Copy link
Member

@sevein sevein left a comment

Choose a reason for hiding this comment

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

Can we add a note to the README? LGTM.

Copy link
Contributor

@jrwdunham jrwdunham left a comment

Choose a reason for hiding this comment

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

Yes, with mention in the README this looks good to me.

@scollazo
Copy link
Contributor Author

scollazo commented Feb 7, 2018

Updated README to tell about the new archivematica_src_search_enabled var, and to point to the README.md files in archivematica.git

README.md Outdated
@@ -56,6 +56,10 @@ The default Archivematica 1.7 logging sends the events to the standard streams,

The log file sizes and the directories to store the logs are configurable for each service. The default values can be found in [`defaults/main.yml`](defaults/main.yml).

Disable ElasticSearch use
Copy link
Member

Choose a reason for hiding this comment

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

s/ElasticSearch/Elasticsearch

README.md Outdated
Disable ElasticSearch use
-------------------------

The default Archivematica install relies on ElasticSearch for different features (Archival storage, Backlog and Appraisal tabs). If you need to disable them, the boolean environment variable `archivematica_src_search_enabled` has to be set to `False`
Copy link
Member

Choose a reason for hiding this comment

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

Nitpick: archivematica_src_search_enabled isn't an environment variable. Role variable?

@jrwdunham
Copy link
Contributor

@sevein What do you think? Can this be merged now?

Copy link
Member

@sevein sevein left a comment

Choose a reason for hiding this comment

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

Small cosmetic suggestion but I think it could be fixed later if we're in a rush. 👍

README.md Outdated
Disable Elasticsearch use
-------------------------

The default Archivematica install relies on ElasticSearch for different features (Archival storage, Backlog and Appraisal tabs). If you need to disable them, the role variable `archivematica_src_search_enabled` has to be set to `False`
Copy link
Member

Choose a reason for hiding this comment

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

s/ElasticSearch/Elasticsearch

@jrwdunham jrwdunham merged commit 9665d63 into qa/1.7.x Feb 14, 2018
@jrwdunham jrwdunham deleted the dev/allow-indexless-install branch February 14, 2018 00:12
@jrwdunham
Copy link
Contributor

I fixed the typo. I also used this role in a deploy-pub Vagrant deploy and confirmed it can be used to correctly deploy AM headlessly. See artefactual-labs/archivematica-acceptance-tests#67 (comment).

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.

3 participants