Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traceback on container exit #262

Closed
peb-adr opened this issue Oct 19, 2023 · 0 comments · Fixed by #264
Closed

Traceback on container exit #262

peb-adr opened this issue Oct 19, 2023 · 0 comments · Fixed by #264
Assignees
Labels
Milestone

Comments

@peb-adr
Copy link
Member

peb-adr commented Oct 19, 2023

This is basically the same issue as OpenSlides/openslides-media-service#72.

a85e867 Is the commit which introduced it for this service.
The rest of the problem is the same so I'll copy paste it to here.

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/gunicorn/arbiter.py", line 609, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.10/site-packages/gunicorn/workers/base.py", line 142, in init_process
    self.run()
  File "/usr/local/lib/python3.10/site-packages/gunicorn/workers/sync.py", line 126, in run
    self.run_for_one(timeout)
  File "/usr/local/lib/python3.10/site-packages/gunicorn/workers/sync.py", line 85, in run_for_one
    self.wait(timeout)
  File "/usr/local/lib/python3.10/site-packages/gunicorn/workers/sync.py", line 37, in wait
    ret = select.select(self.wait_fds, [], [], timeout)
TypeError: exit expected at most 1 argument, got 3

sys.exit is set as handler for SIGTERM and SIGINT.
However is seems there was a missed detail. The handler will be called with two arguments (source).
@jsangmeister I guess an easy fix would be adding an (anonymus) function that ignores the the args and then calls exit. But I trust you can find the best suitable solution for this ;) .

@peb-adr peb-adr added the bug label Oct 19, 2023
@peb-adr peb-adr added this to the 4.1 milestone Oct 19, 2023
@jsangmeister jsangmeister self-assigned this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants