micrometer-metrics/micrometer

StatsdMeterRegistry doesn't renew resolved ip address on DNS change

Open

#1 252 ouverte le 27 févr. 2019

Voir sur GitHub
 (15 commentaires) (2 réactions) (0 assignés)Java (935 forks)batch import
enhancementhelp wantedregistry: statsd

Métriques du dépôt

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

Description

Hello everyone.

I've faced with the problem that StatsdMeterRegistry doesn't aware of DNS changes after it was start()ed. As a result, metrics aren't delivered to Telegraf and there is no warning or so in logs.

How to reproduce:

  1. Create record in /etc/hosts file, e.g.: 127.0.0.1 telegraf.kube-system.svc.cluster.local.
  2. Start application with StatsdMeterRegistry enabled and use host described above.
  3. Change record in /etc/hosts file to another ip (127.0.0.2 for example)

Actual result: Using wireshark/tcpdump you can see that all metrics are being sent to 127.0.0.1 even after step 3.

Expected result: StatsdMeterRegistry respect DNS name change.

Can be reproduced on real network/DNS server. Security manager disabled. Micrometer version: 1.1.3, reproducible on 1.0.7 as well.

Guide contributeur

Direction de recherche
Examinez comment StatsdMeterRegistry résout le nom d'hôte initialement et s'il utilise un cache DNS qui ne se rafraîchit pas. Vérifiez s'il existe un mécanisme pour re résoudre périodiquement l'adresse ou si l'InetSocketAddress résolue est stockée de manière permanente.
Stack technique
java
Domaine
observability
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaNetworking
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60