Skip to content

Commit

Permalink
api: drop_overload is not yet implemented in Envoy. (#11104)
Browse files Browse the repository at this point in the history
This was added before we started annotating fields missing
implementations. Fixed with a [#not-implemented-hide:] annotations for
now, the plan is to move things like this to
#11085 when it lands.

Signed-off-by: Harvey Tuch <htuch@google.com>
  • Loading branch information
htuch authored May 7, 2020
1 parent 10e7c9d commit a250a87
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions api/envoy/api/v2/endpoint.proto
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ message ClusterLoadAssignment {
// Load balancing policy settings.
// [#next-free-field: 6]
message Policy {
// [#not-implemented-hide:]
message DropOverload {
// Identifier for the policy specifying the drop.
string category = 1 [(validate.rules).string = {min_bytes: 1}];
Expand Down Expand Up @@ -65,6 +66,7 @@ message ClusterLoadAssignment {
// "throttle"_drop = 60%
// "lb"_drop = 20% // 50% of the remaining 'actual' load, which is 40%.
// actual_outgoing_load = 20% // remaining after applying all categories.
// [#not-implemented-hide:]
repeated DropOverload drop_overloads = 2;

// Priority levels and localities are considered overprovisioned with this
Expand Down
2 changes: 2 additions & 0 deletions api/envoy/config/endpoint/v3/endpoint.proto
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ message ClusterLoadAssignment {
option (udpa.annotations.versioning).previous_message_type =
"envoy.api.v2.ClusterLoadAssignment.Policy";

// [#not-implemented-hide:]
message DropOverload {
option (udpa.annotations.versioning).previous_message_type =
"envoy.api.v2.ClusterLoadAssignment.Policy.DropOverload";
Expand Down Expand Up @@ -74,6 +75,7 @@ message ClusterLoadAssignment {
// "throttle"_drop = 60%
// "lb"_drop = 20% // 50% of the remaining 'actual' load, which is 40%.
// actual_outgoing_load = 20% // remaining after applying all categories.
// [#not-implemented-hide:]
repeated DropOverload drop_overloads = 2;

// Priority levels and localities are considered overprovisioned with this
Expand Down
2 changes: 2 additions & 0 deletions generated_api_shadow/envoy/api/v2/endpoint.proto

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions generated_api_shadow/envoy/config/endpoint/v3/endpoint.proto

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit a250a87

Please sign in to comment.