twitter/finagle

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

Open

#526 geöffnet am 13. Juli 2016

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (1.435 Forks)batch import
good first issue

Repository-Metriken

Stars
 (8.864 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem besteht darin, IPv6 Unterstützung für die Endpoint Klasse in der Zipkin Integration von Finagle hinzuzufügen. Die Hauptänderung betrifft die Klasse com.twitter.finagle.zipkin.core.Endpoint, die derzeit nur IPv4 unterstützt. Die Zipkin API Spezifikation (https://github.com/openzipkin/zipkin api/blob/master/thrift/zipkinCore.thrift#L276) zeigt, dass Endpunkte jetzt optional ein ipv6 Feld als 16 Byte Adresse enthalten können. Zur Implementierung müssen Sie die Scala Klasse Endpoint ändern, um ein ipv6 Feld hinzuzufügen, ihre Serialisierung/Deserialisierung für das neue Feld aktualisieren und die Abwärtskompatibilität mit reinen IPv4 Endpunkten sicherstellen. Das Problem erwähnt auch einen Mitwirkenden (@sveinnfannar), der weiteren Kontext liefern könnte. Überprüfen Sie die vorhandene Codebasis, um die aktuelle Endpoint Implementierung und die Thrift Bindung zu verstehen, und fügen Sie dann entsprechend die ipv6 Unterstützung hinzu.
Tech Stack
scala
Domain
backendobservability
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Scala knowledgeFamiliarity with ThriftUnderstanding of Zipkin endpoint
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40