ipfs/kubo

Optimize sharness tests

Open

#4.961 geöffnet am 22. Apr. 2018

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (2.725 Forks)batch import
help wantedkind/test

Repository-Metriken

Stars
 (13.906 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 4T 2h) (28 gemergte PRs in 30 T)

Beschreibung

Some sharness tests take way more time to complete than they have to.

Results from https://ci.ipfs.team/job/IPFS/job/go-ipfs/job/PR-4959/7/testReport/sharnessLinux/ (sort by duration) / https://github.com/ipfs/go-ipfs/pull/4959

  • - t0181-private-network / 12 - try connecting node in public network with priv networks - should timeout way sooner
  • - t0130-multinode - adds a tree of 5k files multiple times, might be a bit overkill, at least of something that isn't a benchmark
  • - t0250-files-api is huuuuge, would be good to split into some separate tests to make use of make parallelism
  • - t0260-sharding operates on quite large directories, not sure if we can do much about it
  • - t0040-add-and-cat does some stuff on large files, but majority of the time spent there seems to be spent in the ton of tests there, might be worth to split too

Contributor Guide

Research-Richtung
Profileren Sie jeden im Issue aufgeführten Test, identifizieren Sie die Hauptzeitfresser und schlagen Sie Optimierungen vor, wie die Reduzierung der Dateigröße, das Verkürzen von Timeouts oder das Aufteilen großer Testdateien in kleinere parallelisierbare Tests.
Tech Stack
goshell
Domain
testingperformance
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitGoShell scripting
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70