valkey-io/valkey

Updated CI testing to improve catching tests early.

Geschlossen

#625 geöffnet am 11.06.2024

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (1.138 Forks)batch import
help wanted

Repository-Metriken

Stars
 (25.779 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 17T 7h) (74 gemergte PRs in 30 T)

Beschreibung

We have had a recent string of issues with regressions being introduced into the daily rest run, that weren't being caught in the daily. After reviewing the daily failures, me and some other contributors came up with the following recommendation.

  1. Add in a second test suite that uses an allocator that is not jemalloc that has "no malloc size" enabled.
  2. Add a new CI target tests with TLS, but skip all of the "slow" tests. Included here is that we should add a slow tag to the payload fuzzer as they take a significant amount of time with TLS.
  3. Add a build target that actually builds the legacy unit test, to make sure they compile correctly.

We also discussed maybe adding in the sentinel tests as part of the build, however it wasn't clear if they regularly caught issues.

Contributor Guide

Research-Richtung
Überprüfe die bestehende CI Konfiguration und Teststruktur. Setze die drei Empfehlungen um: (1) füge eine zweite Testsuite mit einem Nicht jemalloc Allokator mit aktiviertem 'no malloc size' hinzu, (2) füge ein neues CI Ziel für Tests mit TLS hinzu, überspringe langsame Tests (füge ein Slow Tag zum Payload Fuzzer hinzu), (3) füge ein Build Ziel hinzu, das den Legacy Unit Test kompiliert, um sicherzustellen, dass sie korrekt kompilieren.
Tech Stack
c
Domain
backenddatabasetesting qa
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitCCI/CD
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75