search

Islandora / documentation

Contains islandora's documentation and main issue queue.
MIT License
104 stars 71 forks source link

FITS cleanup #1638

Open alxp opened 4 years ago

alxp commented 4 years ago

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 commented 3 years ago

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 commented 3 years ago

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