fix: Fix Swarm mode event handler, split container/service event filters #1

Merged

w33ble merged 3 commits from refs/pull/1/head into main 2026-05-17 13:36:04 -06:00

Conversation 0 Commits 3 Files changed 1 +9 -15

w33ble commented 2026-05-17 13:05:40 -06:00 (Migrated from git.w33ble.com)

Copy link

The original event filter required status='start' AND Action='update'
simultaneously, which never matches any Docker event since these fields
are mutually exclusive. This broke live Swarm mode updates entirely.

• Change filter to listen for both container and service event types
• Remove broken client.services.get(container_id) call that always
  threw NotFound (container IDs and service IDs are different namespaces)
• Use event.Actor.ID for service update events
• Remove dead code (unused client.services.list() call)
• Add debug logging for raw Docker events

The original event filter required status='start' AND Action='update' simultaneously, which never matches any Docker event since these fields are mutually exclusive. This broke live Swarm mode updates entirely. - Change filter to listen for both container and service event types - Remove broken client.services.get(container_id) call that always threw NotFound (container IDs and service IDs are different namespaces) - Use event.Actor.ID for service update events - Remove dead code (unused client.services.list() call) - Add debug logging for raw Docker events

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

w33ble/docker-traefik-cloudflare-companion!1

No description provided.