Skip to content
This repository has been archived by the owner on Sep 17, 2021. It is now read-only.

Use system supervisord and modular configuration file #47

Merged
merged 2 commits into from
Sep 15, 2014

Conversation

lucab
Copy link
Contributor

@lucab lucab commented Sep 1, 2014

Rework configuration files and docs to use system supervisord and avoid duplicating system config.

@scriptsrc
Copy link
Contributor

I like modifying the instructions to have them clone to /usr/local/src/ or /opt/ much better than in ~.

Should the quickstart1.rst instruct users to cd /usr/local/src/ before performing the repo clone?

Will they need to sudo to another user to have the appropriate permissions to clone into /usr/local/src ?

I see that you have supervisord running the web API as the www-data user. Will this user have permissions to append entries to the log files?

Luca Bruno added 2 commits September 5, 2014 14:22
When installed from distro package, supervisor can pick up
program-specific configuration files under /etc/supervisor/conf.d/
This commit provides a modular supervisord which can be dropped
there, omitting daemon configuration.

Signed-off-by: Luca Bruno <luca.bruno@rocket-internet.de>
Update all docs to point out how to use security_monkey with
system-wise installed supervisor.

Signed-off-by: Luca Bruno <luca.bruno@rocket-internet.de>
@lucab lucab force-pushed the lucab/to-upstream/system-supervisord branch from 9631f13 to bc33d94 Compare September 5, 2014 12:35
@lucab
Copy link
Contributor Author

lucab commented Sep 5, 2014

It depends on your setup. /usr/local/src is owned by root:staff, so either you go with sudo or you add your user to the the "staff" group.

Regarding the log, I'm currently writing it to a file in /var/log/seurity_monkey/ which is a dir owned by www-data:www-data.

I may amend the conf if you want, it's just loosely based on my setup as an example to be later tweaked.

scriptsrc pushed a commit that referenced this pull request Sep 15, 2014
Use system supervisord and modular configuration file
@scriptsrc scriptsrc merged commit 89a14cd into Netflix:develop Sep 15, 2014
@scriptsrc scriptsrc mentioned this pull request Sep 15, 2014
@scriptsrc scriptsrc mentioned this pull request Oct 30, 2014
@lucab lucab deleted the lucab/to-upstream/system-supervisord branch November 11, 2014 08:46
markofu pushed a commit to markofu/security_monkey that referenced this pull request Sep 1, 2015
…ervisord

Use system supervisord and modular configuration file
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants