Skip to content
This repository was archived by the owner on Jun 18, 2021. It is now read-only.

Move most IDR config to this repo#128

Merged
joshmoore merged 5 commits intoome:masterfrom
manics-archive:idr-refactor-groupvars
Oct 5, 2016
Merged

Move most IDR config to this repo#128
joshmoore merged 5 commits intoome:masterfrom
manics-archive:idr-refactor-groupvars

Conversation

@manics
Copy link
Copy Markdown
Member

@manics manics commented Sep 27, 2016

Moves most of the IDR configuration into this public repo.
Adds placeholders for variables which should remain secret, where possible a public default is given:

$ git grep idr_secret
idr-playbooks/files/IDR-OMERO-52-omero.j2:config set omero.web.public.password {{ idr_secret_omero_web_public_password | default("") }}
idr-playbooks/group_vars/omero-hosts.yml:omero_system_uid: "{{ idr_secret_omero_system_uid | default(546) }}"
idr-playbooks/group_vars/omero-hosts.yml:omero_logmonitor_slack_token: "{{ idr_secret_omero_logmonitor_slack_token | default(None) }}"

Most of these playbooks should therefore run without any private configuration.
Note: work on changing omero_system_uid in the IDR is ongoing.

Testing

Ansible group_vars can be in two locations, the playbook directory and the inventory directory, with the former taking precedence. This means when you run a playbook in idr-playbooks idr-playbooks/group_vars is automatically included, and if you also specify an inventory (ansible-playbook -i INVENTORY/PATH) INVENTORY/PATH/group_vars will be included.

I've only tested this on top of #124

@manics
Copy link
Copy Markdown
Member Author

manics commented Sep 27, 2016

What would be ideal is if we could automatically create a public user if it doesn't already exist, but that probably requires a custom omero ansible module.

@joshmoore
Copy link
Copy Markdown
Member

Merging for the upcoming testing via demo3 deployment.

@joshmoore joshmoore merged commit 31aaa44 into ome:master Oct 5, 2016
@manics manics deleted the idr-refactor-groupvars branch October 5, 2016 12:04
manics pushed a commit to manics-archive/ome-infrastructure that referenced this pull request Nov 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants