nodejs/node

Re-enable test-ubsan.yml

Fermée

#52 753 ouverte le 29 avr. 2024

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)JavaScript (35 535 forks)batch import
help wantedmetastale

Métriques du dépôt

Stars
 (117 218 étoiles)
Métriques de merge PR
 (Merge moyen 18j 17h) (219 PRs mergées en 30 j)

Description

During the latest Node.js major release (https://github.com/nodejs/node/pull/52505) we have had to disable test-ubsan.yml due to failing tests on a recent V8 update (https://github.com/nodejs/node/pull/52560).

I have tried to make it work, but due to the short time frame I had to fix it, we have decided to simply disable it for now.

Some insights:

  • It seems clang provides a better output and some docs we could use in the suppressions file.
    • Changing from g++ to clang for UBSAN will require changing all the suppression file

Guide contributeur

Direction de recherche
Regardez le commit de mise à jour de V8 (nodejs/node#52560) pour identifier les tests échoués. Exécutez test ubsan localement avec clang pour obtenir de meilleurs messages d'erreur. Mettez à jour le fichier de suppression en conséquence.
Stack technique
javascriptcppnodejsgithub actions
Domaine
ci cdtestingbackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
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
C++GitNode.js build systemUBSAN
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40