-
Notifications
You must be signed in to change notification settings - Fork 6
Description
First, thanks for the great role. Pretty useful and concise.
As it's written, it's kind of annoying to update the server; it requires backing up saves and deleting the factorio directory to get the ansible script to extract the updated game files. I'm not sure what the optimal solution is (perhaps writing a version file or something, and doing a 'creates' on the extract task on that version file versus the whole directory. Handing upgrades (e.g. where a different factorio version is defined in your playbook) seamlessly would be quite a bit easier.
In a similar vein, the role could use a handler to restart the server on changes.
As a complete aside, providing a list of mod urls for the role to grab would be nice, but mods.factorio.com requires login for download; though player-data.json might be a way around that: https://forums.factorio.com/viewtopic.php?f=49&t=14395
BTW, it worked fine on debian jessie; other than my SSL certs being out of date (so verification of the download url failed at first). In general feedback, it's a little unintuitive that I had to define visibility and description as a factorio_server_settings to get it to actually run; defining some minimally viable defaults (or denoting that on your how to use page) might be a good idea.