Certain jobs fail to register.

[ccarney16]

Hello! I saw your project and looked into migrating over due to support of dynamically adding in cron jobs. However I noticed that upon migrating over, all run jobs and certain exec jobs are not being seen. I largely use container labels to create and run tasks and have taken into account the change in the label prefix (ofelia to chadburn).

These examples tasks are taken from a docker-compose project I maintain on github that currently uses ofelia:

What does work with chadburn:

    labels:
      chadburn.enabled: "true"
      chadburn.job-exec.pqueue.schedule: "@every 1m"
      chadburn.job-exec.pqueue.command: "php /var/www/html/artisan schedule:run"
      chadburn.job-exec.pqueue.user: "nginx"

What does not work with chadburn:

    labels:
      chadburn.enabled: "true"
      chadburn.job-run.le-renew.schedule: "0 0 0 * * *"
      chadburn.job-run.le-renew.container: pterodactyl_certbot_1

    labels:
      chadburn.enabled: "true"
      chadburn.job-exec.daemon-reload.schedule: "${DAEMON_RELOAD_SCHEDULE:-0 30 0 * * *}"
      chadburn.job-exec.daemon-reload.command: "kill -15 1"

I am not too sure if there are any additional configuration needed. but any assistance would be appreciated!

Also as a sidenote, would it be possible to omit the container option in run jobs?

[airesch]

The 6-part cron schedules don't work like they do in ofelia, try using 5 parts
chadburn.job-run.le-renew.schedule: "0 0 0 * * *"
needs to be instead:
chadburn.job-run.le-renew.schedule: "0 0 * * *"
This threw me at first as well for the original dynamic update ofelia PR and may have contributed to it never being accepted

[ccarney16]

yeah, that only fixed half of the problem, the exec jobs now work fine, but run jobs are still not seen upon update. I have containers that need to be started up every so often.

[miggland]

It would be a good idea to fix the README - this same error just cost me a bunch of time as well. I even checked the syntax against that given on the front page of this project, which explicitly states "Go syntax", with 6 entries!

I'm pretty sure I changed it to 6 at some point to make it work, and now I have to change it back... not sure when this change happened.

[miggland]

I found out when I made the change - end of december. My own commit message was "Update Cron definitions to fit Go version used in Chadburn". The day before I switched from Ofelia to Chadburn.

-> At the time, Go syntax for cron was required. Since then, it has apparently changed and my schedules stopped working. I wasn't paying attention, so I didn't notice it happening.

[maietta]

Hi @miggland.

Thanks for letting me know, although, I don't seem to be getting alerts for every issue raised here.

I do plan to fully rewrite Chadburn or at minimum refactor the project to address a large number of issues people are having. When I decided to hard fork this project, I didn't realize just how many issues people were having.

I plan to get started fairly soon, after I can get some of my paid projects out of the way.

[miggland]

Hi @maietta
That sounds like a big plan, and I'm sure Chadburn will work better afterwards, even though (when it runs :)) it does exactly what I expect.

Until then, maybe just remove the wrong info in the Readme?

`
Scheduling format is the same as the Go implementation of cron. E.g. @every 10s or 0 0 1 * * * (every night at 1 AM).

Note: the format starts with seconds, instead of minutes.
`

[CybotTM]

same problem here:

only job with schedule "5-59/1 * * * *" will be registered, the other two are not ("5-59/10 * * * * *", "10 */1 * * * *"), but all three get deregistered.

$ docker run \
    -l "chadburn.enabled=true" \
    -l "chadburn.job-exec.o.command=date +%Y-%m-%dT%H:%M:%S.%N-" \
    -l "chadburn.job-exec.o.schedule=5-59/10 * * * * *" \
    -d --name test_o nginx
$ docker run \
    -l "chadburn.enabled=true" \
    -l "chadburn.job-exec.oo.command=date +%Y-%m-%dT%H:%M:%S.%N--" \
    -l "chadburn.job-exec.oo.schedule=5-59/1 * * * *" \
    -d --name test_oo nginx
$ docker run \
    -l "chadburn.enabled=true" \
    -l "chadburn.job-exec.ooo.command=date +%Y-%m-%dT%H:%M:%S.%N---" \
    -l "chadburn.job-exec.ooo.schedule=10 */1 * * * *" \
    -d --name test_ooo nginx

$ docker logs -f --tail 20 chadburn

ron_utils.go:13 ▶ DEBUG added
scheduler.go:45 ▶ NOTICE New job registered "oo" - "date +%Y-%m-%dT%H:%M:%S.%N--" - "5-59/1 * * * *" - ID: 22
cron_utils.go:13 ▶ DEBUG wake
cron_utils.go:13 ▶ DEBUG run
common.go:127 ▶ NOTICE [Job "oo" (b236fdc815b9)] Started - date +%Y-%m-%dT%H:%M:%S.%N--
common.go:127 ▶ NOTICE [Job "oo" (b236fdc815b9)] Output: 2022-04-13T06:44:00.083836027--
common.go:127 ▶ NOTICE [Job "oo" (b236fdc815b9)] Finished in "81.076233ms", failed: false, skipped: false, error: none
cron_utils.go:13 ▶ DEBUG wake
cron_utils.go:13 ▶ DEBUG run
common.go:127 ▶ NOTICE [Job "oo" (f13733149d3b)] Started - date +%Y-%m-%dT%H:%M:%S.%N--
common.go:127 ▶ NOTICE [Job "oo" (f13733149d3b)] Output: 2022-04-13T06:45:00.082202380--
common.go:127 ▶ NOTICE [Job "oo" (f13733149d3b)] Finished in "80.747886ms", failed: false, skipped: false, error: none

$ docker stop test_o test_oo test_ooo
$ docker rm test_o test_oo test_ooo

$ docker logs -f --tail 20 chadburn

cron_utils.go:13 ▶ DEBUG added
scheduler.go:45 ▶ NOTICE New job registered "oo" - "date +%Y-%m-%dT%H:%M:%S.%N--" - "5-59/1 * * * *" - ID: 22
cron_utils.go:13 ▶ DEBUG wake
cron_utils.go:13 ▶ DEBUG run
common.go:127 ▶ NOTICE [Job "oo" (b236fdc815b9)] Started - date +%Y-%m-%dT%H:%M:%S.%N--
common.go:127 ▶ NOTICE [Job "oo" (b236fdc815b9)] Output: 2022-04-13T06:44:00.083836027--
common.go:127 ▶ NOTICE [Job "oo" (b236fdc815b9)] Finished in "81.076233ms", failed: false, skipped: false, error: none
cron_utils.go:13 ▶ DEBUG wake
cron_utils.go:13 ▶ DEBUG run
common.go:127 ▶ NOTICE [Job "oo" (f13733149d3b)] Started - date +%Y-%m-%dT%H:%M:%S.%N--
common.go:127 ▶ NOTICE [Job "oo" (f13733149d3b)] Output: 2022-04-13T06:45:00.082202380--
common.go:127 ▶ NOTICE [Job "oo" (f13733149d3b)] Finished in "80.747886ms", failed: false, skipped: false, error: none
scheduler.go:50 ▶ NOTICE Job deregistered (will not fire again) "o" - "date +%Y-%m-%dT%H:%M:%S.%N-" - "5-59/10 * * * * *" - ID: 0
cron_utils.go:13 ▶ DEBUG removed
scheduler.go:50 ▶ NOTICE Job deregistered (will not fire again) "oo" - "date +%Y-%m-%dT%H:%M:%S.%N--" - "5-59/1 * * * *" - ID: 22
cron_utils.go:13 ▶ DEBUG removed
scheduler.go:50 ▶ NOTICE Job deregistered (will not fire again) "ooo" - "date +%Y-%m-%dT%H:%M:%S.%N---" - "10 */1 * * * *" - ID: 0
cron_utils.go:13 ▶ DEBUG removed

[CybotTM]

Hi @maietta , thanks for your work on this.