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.

Sign up for GitHub

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

Jump to bottom

Use a process watcher instead of cron when running under Docker #51

Closed
bschmalhofer opened this issue May 28, 2020 · 2 comments
Closed

Use a process watcher instead of cron when running under Docker #51

bschmalhofer opened this issue May 28, 2020 · 2 comments
Milestone
OTOBO 10.0.1

Comments

@bschmalhofer
Copy link
Contributor

The current bin/docker/entrypoint.sh runs under der user 'otobo' which can't, and shouldn't, start the Cron Daemon. This means that currently no cronjobs are executed in the docker container.

Therefore I propose to forego cronjobs and watch the Daemon.pl with a tool like supervisor. The downside it that no othere cronjobs can be supported. However my understanding is that all periodic actions should be handled by the Daemon anyways.
bschmalhofer added a commit to bschmalhofer/otobo that referenced this issue May 28, 2020
@bschmalhofer
Issue RotherOSS#51: first stab at running Daemon and Cron in own shell
0ba0fcc 
Not really tested yet.
@bschmalhofer
Copy link
Contributor Author

Thinking about it, it is probably best to stick with the Docker philosophy. Only one service per running Container. Thus put the Daemon into a separate container and watch it with cron. The different users can be handled in bin/docker/entrypoint.sh. Getting rid of cron is an idea that can be considered in future.

bschmalhofer added a commit to bschmalhofer/otobo that referenced this issue May 29, 2020
@bschmalhofer
Issus RotherOSS#51: move two commands into the Dockerfile
4700cb6 
Not sure whether this really makes sense.
The idea was to make the entrypoint smaller.
Also use more vars in the Dockerfile, less hardcoding.
bschmalhofer added a commit to bschmalhofer/otobo that referenced this issue May 29, 2020
@bschmalhofer
Issue RotherOSS#51: set PATH when starting the Daemon per Cron
f126c0a
bschmalhofer added a commit to bschmalhofer/otobo that referenced this issue May 29, 2020
@bschmalhofer
Issue RotherOSS#51: update the readme file
b53fc20 
There are now three containers.
No need to restart the Daemon.
No need for sudo when the user is in the docker group.
bschmalhofer added a commit to bschmalhofer/otobo that referenced this issue May 29, 2020
@bschmalhofer
Issue RotherOSS#51: fix list of automatically generated config cache …
43956ca 
…files
svenoe added a commit that referenced this issue May 29, 2020
@svenoe
Merge pull request #58 from bschmalhofer/issue-51-cron_service
f76f4f1 
Issue #51: fix list of automatically generated config cache files
@bschmalhofer
Copy link
Contributor Author

Closing because cron is running now in a dedicated container.

@bschmalhofer bschmalhofer added this to the OTOBO 10.0 milestone Jul 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
OTOBO 10.0.1
Development

No branches or pull requests
1 participant
@bschmalhofer