nodejs/node

Re-enable test-ubsan.yml

Open

#52.753 aberto em 29 de abr. de 2024

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)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

Guia do colaborador

Pilha de tecnologia
javascriptcpp
Domain
ci cdtesting
Tipo Issue
test
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 days
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
Node.js build systemUBSANC++ compiler basicsCI configuration
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30
Direção de pesquisa
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.