nodejs/node

Re-enable test-ubsan.yml

Open

#52 753 ouverte le 29 avr. 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (117 218 stars) (35 535 forks)batch import
help wantedmetastale

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

Stack technique
javascriptcpp
Domaine
ci cdtesting
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.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Node.js build systemUBSANC++ compiler basicsCI configuration
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Examine the disabled test ubsan.yml and the suppression files referenced in the insights. Investigate the V8 update that caused failures (PR #52560) and understand why g++ was used. Look into switching to clang for better output and updating the suppression file accordingly. Consult the linked PR #52505 for context.