Ampel / ampel (push) Successful in 41s
WAS: Neues Paket src/rippy (core/drives/rip). detection.py,
makemkv_daten.py und notify.py lagen je ZWEIMAL im Repo — unter
docker/api UND docker/worker, byte-identisch. Jetzt gibt es sie einmal;
beide Container importieren dieselbe Datei. Verhalten unveraendert.
WARUM: Es gab kein geteiltes Paket zwischen den Containern, deshalb die
Kopien. docker/api/db.py sagt es im Kopfkommentar selbst: "Wer die
Struktur aendert, aendert BEIDE Dateien." Ein Waechter-Test
(test_zwillinge_sind_byteweise_identisch) hat das mechanisch
abgesichert — er war noetig, weil die Konstruktion falsch war. Erste
Etappe des v2-Plans (KONZEPT-V2.md §8.3).
IM EINZELNEN:
- src/rippy/{core/notify, drives/detection, rip/makemkv_daten}.py
- conftest.py in der Wurzel legt src/ auf den sys.path (die Ampel ruft
pytest dort auf).
- Beide Dockerfiles kopieren src/rippy nach /app/rippy — /app ist
Arbeitsverzeichnis und uvicorn-App-Dir, also ohne PYTHONPATH findbar.
- worker_setup_paket packt das Paket ausdruecklich mit ins Zip: die
Schleife sah nur die oberste Ebene, ein Unterordner waere nie
mitgekommen und der Windows-Worker beim Start gestorben.
- Die zwei Test-Dateien fuer makemkv_daten sind zu einer verschmolzen
(die Faelle aus beiden). Damit entfaellt auch der importlib-Umweg im
Worker-Test: der war noetig, weil bei "pytest -q" aus der Wurzel
docker/api zuerst eingesammelt wird und jeder weitere Import nur noch
den sys.modules-Cache trifft — die Worker-Kopie wurde also nie
angefasst. Netto -13 doppelte Tests, +2 neue (siehe unten).
EIN FEHLER, DEN DER UMBAU FAST AUSGELIEFERT HAETTE: In tasks.py steht
der detection-Import in einem "try/except ImportError" — absichtlich,
denn der native Windows-Worker hat kein fcntl und soll trotzdem
starten. Das except verschluckt aber JEDEN ImportError, auch einen
falschen Modulpfad. Auf Linux haette der Worker ab sofort still
detect_disc_type=None gesetzt und jeden Rip verweigert, ohne dass
irgendwo ein Fehler stuende — genau die Klasse "still scheiternder
Hintergrund-Prozess" aus AGENTS.md. Gefunden ueber eine zweite Suche
mit eingerueckten Treffern.
Waechter dagegen: src/rippy/test_paket.py prueft mit
importlib.util.find_spec (fuehrt nichts aus, laeuft also auch auf
Windows ohne fcntl), dass es die drei Modulpfade wirklich gibt. Beim
Wegnehmen von detection.py gegengeprueft: Test wird rot, nach
Wiederherstellen gruen.
GEMESSEN: ruff sauber, 290 Tests gruen + 1 uebersprungen (lokal, ohne
die zwei Linux-only-Module). Kein Modul liegt noch doppelt im Repo.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
111 lines
3.6 KiB
Python
111 lines
3.6 KiB
Python
"""Laufwerks-Discovery über /sys und ioctls — ehrlich, ohne udev.
|
|
|
|
Warum kein udevadm mehr: Im Container läuft kein udevd, die udev-Datenbank ist
|
|
leer — `udevadm info` lieferte schlicht nichts (der alte Weg zeigte deshalb nie
|
|
ein Laufwerk an). /sys/class/block/<name>/device/{vendor,model} kommt dagegen
|
|
direkt vom Kernel und funktioniert überall.
|
|
"""
|
|
|
|
import glob
|
|
import os
|
|
from fcntl import ioctl
|
|
|
|
from rippy.drives.detection import (
|
|
CDS_DISC_OK,
|
|
classify,
|
|
disc_size_bytes,
|
|
disc_status,
|
|
drive_status,
|
|
)
|
|
|
|
# include/uapi/linux/cdrom.h
|
|
CDROMEJECT = 0x5309
|
|
CDROM_LOCKDOOR = 0x5329 # 1 = Tür verriegeln, 0 = entriegeln
|
|
CDROM_DRIVE_STATUS = 0x5326
|
|
CDS_NO_DISC = 1
|
|
CDS_TRAY_OPEN = 2
|
|
|
|
AUSWURF_WARTEN_SEKUNDEN = 5
|
|
|
|
|
|
def eject(device_path: str) -> None:
|
|
"""Wirft die Disc aus und prüft es nach. Wirft OSError, wenn sie drin bleibt.
|
|
|
|
⚠️ ERST ENTRIEGELN (Befund 26.07.2026, am laufenden System gemessen): Ein
|
|
nacktes CDROMEJECT wird von einem verriegelten Laufwerk mit ERFOLG quittiert
|
|
und tut nichts. MakeMKV verriegelt die Tür während des Rips
|
|
(`CDROM_LOCKDOOR 1`) und entriegelt sie nicht wieder — danach blieb die
|
|
Schublade zu, während Rippy „Disc ausgeworfen" ins Log schrieb. Gegenprobe
|
|
an derselben Disc: mit `CDROM_LOCKDOOR 0` davor geht sie auf (Status 2).
|
|
Deshalb macht das Werkzeug `eject` immer beides.
|
|
|
|
Gleichlautend in worker/ripping.wirf_disc_aus — es gibt kein geteiltes Paket
|
|
zwischen den Containern; dort steht die ausführliche Herleitung.
|
|
"""
|
|
import time
|
|
|
|
fd = os.open(device_path, os.O_RDONLY | os.O_NONBLOCK)
|
|
try:
|
|
try:
|
|
ioctl(fd, CDROM_LOCKDOOR, 0)
|
|
except OSError:
|
|
pass # nicht verriegelt oder ioctl unbekannt — Auswurf trotzdem versuchen
|
|
ioctl(fd, CDROMEJECT, 0)
|
|
# Nachsehen statt hoffen. „Kein Datenträger" zählt mit: ein
|
|
# Slot-Laufwerk hat keine Schublade.
|
|
for _ in range(AUSWURF_WARTEN_SEKUNDEN):
|
|
if ioctl(fd, CDROM_DRIVE_STATUS, 0) in (CDS_TRAY_OPEN, CDS_NO_DISC):
|
|
return
|
|
time.sleep(1)
|
|
raise OSError(
|
|
"Das Laufwerk hat den Auswurf angenommen, die Disc ist aber noch "
|
|
"drin. Blockiert etwas die Schublade, oder läuft noch ein Zugriff?"
|
|
)
|
|
finally:
|
|
os.close(fd)
|
|
|
|
|
|
def list_optical_devices() -> list:
|
|
"""Alle optischen Laufwerke, die der Container sieht (devices: in compose)."""
|
|
return sorted(glob.glob("/dev/sr[0-9]*"))
|
|
|
|
|
|
def read_sys_attr(device_name: str, attr: str) -> str:
|
|
pfad = f"/sys/class/block/{device_name}/device/{attr}"
|
|
try:
|
|
with open(pfad, encoding="ascii", errors="replace") as f:
|
|
return f.read().strip()
|
|
except OSError:
|
|
return ""
|
|
|
|
|
|
def device_info(device_path: str) -> dict:
|
|
"""Baut den Geräte-Eintrag fürs UI: Name aus /sys, Disc-Status per ioctl."""
|
|
name = os.path.basename(device_path)
|
|
vendor = read_sys_attr(name, "vendor")
|
|
model = read_sys_attr(name, "model")
|
|
|
|
try:
|
|
status_code = drive_status(device_path)
|
|
except OSError:
|
|
status_code = -1
|
|
|
|
disc_type = "unknown"
|
|
status = "empty"
|
|
if status_code == CDS_DISC_OK:
|
|
status = "ready"
|
|
try:
|
|
disc_type = classify(disc_status(device_path), disc_size_bytes(device_path))
|
|
except OSError:
|
|
disc_type = "unknown"
|
|
|
|
return {
|
|
"id": name,
|
|
"name": " ".join(teil for teil in (vendor, model) if teil) or f"Laufwerk {name}",
|
|
"type": disc_type,
|
|
"path": device_path,
|
|
"status": status,
|
|
"model": model,
|
|
"serial": read_sys_attr(name, "wwid"),
|
|
}
|