fault: remove support for integer percentages (#4626)

This commit removes support for the previously deprecated integer
percentages in the fault subsystem.

Signed-off-by: Venil Noronha <veniln@vmware.com>

Mirrored from https://github.com/envoyproxy/envoy @ 7ccf67db45db8c4e9bc406488c8f1f2a8566f506
pull/620/head
data-plane-api(CircleCI) 6 years ago
parent 94eb5c5c4d
commit 182864293f
  1. 8
      envoy/config/filter/fault/v2/fault.proto
  2. 20
      envoy/config/filter/http/fault/v2/fault.proto

@ -24,13 +24,7 @@ message FaultDelay {
// supported.
FaultDelayType type = 1 [(validate.rules).enum.defined_only = true];
// An integer between 0-100 indicating the percentage of operations/connection requests
// on which the delay will be injected.
//
// .. attention::
//
// Use of integer `percent` value is deprecated. Use fractional `percentage` field instead.
uint32 percent = 2 [(validate.rules).uint32.lte = 100, deprecated = true];
reserved 2;
oneof fault_delay_secifier {
option (validate.required) = true;

@ -13,13 +13,7 @@ import "validate/validate.proto";
// Fault Injection :ref:`configuration overview <config_http_filters_fault_injection>`.
message FaultAbort {
// An integer between 0-100 indicating the percentage of requests/operations/connections
// that will be aborted with the error code provided.
//
// .. attention::
//
// Use of integer `percent` value is deprecated. Use fractional `percentage` field instead.
uint32 percent = 1 [(validate.rules).uint32.lte = 100, deprecated = true];
reserved 1;
oneof error_type {
option (validate.required) = true;
@ -50,12 +44,12 @@ message HTTPFault {
// Specifies a set of headers that the filter should match on. The fault
// injection filter can be applied selectively to requests that match a set of
// headers specified in the fault filter config. The chances of actual fault
// injection further depend on the value of the :ref:`percent
// <envoy_api_field_config.filter.http.fault.v2.FaultAbort.percent>` field. The filter will
// check the request's headers against all the specified headers in the filter
// config. A match will happen if all the headers in the config are present in
// the request with the same values (or based on presence if the *value* field
// is not in the config).
// injection further depend on the value of the :ref:`percentage
// <envoy_api_field_config.filter.http.fault.v2.FaultAbort.percentage>` field.
// The filter will check the request's headers against all the specified
// headers in the filter config. A match will happen if all the headers in the
// config are present in the request with the same values (or based on
// presence if the *value* field is not in the config).
repeated envoy.api.v2.route.HeaderMatcher headers = 4;
// Faults are injected for the specified list of downstream hosts. If this

Loading…
Cancel
Save