buildstream-migration / bst-staging

GNU Lesser General Public License v2.1
0 stars 0 forks source link

Rare exception when terminating processes #20

Open Cynical-Optimist opened 4 years ago

Cynical-Optimist commented 4 years ago

See original issue on GitLab In GitLab by [Gitlab user @tristanvb] on May 2, 2017, 11:22

In some corner cases, we are experiencing the symptoms described in this python upstream bug report

This is mostly harmless, but is nevertheless a stack trace thrown at the user for no good reason, e.g.:

Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/usr/lib/python3.5/multiprocessing/util.py", line 288, in _exit_function
    _run_finalizers(0)
  File "/usr/lib/python3.5/multiprocessing/util.py", line 248, in _run_finalizers
    items = [x for x in list(_finalizer_registry.items()) if f(x)]
RuntimeError: dictionary changed size during iteration
Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @tristanvb] on Aug 22, 2017, 18:22

This hasn't happened in a long time, I think it's been addressed by using os._exit() from SIGTERM handlers.

Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @tristanvb] on Aug 22, 2017, 18:22

closed

Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @tacgomes] on Jul 27, 2018, 11:38

reopened

Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @tacgomes] on Jul 27, 2018, 11:39

This is frequently happening when building the freedesktop-sdk in the CI: https://gitlab.com/BuildStream/buildstream/-/jobs/84724565

Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @franred] on Aug 7, 2018, 10:28

In python upstream bug report is marked as resolved and ported to 3.5, 3.6 and 2.7. So I would expect that the issue goes away when the distros ship the new version of 3.5.

[Gitlab user @tacgomes], [Gitlab user @tristanvb], Should we mark this as won't fix or force the buildstream to use 3.5.3 at least patch ported to 3.5 MR ?

Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @tacgomes] on Aug 7, 2018, 11:34

Note that on the link that I've posted above the Python version used seems to be 3.5.

Cynical-Optimist commented 4 years ago

In GitLab by [Gitlab user @tacgomes] on Aug 17, 2018, 16:28

marked this issue as related to #593