micrometer-metrics/micrometer

DNS, SSL, and TCP metrics for Apache HttpClient

Fermée

#1 617 ouverte le 1 oct. 2019

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (1 personne assignée)Java (1 134 forks)batch import
enhancementhelp wantedinstrumentationmodule: micrometer-core

Métriques du dépôt

Stars
 (4 882 étoiles)
Métriques de merge PR
 (Merge moyen 1j 21h) (100 PRs mergées en 30 j)

Description

Add metrics for:

  1. DNS resolution time (inject custom DnsResolver, delegating to the user-provided resolver, and timing).
  2. SSL handshake time (inject SSLConnectionSocketFactory, delegating).
  3. TCP connect time (also in custom SSLConnectionSocketFactory).

Apply these on custom PoolingHttpClientConnectionManager.

Feedback suggests that DNS resolution/SSL handshake can go up dramatically in noisy neighbor situations.

Originally posted by @jkschneider in https://github.com/micrometer-metrics/micrometer/issues/533#issuecomment-446713248

Guide contributeur

Direction de recherche
Investigate injecting custom DnsResolver and SSLConnectionSocketFactory into PoolingHttpClientConnectionManager. Review Apache HttpClient documentation for hooks. Examine existing Micrometer instrumentation patterns for timing using Timer. Consider the noisy neighbor scenario mentioned in the issue. Look at the original comment by jkschneider for context.
Stack technique
java
Domaine
observability
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaApache HttpClientMicrometerSSL/TLS basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35