-
-
Notifications
You must be signed in to change notification settings - Fork 30.4k
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
3.12.4 breakslogging.config.DictConfig
with logging.handlers.QueueHandler
on read-only file systems
#120868
Labels
type-bug
An unexpected behavior, bug, or error
Comments
4 tasks
provinzkraut
added a commit
to provinzkraut/cpython
that referenced
this issue
Jun 22, 2024
…ng.Manager was created eagerly
provinzkraut
added a commit
to provinzkraut/cpython
that referenced
this issue
Jun 22, 2024
…ng.Manager was created eagerly
provinzkraut
added a commit
to provinzkraut/cpython
that referenced
this issue
Jun 22, 2024
vsajip
pushed a commit
that referenced
this issue
Jun 27, 2024
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Jun 27, 2024
…QueueHandler` (pythonGH-120872) (cherry picked from commit 7d9c685) Co-authored-by: Janek Nouvertné <[email protected]>
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Jun 27, 2024
…QueueHandler` (pythonGH-120872) (cherry picked from commit 7d9c685) Co-authored-by: Janek Nouvertné <[email protected]>
mrahtz
pushed a commit
to mrahtz/cpython
that referenced
this issue
Jun 30, 2024
noahbkim
pushed a commit
to hudson-trading/cpython
that referenced
this issue
Jul 11, 2024
estyxx
pushed a commit
to estyxx/cpython
that referenced
this issue
Jul 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bug report
Bug description:
When using
logging.config.dictConfig
to configure alogging.handlers.QueueHandler
on a read only file system, it works fine in 3.12.3, but crashes in 3.12.4.Reproducing
Run
docker run --rm --read-only -it $(docker build -q .)
From my understanding, this is related to #119819 et al. and the change that caused this was introduced in #120030: It introduces creating ephemeral instances of
multiprocessing.Manager
which, on init, tries to acquire a temporary directory.cpython/Lib/logging/config.py
Line 785 in 879d1f2
CPython versions tested on:
3.12
Operating systems tested on:
Linux
Linked PRs
logging.config
when usingQueueHandler
#120872logging.config
when usingQueueHandler
(GH-120872) #121077logging.config
when usingQueueHandler
(GH-120872) #121078The text was updated successfully, but these errors were encountered: