fix(windows): Restzeit blieb ewig „wird gemessen" — zwei Docker-Annahmen
Ampel / ampel (push) Successful in 1m32s
Ampel / ampel (push) Successful in 1m32s
Commander: „Über den kompletten vorgang steht dort ‚Restzeit wird gemessen'
aber messung wird nicht abgeschlossen. Das heißt man hat kein ETA"
Zwei Ursachen, beide unabhaengig, beide toedlich fuer sich allein.
## 1. Die Messreihe lag nur in Redis
`eta.py` schrieb sie ausschliesslich in den Cache — mit der Begruendung im
Modul-Kopf: „Der Cache (Redis) ist schon da". Im Container stimmt das. Auf
einem Windows-PC gibt es kein Redis: `cache_get` gab bei JEDEM Aufruf None
zurueck, `beobachtung_hinzufuegen` legte also jedes Mal eine frische Reihe mit
EINEM Punkt an — und `restzeit_sekunden` braucht `MINDEST_PUNKTE = 2`.
Jetzt wird in beide Ablagen geschrieben: in den Cache, wo es einen gibt (er
ueberlebt einen API-Neustart), und in ein Woerterbuch im Prozess. Das ist ein
paar Zahlen gross, gilt nur fuer die Dauer eines Jobs, und im eigenstaendigen
Betrieb gibt es ohnehin nur diesen einen Prozess. Alte Reihen werden nach
einem Tag weggeraeumt.
## 2. Der Ereignisstrom rechnete die Restzeit gar nicht
Die Rechnung stand nur in `/jobs`. Der SSE-Schnappschuss baute seine Jobs mit
dem nackten `_job_row_to_model` — also ohne Restzeit. **Seit der Umstellung
auf den Ereignisstrom (V2-3) liest die Oberflaeche aber genau diesen
Schnappschuss und nicht mehr `/jobs`.** Die Restzeit wurde also brav berechnet
und niemandem gezeigt.
Beides jetzt in `jobs_fuer_ui()` — dieselbe Lehre wie bei
`laufwerke_mit_disc` heute frueh: Eine Auskunft in zwei Fassungen ist eine
Fassung zu viel.
Nebenbei: Unlesbare Einstellungen duerfen die Jobliste nicht umwerfen. Seit
sie auch den Schnappschuss baut, haengt daran die ganze Oberflaeche — zwei
Snapshot-Tests wurden davon prompt rot.
## Beweis
Job in der Datenbank, Fortschritt 10 % -> 25 % ueber 130 s:
nach 1. Messpunkt : eta_text='' (richtig, eine Messung reicht nicht)
nach 2. Messpunkt : 674 s, „noch ca. 11 min"
877 Tests gruen, ruff sauber.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
aeb3128a85
commit
548742e771
@@ -157,3 +157,73 @@ def test_fertige_und_wartende_jobs_bekommen_keine_eta():
|
||||
speicher.get, lambda k, v, expire=None: speicher.__setitem__(k, v))
|
||||
assert ergebnis == {"sekunden": -1, "text": ""}
|
||||
assert speicher == {} # nichts geschrieben
|
||||
|
||||
|
||||
# ── Ohne Redis muss die Schaetzung trotzdem zustande kommen ─────────────
|
||||
#
|
||||
# Commander 29.08.2026: „Über den kompletten vorgang steht dort ‚Restzeit wird
|
||||
# gemessen' aber messung wird nicht abgeschlossen. Das heißt man hat kein ETA"
|
||||
#
|
||||
# Die Messreihe lag nur im Cache, und der Modul-Kopf sagte „Der Cache (Redis)
|
||||
# ist schon da". Im Container stimmt das. Auf einem Windows-PC gab `cache_get`
|
||||
# bei JEDEM Aufruf None zurueck — also jedes Mal eine frische Reihe mit EINEM
|
||||
# Punkt, und `restzeit_sekunden` braucht zwei.
|
||||
|
||||
|
||||
def _ohne_cache():
|
||||
"""Ein Cache, der nichts behaelt — genau wie Redis, das es nicht gibt."""
|
||||
return (lambda k: None), (lambda k, v, expire=None: False)
|
||||
|
||||
|
||||
def test_ohne_cache_kommt_trotzdem_eine_schaetzung():
|
||||
"""DER Fall des Commanders."""
|
||||
import eta
|
||||
|
||||
eta._REIHEN.clear()
|
||||
lesen, schreiben = _ohne_cache()
|
||||
# Zwei Messpunkte, 120 s auseinander, 10 % Fortschritt.
|
||||
eta.aktualisiere_und_schaetze("j1", "running", 10, 1000.0, lesen, schreiben)
|
||||
ergebnis = eta.aktualisiere_und_schaetze("j1", "running", 20, 1120.0, lesen, schreiben)
|
||||
|
||||
assert ergebnis["sekunden"] > 0, "ohne Redis kam nie eine Zahl heraus"
|
||||
assert ergebnis["text"], "und damit stand dauerhaft „wird gemessen"
|
||||
|
||||
|
||||
def test_der_cache_hat_weiterhin_vorrang():
|
||||
"""Wo es Redis gibt, bleibt Redis die Quelle — es ueberlebt einen
|
||||
API-Neustart, das Woerterbuch im Prozess nicht."""
|
||||
import eta
|
||||
|
||||
eta._REIHEN.clear()
|
||||
aus_dem_cache = {"status": "running", "punkte": [[500.0, 5], [560.0, 15]]}
|
||||
ergebnis = eta.aktualisiere_und_schaetze(
|
||||
"j2", "running", 25, 620.0,
|
||||
lambda k: aus_dem_cache, lambda k, v, expire=None: True)
|
||||
# Drei Punkte, 120 s fuer 20 % -> die Reihe aus dem Cache wurde benutzt.
|
||||
assert ergebnis["sekunden"] > 0
|
||||
|
||||
|
||||
def test_ein_statuswechsel_beginnt_auch_ohne_cache_neu():
|
||||
"""Rip und Kompression haben nichts miteinander zu tun."""
|
||||
import eta
|
||||
|
||||
eta._REIHEN.clear()
|
||||
lesen, schreiben = _ohne_cache()
|
||||
eta.aktualisiere_und_schaetze("j3", "running", 50, 1000.0, lesen, schreiben)
|
||||
eta.aktualisiere_und_schaetze("j3", "running", 90, 1100.0, lesen, schreiben)
|
||||
neu = eta.aktualisiere_und_schaetze("j3", "transcoding", 5, 1110.0, lesen, schreiben)
|
||||
assert neu["sekunden"] == -1, "nach dem Wechsel gibt es noch keine Aussage"
|
||||
|
||||
|
||||
def test_alte_reihen_wachsen_nicht_unbegrenzt():
|
||||
"""Wer einen Vorrat anlegt, raeumt ihn auch weg (AGENTS.md)."""
|
||||
import eta
|
||||
|
||||
eta._REIHEN.clear()
|
||||
lesen, schreiben = _ohne_cache()
|
||||
eta.aktualisiere_und_schaetze("alt", "running", 10, 0.0, lesen, schreiben)
|
||||
# Einen Tag spaeter ein anderer Job -> der alte fliegt raus.
|
||||
eta.aktualisiere_und_schaetze("neu", "running", 10,
|
||||
eta.REIHE_HALTBAR_SEKUNDEN + 10.0, lesen, schreiben)
|
||||
assert eta.schluessel("alt") not in eta._REIHEN
|
||||
assert eta.schluessel("neu") in eta._REIHEN
|
||||
|
||||
Reference in New Issue
Block a user