ocaml/dune

Flush output buffers on SIGTERM

Offen

#3.882 geöffnet am 21.10.2020

Auf GitHub ansehenContributor Guide
 (12 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)OCaml (478 Forks)github user discovery
good first issue

Repository-Metriken

Stars
 (1.882 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 20h) (345 gemergte PRs in 30 T)

Beschreibung

We're currently using dune runtest in OCaml-CI as the standard way of running tests for a package. If some of the tests for a package get stuck in a deadlock, the CI job will kill the process after 60 minutes. Unfortunately, this means that the logs of the stalled test run are lost.

We could switch to using dune runtest --no-buffer in the CI, but this creates less readable (interleaved) test output. One possible alternative would be for Dune to flush its output buffers before responding to SIGTERM (probably keeping the old behaviour for SIGKILL). What do you think?

Contributor Guide

Research-Richtung
Untersuche, wie Dune derzeit die Ausgabepufferung und Signalbehandlung (SIGTERM) handhabt. Sieh dir die Hauptschleife und die Signalhandler an, um zu verstehen, wo du ein Leeren der stdout/stderr Puffer vor dem Beenden hinzufügen kannst. Stelle sicher, dass SIGKILL nicht geleert wird, wie gewünscht.
Tech Stack
Keine
Domain
build systemdevopsdeveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
OCamlsignal handling basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50