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

iRODS: v2 (remote pgsql, tiering, bugfixes, variables and new certificate) #580

Merged
merged 30 commits into from
Jul 19, 2022

Conversation

scimerman
Copy link
Contributor

  • added tiering playbook that configure 3 levels data migration
  • split configuration for icat.yml into _pre configuration and main configuration
  • added pgsql_remote.yml playbook, to configure icat to use remote sql server
    • added bugfixes (templates/database_connect.py - used only on irods-server version 4.2.11) to correctly use the ssl when using remote pgsql server
  • moved the pgsql.yml into postgres _local, as to differentiate from the pgsql_remote.yml
  • updated readme file
  • shortened and consolidated variable names
    • secret variables and now called icat_ and irods_ to avoid confusion
    • non-secret irods_ variable are now shortened to ir_ to save space and avoid further confusion
  • custom variable per machine:
    • custom variables are now using variables from static_inventory
      • if those are missing, they use group variables
      • if also those are also missing, they use default playbook variables
    • secret variables per machine, are now linked to ones defined secrets.yml, f.e. in static_inventories/nibbler_cluster.yml
      irods-catalogus:
           ir_db_pwd: '{{ icat_test_db_pwd }}'   # for the test machine, vs
      irods-catalogus:
           ir_db_pwd: '{{ icat_db_pwd }}'        # for the production machine
      
  • uploaded new key (encrypted) and certificate (unencrypted) for the umcg-icat01.gcc.rug.nl
    (followed the convention of intermediate certificates order - sorted them correctly)
  • updated davrods configuration: port definition in docker-compose and ServerName in vhost.conf ended in error due to new docker version

roles/irods/README.md Outdated Show resolved Hide resolved
roles/irods/README.md Outdated Show resolved Hide resolved
roles/irods/README.md Outdated Show resolved Hide resolved
roles/irods/README.md Outdated Show resolved Hide resolved
group_vars/irods.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@pneerincx pneerincx left a comment

Choose a reason for hiding this comment

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

Wow, large PR :) Looks mostly good. A few minor issues/suggestions. See inline comments.

pneerincx and others added 5 commits June 9, 2022 17:09
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
scimerman and others added 7 commits June 13, 2022 12:11
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Co-authored-by: Pieter Neerincx <pieter.neerincx@gmail.com>
Copy link
Contributor

@pneerincx pneerincx left a comment

Choose a reason for hiding this comment

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

See 2 more inline comments.

@scimerman scimerman requested a review from pneerincx July 18, 2022 09:36
Copy link
Contributor

@pneerincx pneerincx left a comment

Choose a reason for hiding this comment

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

Small YAML indentation issue in playbook; see inline comment.

@scimerman scimerman requested a review from pneerincx July 19, 2022 10:57
@pneerincx pneerincx merged commit 7cbba0f into rug-cit-hpc:develop Jul 19, 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.

2 participants