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

Better supervisor config #146

Closed
wants to merge 2 commits into from

Conversation

amoffat
Copy link
Contributor

@amoffat amoffat commented May 24, 2017

now that we have http://chaosthebot.com:8081/ for logs, let's make the log files a little more manageable

also, we'll lean on supervisor for restarting things

@chaosbot
Copy link
Collaborator

🙅 This PR did not meet the required vote threshold and will not be merged. Closing.

Open a new PR to restart voting.

@hongaar
Copy link
Member

hongaar commented May 24, 2017

@amoffat probably try this again after #153 merged

@amoffat amoffat mentioned this pull request May 24, 2017
chaosbot added a commit that referenced this pull request May 24, 2017
#184: Better supervisor config

Description:
let's try this again #146, now that we have a democracy back
-----
now that we have http://chaosthebot.com:8081/ for logs, let's make the log files a little more manageable

also, we'll lean on supervisor for restarting things

:ok_woman: PR passed with a vote of 1 for and 0 against, with a
weighted total of 1.0 and a threshold of 1.0.

Vote record:
@amoffat: 1
@amoffat amoffat deleted the better-supervisor-config branch May 24, 2017 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants