-
Notifications
You must be signed in to change notification settings - Fork 53
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
Flysystem Install #72
Conversation
My gist was failing to deploy some openseadragon changes, so I updated it and the install instructions. This PR is good to go now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as advertised. Yay!
Note: on CentOS the last line of the gist didn't work (jar copying failed); but it wasn't any trouble to run that line's commands one at a time.
Ok, took me a bit to understand but if I understand correctly the derivative "files" are not moved to Fedora but the derivative media objects are. |
@whikloj Yes. That's it. Once Fedora 5 lands we'll actually wire those Media up as external content. The approach in this PR would remain (and probably be improved) to work with something like Trellis which won't support the feature. |
@Islandora-Devops/committers @manez @MarcusBarnes @ysuarez @mjordan @rosiel @bryjbrown @DigitLib and whoever else^^ Ok, Travis is green everywhere except islandora-deprecated/islandora_image#24, which requires Islandora/documentation#92 to get merged first. I've responded to a lot of feedback already, but more is welcome. I've cycled my box using these instructions and everything works as I expect it to. Some things I've done:
|
You should be able to test now without having to run that bash script. Pull in these changes and |
@dannylamb ansible-role-crayfish needs to move to |
@whikloj Done. Thx. I manually set the crayfish version in my inventory to test before islandora-deprecated/ansible-role-crayfish#11 landed and didn't notice this. I assume this means we'll need to run |
No I think when you run |
Nope it doesn't overwrite, easiest is to remove the contents of your roles/external directory. |
Ubuntu works after removing the roles/external directory. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also works on Centos.
Install code for Islandora/documentation#769
Using this PR to install with Flysystem support
vagrant up
vagrant ssh
curl https://gist.githubusercontent.com/dannylamb/27ad287ae92afd7139a98435446550d6/raw/6e9f369139528d3445e19e187e6a22a5a1386d4d/gistfile1.txt | sudo bash
article
bundle, but that's ok. It's benign, but I have to sort that out inislandora_core_feature
.Testing Flysystem Support
I've been testing by adding an Image with a Preservation Master and watching it go. It's easiest to see what's going on by checking Gemini. Before doing anything, I confirm that the taxonomy terms are in and mapped.
Then go add an Image node and give it a Preservation Master media.
http://localhost:8000/_flysystem/fedora/2018-07/...
The file you uploaded will wind up in Fedora, but the derivatives will land in Drupal's public filesystem. Check out Gemini again and it will have all the relevant nodes/media/files mapped.
Note that files in Fedora are mapped in Gemini, whilst the others are not. Conversely, for Media, if the source is not in Fedora it is mapped in Gemini. Media whose source are in Fedora are not mapped, but looked up using the
describedby
link header in Fedora responses.You should be able to confirm that all the resources that should be are present in Fedora.