ipfs/kubo

Optimize sharness tests

Open

#4 961 ouverte le 22 avr. 2018

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Go (2 725 forks)batch import
help wantedkind/test

Métriques du dépôt

Stars
 (13 906 stars)
Métriques de merge PR
 (Merge moyen 4j 2h) (28 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Profitez chaque test listé dans le problème, identifiez les principaux goulets d'étranglement temporels et proposez des optimisations telles que la réduction de la taille des fichiers, le raccourcissement des délais d'attente ou la division des grands fichiers de test en tests plus petits parallélisables.
Stack technique
goshell
Domaine
testingperformance
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitGoShell scripting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70