aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/proto
diff options
context:
space:
mode:
authorGravatar Juanli Shen <juanlishen@google.com>2018-07-13 19:52:59 -0700
committerGravatar Juanli Shen <juanlishen@google.com>2018-07-13 19:52:59 -0700
commitbe40b0d3a8cf2e37c80b2c248111051fa8bdf7bc (patch)
tree79384a0045d5bbc550666f047d088f9582755086 /src/proto
parentf9f5c67aff91e4ad26371b0a2482a5011ab45226 (diff)
Add server load reporting service
Diffstat (limited to 'src/proto')
-rw-r--r--src/proto/grpc/lb/v1/load_reporter.proto4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/proto/grpc/lb/v1/load_reporter.proto b/src/proto/grpc/lb/v1/load_reporter.proto
index c2e4f23f6e..d57a37fed7 100644
--- a/src/proto/grpc/lb/v1/load_reporter.proto
+++ b/src/proto/grpc/lb/v1/load_reporter.proto
@@ -114,6 +114,10 @@ message Load {
// num_calls_in_progress is the only valid entry. If in_progress_report is not
// set, num_calls_in_progress will be ignored. If in_progress_report is set,
// fields other than num_calls_in_progress and orphaned_load will be ignored.
+ // TODO(juanlishen): A Load is either an in_progress_report or not. We should
+ // make this explicit in hierarchy. From the log, I see in_progress_report_
+ // has a random num_calls_in_progress_ when not set, which might lead to bug
+ // when the balancer process the load report.
oneof in_progress_report {
// The number of calls in progress (instantaneously) per load balancer id.
int64 num_calls_in_progress = 5;