Skip to content
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

Add support for Private Servers and for Memory Segments #8

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

ross-p
Copy link

@ross-p ross-p commented Apr 10, 2017

Add Private Server Support

Copied private server work from @Parakoopa and modified it to be backward-compatible. All existing ansible command-lines will continue to work as they previously did.

New optional ansible-playbook environment vars:

  • -e screeps_basic_auth=1
  • -e screeps_hostname=http://your.private.server.name:21025

Add Memory Segment Support

By default stats are still read from Memory.stats, however you can now optionally load stats from a memory segment instead.

New optional ansible-playbook environment vars:

  • -e screeps_stats_source=segment=0 will load the stats key from memory segment 0. Change to whatever segment number you wish to use.

Add Documentation

Added documentation (thanks @ags131) to help other people work with this code base in the future.

Developer Note

If you merge this, you'll also need to publish a new Docker Hub image of screepers/screeps-statsd. Don't forget to do that. ;)

@AlinaNova21
Copy link
Member

Pinging @ButAds :)
I can build and push it to screepers on dockerhub once merged :)

@ross-p
Copy link
Author

ross-p commented Jul 9, 2017

@ButAds I had to add back the node:image element of the docker-compose.yml.j2 since ansible was unable to start the docker containers without that. Otherwise your recent changes are merged.

@TimMeissner
Copy link

docker-compose.yml file is missing in your repo btw

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

Successfully merging this pull request may close these issues.

4 participants