Skip to content
This repository has been archived by the owner on Oct 10, 2021. It is now read-only.

Add CI Testing #5

Merged
merged 3 commits into from
Mar 2, 2018
Merged

Conversation

jonathangreen
Copy link
Contributor

Ticket:
Islandora/documentation#794

Doesn't test cent, but we should once cent is supported.

Currently fails, and will until islandora-deprecated/ansible-role-tomcat8#6 is merged.

.travis.yml Outdated
services: docker

env:
- distro: centos7
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good call. missed that.

@jonathangreen
Copy link
Contributor Author

Something more going on here then I thought... I'll take a look into it when a get a chance today.

@jonathangreen
Copy link
Contributor Author

A big part of my issues here was making sure the Blazegraph configuration files are found correctly. I added a second way to specify it, which puts it in the Blazegraph web.xml.

Its a little ugly to just template the web.xml file, and if anyone has better ideas let me know. I made it so you can switch it off and also put it in the java_opts if you want, but that requires the help of whatever role sets the java_opts variables.

I made some notes in the readme about all of this.

Copy link
Contributor

@whikloj whikloj left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants