fix(cron): make sure cron container only returns one container id #1

2025-09-29T20:26:56Z

kolaente commented

2025-09-29 20:26:56 +00:00

This fixes a bug where another app with the same name was matched by the cron. For example, if the pretix stack is called tickets_domain_de and there is another app with a name like traefik-tickets_domain_de the regex would match both and try to execute with the latter container (failing).

This fixes a bug where another app with the same name was matched by the cron. For example, if the pretix stack is called `tickets_domain_de` and there is another app with a name like `traefik-tickets_domain_de` the regex would match both and try to execute with the latter container (failing).

kolaente added 1 commit 2025-09-29 20:26:57 +00:00

fix: d67f375d47

simon approved these changes 2025-10-02 12:05:57 +00:00

simon left a comment

oh, nasty issues! tysm for the find!

simon merged commit f277fe7070 into main

2025-10-02 12:06:04 +00:00

simon referenced this issue from a commit

2025-10-02 12:06:04 +00:00

Merge pull request 'fix(cron): make sure cron container only returns one container id' (#1) from kolaente/pretix:fix/cron into main

kolaente commented

2025-10-02 13:07:18 +00:00

On second thought (but I would need to check this): maybe we could do this without the docker ps and pass the docker app name directly to docker exec?

On second thought (but I would need to check this): maybe we could do this without the `docker ps` and pass the docker app name directly to `docker exec`?

kolaente deleted branch fix/cron

2025-10-02 13:07:25 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/pretix#1

Allow edits from maintainers