Skip to content

cron in Docker image#1871

Merged
Alkarex merged 5 commits intoFreshRSS:devfrom
Alkarex:cron-in-docker
Apr 26, 2018
Merged

cron in Docker image#1871
Alkarex merged 5 commits intoFreshRSS:devfrom
Alkarex:cron-in-docker

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Apr 23, 2018

@Alkarex Alkarex added this to the 1.11.0 milestone Apr 23, 2018
Copy link
Copy Markdown
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Apr 25, 2018

@kordianbruck Does that do the job for you?

```sh
sudo docker run -dit --restart unless-stopped --log-opt max-size=10m \
-v $(pwd)/data:/var/www/FreshRSS/data \
-e 'CRON_MIN=17,37' \
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ups, that does not work in that case. Fix needed.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in ecf366d

@Alkarex Alkarex merged commit 4381117 into FreshRSS:dev Apr 26, 2018
@Alkarex Alkarex deleted the cron-in-docker branch April 26, 2018 10:00
@kordianbruck
Copy link
Copy Markdown

@Alkarex this looks great! thanks!

Alkarex added a commit that referenced this pull request Apr 26, 2018
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this pull request Mar 25, 2021
* cron in Docker image

FreshRSS#1869

* Fix cron CMD

* Minor readme

* Docker run d instead of dit

There should not be a need for STDIN or TTY

* Minor sed param
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.

3 participants