MagicStack / MagicStack/uvloop

uvloop runs at_fork hook when running subprocess

Offen
#749 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Cython
Sterne
11.9k
Forks
616
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Speaking about `os.register_at_fork` python docs [explicitly state](https://docs.python.org/dev/library/os.html#os.register_at_fork):
> These calls are only made if control is expected to return to the Python interpreter. A typical [subprocess](https://docs.python.org/dev/library/subprocess.html#module-subprocess) launch will not trigger them as the child is not going to re-enter the interpreter.

The standard `asyncio` loop behaves this way. But `uvloop` doesn't. Repro:
```python
import asyncio
import os

import uvloop

def hook():
os.write(2, b"TRIGGERED\n")


async def run():
process = await asyncio.create_subprocess_exec(
"uptime",
stdout=asyncio.subprocess.PIPE,
stderr=asyncio.subprocess.PIPE,
start_new_session=True
)

stdout, stderr = await process.communicate()
print(f"STDOUT: {stdout.decode('utf-8').strip()}")

if __name__ == "__main__":
os.register_at_fork(after_in_child=hook)

print("asyncio")
asyncio.run(run())

print("uvloop")
uvloop.install()
asyncio.run(run())
```

Output:
```
asyncio
STDOUT: 16:56:42 up 35 days, 6:20, 1 user, load average: 5.21, 4.34, 4.27
uvloop
TRIGGERED
STDOUT: 16:56:42 up 35 days, 6:20, 1 user, load average: 5.21, 4.34, 4.27
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, uvloops Verarbeitung von asyncio.create_subprocess_exec und den Fork-Pfad zu verfolgen, der nach uvloop.install() verwendet wird, und vergleiche ihn mit der standardmäßigen asyncio-Schleife und dem Verhalten von os.register_at_fork. Führe die bereitgestellte Reproduktion mit dem uptime-Subprozess erneut aus; fertig ist es, wenn der after_in_child-Hook für diesen Subprozessstart nicht ausgelöst wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
operating-systems
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
58/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.