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

Enable multiple gods on one system, run by non-root users. #116

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

Conversation

lansing
Copy link

@lansing lansing commented Oct 21, 2012

The primary change in this pull request causes god to create and use a socket file in a user-specific directory when /var/run/god is not writable. Without this change, spinning up a second god on a box that already has one running will either clobber the first god's socket file, or fail to create a socket file and die at startup.

With this change, each user on a system can run its own independent god. I've been using this patch since February in order to run staging deployments for multiple apps on one server, while reusing the same god configs that are used in these apps' production deployments.

This pull request also includes tests for the changes.

@chewi
Copy link
Contributor

chewi commented Apr 24, 2013

Thanks, I really needed this. However, I felt it was really hampered by the need to specify the -p all the time so I added support for a GOD_PORT environment variable. By setting this to 0, one user's instance of god will never clash with another's. I also rebased your changes against the latest. Check it out at chewi/god@0230ddac26dc43c1280730f7dd512c7416a4c8f0.

@eric
Copy link
Collaborator

eric commented Jun 3, 2013

Fixing the location of the socket file to no longer be /tmp is definitely something we've talked about, but I'm definitely concerned with the upgrade path for something like this.

It seems like we need a way to gracefully handle the /tmp location when someone updates the gem on a running system.

Would be interested in input from @mojombo, @jnewland, @tmm1.

@chewi
Copy link
Contributor

chewi commented Oct 9, 2013

I've just rebased my branch against 0.13.3. I'm keen to see this merged as deploying a non-standard version is a real PITA. I can't rely on bundler's ability to use git because that doesn't install the god script to bin. I'm not sure what to do about this migration issue.

@nima22
Copy link

nima22 commented May 25, 2015

http://developers.google.com/on system

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.

None yet

4 participants