twitter/finagle

Add com.twitter.finagle.zipkin.core.Endpoint.ipv6

Open

#526 aberto em 13 de jul. de 2016

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)Scala (1.435 forks)batch import
good first issue

Métricas do repositório

Stars
 (8.864 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

We currently support logging of only IPv4 addresses. Starting with Zipkin 1.4, endpoints can omit IPv4 (by setting Endpoint.ipv4 to 0), and optionally log Endpoint.ipv6 as the raw 16byte address. https://github.com/openzipkin/zipkin-api/blob/master/thrift/zipkinCore.thrift#L276

It looks like the most pivotal change is updating com.twitter.finagle.zipkin.core.Endpoint.ipv6

cc @sveinnfannar

Guia do colaborador

Direção de pesquisa
A questão é adicionar suporte a IPv6 à classe Endpoint na integração Zipkin do Finagle. A principal alteração está na classe com.twitter.finagle.zipkin.core.Endpoint, que atualmente suporta apenas IPv4. A especificação da API Zipkin (https://github.com/openzipkin/zipkin api/blob/master/thrift/zipkinCore.thrift#L276) mostra que os endpoints agora podem opcionalmente incluir um campo ipv6 como um endereço de 16 bytes. Para implementar isso, você precisará modificar a classe Scala Endpoint para adicionar um campo ipv6, atualizar sua serialização/desserialização para lidar com o novo campo e garantir compatibilidade com versões anteriores para endpoints apenas IPv4. A questão também menciona um contribuidor (@sveinnfannar) que pode ter mais contexto. Revise a base de código existente para entender a implementação atual do Endpoint e a vinculação Thrift, depois adicione o suporte a ipv6 de acordo.
Pilha de tecnologia
scala
Domain
backendobservability
Tipo Issue
Funcionalidade
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-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Scala knowledgeFamiliarity with ThriftUnderstanding of Zipkin endpoint
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40