Pins down grok to 3.1.0

[dannylamb]

GitHub Issue: Islandora/documentation#1285

Also see islandora-deprecated/ansible-role-grok#8, which updates the default variable in the role to match.

What does this Pull Request do?

Updates Grok to v3.1.0, which does not seg fault when converting tiffs on Ubuntu 18.04 like v2.3.0 does.

How should this be tested?

• Pull in this PR
• vagrant up

After it's done building, put a Tiff in your playbook's root directory, ssh in, and run convert on it.

• vagrant ssh
• cd /home/ubuntu/islandora
• convert my.tiff my.jp2

No seg fault! You can run identify on the resulting jp2 to confirm it's ok.

Interested parties

@Islandora-Devops/committers

[elizoller]

I am testing this now

[whikloj]

This worked for me 👍

[elizoller]

The command line convert statement worked but when I upload a TIF to a media object, I encountered an issue:

Drupal\Core\Field\FieldException: Attempt to create a field fits_exiftool_compression_scheme that does not exist on entity type media. in Drupal\field\Entity\FieldConfig->getFieldStorageDefinition() (line 312 of /var/www/html/drupal/web/core/modules/field/src/Entity/FieldConfig.php).

Appears to be related to FITS though and I did get JPEG derivs on my media.

[whikloj]

Did you clear your roles/external/* directories? I didn't see any errors when I uploaded a Tiff.

[elizoller]

It was a fresh clone of the playbook.

[dannylamb]

Exactly when did you get this? The FITS module auto-generates a bunch of fields the first time a file type comes through. If you catch it at just the wrong time, maybe it's a timing issue while it's generating a bunch of fields?

[elizoller]

I was trying to view the node page but I can see everything now - node page, media pages, FITS derivative. I think you're right about the timing issue.
@whikloj are you giving the review or do you want me to do it?

[seth-shaw-unlv]

@elizoller, both of you can review it. 🙂

[elizoller]

thats fancy

[elizoller]

This PR worked as described, I was able to convert a TIF to a JPG via the convert command and via microservice

[dannylamb]

Thanks for moving this one though. I'll make a ticket for the issue you discovered @elizoller.