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.

Sign up for GitHub

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

Jump to bottom

Deploy ms image region #430

Merged
merged 7 commits into from
Sep 6, 2024
Merged

Deploy ms image region #430

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
a9884b0
deploy ms-image-region
khaledk2 Aug 14, 2024
15052eb
use pre_tasks
khaledk2 Aug 14, 2024
45693d7
move ms deployment to omero read only
khaledk2 Aug 25, 2024
eb89e47
fix syntex issue
khaledk2 Aug 26, 2024
832d695
use branch for ome.analysis_tools role
khaledk2 Aug 26, 2024
bb0a48f
add restart ms service
khaledk2 Sep 4, 2024
59dbd88
use ms_image_region and analysis_tools roles from Ansible Galaxy
khaledk2 Sep 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions ansible/idr-ms-image-region.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
- hosts: "{{ idr_environment | default('idr') }}-omeroreadonly-hosts"
sbesson marked this conversation as resolved.
Show resolved Hide resolved
become: true

pre_tasks:
- name: get database host
set_fact:
database_server_url: >-
{{
hostvars[groups[idr_environment | default('idr') + '-database-hosts'][0]]
['ansible_' + (idr_net_iface | default('eth0'))]['ipv4']['address']
}}
sbesson marked this conversation as resolved.
Show resolved Hide resolved
- name: get database password
set_fact: >-
database_user_password: "{{ idr_secret_postgresql_password_ro | default('omero') }}"
Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason to use set_fact rather than using "{{ idr_secret_postgresql_password_ro | default('omero') }}" directly in the variable?

roles:
- role: ome.omero_ms_image_region
omero_ms_image_region_update_nginx: true
sbesson marked this conversation as resolved.
Show resolved Hide resolved
omero_ms_image_region_db_url: {{ datab ase_server_url }}
omero_ms_image_region_db_name: idr
omero_ms_image_region_db_username: omeroreadonly
omero_ms_image_region_db_pass: {{ database_user_password }}
omero_data_dir: /data/OMERO
omero_ms_image_region_session_id: sessionid_{{ idr_environment | default('idr') }}
4 changes: 4 additions & 0 deletions ansible/requirements.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,10 @@
- src: ome.versioncontrol_utils
version: 1.1.0

# this should be updated after merging the PRs and publich the role
- name: ome.omero_ms_image_region
src: https://github.com/khaledk2/ansible-role-omero-ms-image-region/
version: new_ms_release
Copy link
Member

Choose a reason for hiding this comment

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

For the prod122 update we can certainly use the in-progress version but for future deployments this role will need to be released in order to merge this step


######################################################################
# External IDR roles
Expand Down
Loading