Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

XdsClientImpl inconsistently includes nodeId

Abierto
  • ¿Algo no está bien?
#12,750 0 comentarios 0 reacciones 1 asignado Ver en GitHub

Los mantenedores suelen responder en 2 días

@shivaspeaks ya está trabajando en esto.

Desde el 9/4/2026.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

enhancement

When creating https://github.com/grpc/grpc-java/pull/12749 I noticed that the error generated by XdsDependencyManager is:

Status{code=UNAVAILABLE, description=Error retrieving LDS resource the-service-name nodeID: node-id: NOT_FOUND: expected nodeID: node-id, cause=java.lang.IllegalArgumentException: a random exception...}

"nodeID: node-id" is repeated twice. XdsClientImpl adds it once, and XdsDependencyManager adds it again. XdsDependencyManager shouldn't be adding the node id from errors that come from XdsClient. But if you look at CdsLoadBalancer2Test from my PR, you'll see that "Timed out waiting for resource" errors don't have node id duplicated.

XdsClientImpl includes node id in all results originating from ResourceSubscriber.onError(), but it doesn't from onAbsent() and addWatcher(). Apparently it also forgets the absent error details in addWatcher(). We need to fix onAbsent() and errorDescription, and probably save the absent error in lastError for later use in addWatcher(). At that point we can remove the node id from XdsDependencyManager (when it gets the error from xds client; if it generates the error itself, it may still need to include the node id).

Lenguaje dominante
Java
Estrellas
12.1k
Forks
4k
Merge medio
2 d 3 h
PR fusionados (30 d)
30

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de grpc/grpc-java

Todos los issues de grpc/grpc-java

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.