envoyproxy/envoy

server: Remove --restart-epoch command-line option because #9888 computes it automatically

Open

#10.139 aberto em 22 de fev. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)C++ (5.373 forks)batch import
area/statshelp wantedtech debt

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, a deployment using hot-restart specifies a --restart-epoch epoch_number as a command-line argument to the server. Once #9888 is merged, server.hot_restart_generation will be computed automatically, eliminating the need for scripting to bump the epoch manually.

We should probably not remove that command-line option, however, until sometime after #4980 is merged, because there will be a discontinuity in the server.hot_restart_generation stat.

Then there probably should be a merger of the two stats, eventually dropping one.

Guia do colaborador

Direção de pesquisa
Inspecione o código de análise de opções de linha de comando no diretório do servidor, entenda como o epoch de reinicialização é usado atualmente e coordene a remoção com as mesclagens de #9888 e #4980.
Pilha de tecnologia
cpp
Domain
backendinfrastructure
Tipo Issue
Refatoração
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.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C++GitBazel
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40