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

FITS cleanup #1638

Open
alxp opened this issue Oct 9, 2020 · 2 comments
Open

FITS cleanup #1638

alxp opened this issue Oct 9, 2020 · 2 comments
Labels
Subject: Ansible Playbook Related to the maintenance and upkeep associated with an Ansible Playbook for Islandora. Always also Subject:Deployment related to deployment of Islandora. Usually accompanied by the deployment flavour (Ansible/docker)

Comments

@alxp
Copy link
Contributor

alxp commented Oct 9, 2020

The FITS role is causing Ansible install to fail, and I've uncovered several problems with it:

  1. The Ansible role, https://github.com/Islandora-Devops/ansible-role-fits , is failing on the "Download Islandora FITS" emits a warning that composer is being run as root, then fails with an out-of-memory error. The way Islandora roles and Ansible is architected, it looks like it's improper to attempt to install Drupal modules in these roles, so this step should belong in the web server config tasks.

  2. The role attempts to download a Drupal 8 module hosted on Robertson Library's GitHub, that shares a name with the D7 module in Islandora:

    D8: https://github.com/roblib/islandora_fits
    D7: https://github.com/islandora/islandora_fits

    Would it be OK to make a 8.x-1.x branch on islandora/islandora_fits to contain this D8 module? @dannylamb / @Islandora/7-x-1-x-committers

cc. @ajstanley @rosiel

@rosiel
Copy link
Member

rosiel commented Sep 10, 2021

Still relevant. Islandora is still getting code from a roblib repository:
https://github.com/Islandora-Devops/islandora-playbook/blob/dev/inventory/vagrant/group_vars/webserver/drupal.yml#L28
(islandora-rdm/islandora_fits is defined in Packagist: https://packagist.org/packages/islandora-rdm/islandora_fits)

We should probably "give" this code to Islandora. How do we do this?

@manez
Copy link
Member

manez commented Sep 13, 2021

For a full repo like this, probably with the LSAP: https://islandora.ca/lsap

@kstapelfeldt kstapelfeldt added Subject:Deployment related to deployment of Islandora. Usually accompanied by the deployment flavour (Ansible/docker) Subject: Ansible Playbook Related to the maintenance and upkeep associated with an Ansible Playbook for Islandora. Always also and removed Ansible labels Sep 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Subject: Ansible Playbook Related to the maintenance and upkeep associated with an Ansible Playbook for Islandora. Always also Subject:Deployment related to deployment of Islandora. Usually accompanied by the deployment flavour (Ansible/docker)
Projects
Development

No branches or pull requests

4 participants