Outlier detection can eject one too many endpoints

Ouverte Adaptée aux débutants
#13,164 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
82/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
grpc, java

Piste de recherche

Lisez util/src/main/java/io/grpc/util/OutlierDetectionLoadBalancer.java, en particulier les deux emplacements liés dans l’issue (autour des lignes 828–833 et 901–909). Vérifiez comment chacun calcule le pourcentage d’éviction et comment la limite d’éviction est appliquée. Le travail est terminé lorsque les deux chemins tiennent compte du pourcentage après une éviction hypothétique, afin que les évictions ne puissent pas dépasser le maximum configuré.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

With https://github.com/grpc/proposal/pull/574, A50 was updated to match the proper Envoy behavior:

Ejection never takes the ejected percentage above this value, so a value of 0 disables ejection.

Java does allow an ejection to take us over the configured percentage.

https://github.com/grpc/grpc-java/blob/4483d920e85fbdecf890b1fb5890aef6599159f2/util/src/main/java/io/grpc/util/OutlierDetectionLoadBalancer.java#L828-L833

and

https://github.com/grpc/grpc-java/blob/4483d920e85fbdecf890b1fb5890aef6599159f2/util/src/main/java/io/grpc/util/OutlierDetectionLoadBalancer.java#L901-L909

These both should change to consider the percentage after a hypothetical ejection was to occur. E.g. rework ejectionPercentage to something like nextEjectionPercentage instead.

Langage dominant
Java
Étoiles
12.1k
Forks
4k
Merge moyen
2 j 3 h
PR mergées (30 j)
41

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de grpc/grpc-java

Toutes les issues de grpc/grpc-java

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.