envoyproxy/envoy

`schema_validator_tool` should not be `testonly`

Open

#26.390 aberto em 28 de mar. de 2023

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/releasearea/schema_validatorbughelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

Currently the schema_validator_tool is built using envoy_cc_test which marks it testonly and it in turn depends on a bunch of test libs that are likewise testonly

this should not be the case given that this tool is published, and means that it cannot be depended upon as a build target by other non-test build targets

i made an inexpert attempt at doing this, but found myself copying out increasingly large chunks of the test framework. I gave up when i needed to start copying out the test thread code

Guia do colaborador

Direção de pesquisa
Modifique as regras de build para schema validator tool para não usar envoy cc test, mas sim uma regra não testonly, refatorando as dependências para evitar dependências testonly.
Pilha de tecnologia
cpp
Domain
backendbuild system
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Bazel
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40