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>
Signed-off-by: yashwant121 <yadavyashwant36@gmail.com>
  • Loading branch information
cpakulski authored and yashwant121 committed Jul 24, 2020
1 parent 222db8c commit f81ceeb
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 f81ceeb

Please sign in to comment.