Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Complement image: propagate SIGTERM to all workers #13914

Merged
merged 3 commits into from
Sep 26, 2022
Merged

Complement image: propagate SIGTERM to all workers #13914

merged 3 commits into from
Sep 26, 2022

Commits on Sep 26, 2022

  1. Complement image: propagate SIGTERM to all workers 

    This should mean that logs from worker processes are flushed before shutdown.

When a test completes, Complement stops the docker container, which means that
synapse will receive a SIGTERM. Currently, the `complement_fork_starter` exits
immediately (without notifying the worker processes), which means that the
workers never get a chance to flush their logs before the whole container is
vaped. We can fix this by propagating the SIGTERM to the children.
    @richvdh
    richvdh committed Sep 26, 2022
    Configuration menu
    Copy the full SHA
    5f22aa1 View commit details
    Browse the repository at this point in the history

  2. fix lint

    @richvdh
    richvdh committed Sep 26, 2022
    Configuration menu
    Copy the full SHA
    3cabcc3 View commit details
    Browse the repository at this point in the history

  3. Update synapse/app/complement_fork_starter.py

    @richvdh
    richvdh authored Sep 26, 2022
    Configuration menu
    Copy the full SHA
    b5ea2f8 View commit details
    Browse the repository at this point in the history