nodejs/node

Re-enable test-ubsan.yml

Fechada

#52.753 aberto em 29 de abr. de 2024

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)JavaScript (35.535 forks)batch import
help wantedmetastale

Métricas do repositório

Stars
 (117.218 estrelas)
Métricas de merge de PR
 (Mesclagem média 18d 17h) (219 fundiu PRs em 30d)

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

Direção de pesquisa
Veja o commit de atualização do V8 (nodejs/node#52560) para identificar os testes com falha. Execute test ubsan localmente com clang para obter melhores mensagens de erro. Atualize o arquivo de supressão de acordo.
Pilha de tecnologia
javascriptcppnodejsgithub actions
Domain
ci cdtestingbackend
Tipo Issue
Bug
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 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++GitNode.js build systemUBSAN
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40