Skip to content
This repository has been archived by the owner on Apr 30, 2020. It is now read-only.

change commands to sudo. change supervisor to restart #4

Closed
wants to merge 3 commits into from
Closed

change commands to sudo. change supervisor to restart #4

wants to merge 3 commits into from

Conversation

krystofbe
Copy link

@krystofbe krystofbe commented Dec 20, 2016

Hi @jakeheis,

tried Flock today. Such a great tool.
I ran into some problems trying to use flock tools

SupervisordTasks.swift:

  • For security reasons, i think it's better to use sudo for running apt-get and service commands. A well secured sudo setup will ensure that a given user can only execute certain commands.
  • On my Ubuntu 14.04 Box, supervisor already gets started after the installation. So trying to start it again with service supervisor start returns 1 and the whole process errors.
  • double %% static var errorLog = "/var/log/supervisor/%%(program_name)s-%%(process_num)s.err"
    give me this error error: <class 'xmlrpclib.ProtocolError'>, <ProtocolError for 127.0.0.1/RPC2: 500 Internal Server Error>: file: /usr/lib/python2.7/dist-packages/supervisor/xmlrpc.py line: 470 Channging to single % fixes this for me

Best wishes from Germany!

@jakeheis
Copy link
Owner

Thanks for your contribution! I'm going to hold off on merging this until we decide what to do with #5 since the two are tightly related

@jakeheis
Copy link
Owner

eb24630 incorporates some of these changes while keeping sudo out of calls made during deploy (#5). Thanks again for your contribution!

@jakeheis jakeheis closed this Dec 28, 2016
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