Skip to content

Commit

Permalink
examples: fixed fault-injection delay demo (envoyproxy#11715)
Browse files Browse the repository at this point in the history
fault injection example did not work for delay option. After gdb debugging it was discovered that fault injection config requires delay part to be present. Without delay part Envoy assumes that Delay is disabled and does not scan RTDS for delay config updates.

Risk Level: Low:
Testing: Did manual testing as described in https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/fault_injection
Docs Changes: No
Release Notes: No
Fixes: envoyproxy#11095
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
  • Loading branch information
cpakulski authored and songhu committed Jun 25, 2020
1 parent cc38d60 commit aec0c96
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
3 changes: 3 additions & 0 deletions examples/fault-injection/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ services:
ports:
- 9211:9211
- 9901:9901
# Run Envoy as root to grant access to /dev/stdout
environment:
ENVOY_UID: 0
backend:
image: kennethreitz/httpbin@sha256:2c7abc4803080c22928265744410173b6fea3b898872c01c5fd0f0f9df4a59fb
networks:
Expand Down
5 changes: 5 additions & 0 deletions examples/fault-injection/envoy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,11 @@ static_resources:
percentage:
numerator: 0
denominator: HUNDRED
delay:
fixed_delay: 3s
percentage:
numerator: 0
denominator: HUNDRED
- name: envoy.filters.http.router
typed_config: {}
clusters:
Expand Down

0 comments on commit aec0c96

Please sign in to comment.