devops-coop / ansible-minecraft

Ansible role for provisioning a vanilla Minecraft server
https://galaxy.ansible.com/devops-coop/minecraft/
Apache License 2.0
57 stars 38 forks source link

Support Spigot #7

Open benwebber opened 8 years ago

benwebber commented 8 years ago
danizen commented 6 years ago

So, should I expect this to work at this point? Which branch would I use to try it? Obviously, the JDK has to be installed for this to work...

danizen commented 6 years ago

Definitely not working on master branch - I will proceed with the minecraft_url setting to download a copy of Spigot I've built and marshalled in place.

benwebber commented 6 years ago

master worked as of December:

https://travis-ci.org/devops-coop/ansible-minecraft/jobs/315195888

It is possible that the Spigot build script has changed. I don't personally use Spigot. Can you share the error message you got using the master branch?

jaygooby commented 6 years ago

@danizen If you used ansible-galaxy to install from galaxy (ansible-galaxy install devops-coop.minecraft), then you won't have master with spigot support (setting minecraft_server: spigot doesn't fail - it just fetches and installs vanilla minecraft).

You need to use the github repo directly:

ansible-galaxy install git+https://github.com/devops-coop/ansible-minecraft.git

This was still giving me issues, but it's a simple fix: https://github.com/devops-coop/ansible-minecraft/pull/25/commits/ff6c5c8fec0686d445cc7e1e31a24dfb69f5052b

danizen commented 6 years ago

Fair enough. FWIW, we all know we should mark an issue as resolved when it is fixed, and closed when the version containing the fix has "shipped" or been installed.

At the same time, if I use a role for work, I generally clone it and use it from my own repository rather than galaxy, because I don't want it to change under me, and I don't want to manage a galaxy repository in addition to our git installation!