Skip to content

Commit

Permalink
bump envoy to v1.17.0
Browse files Browse the repository at this point in the history
  • Loading branch information
tomxor authored and stanley-cheung committed Jan 23, 2021
1 parent 1580f3a commit f960b8f
Show file tree
Hide file tree
Showing 7 changed files with 17 additions and 14 deletions.
2 changes: 1 addition & 1 deletion net/grpc/gateway/docker/envoy/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.

FROM envoyproxy/envoy:v1.16.1
FROM envoyproxy/envoy:v1.17.0

COPY net/grpc/gateway/examples/echo/envoy.yaml /etc/envoy/envoy.yaml

Expand Down
7 changes: 4 additions & 3 deletions net/grpc/gateway/examples/echo/envoy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ static_resources:
- filters:
- name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
codec_type: auto
stat_prefix: ingress_http
route_config:
Expand All @@ -24,7 +24,8 @@ static_resources:
- match: { prefix: "/" }
route:
cluster: echo_service
max_grpc_timeout: 0s
max_stream_duration:
grpc_timeout_header_max: 0s
cors:
allow_origin_string_match:
- prefix: "*"
Expand All @@ -50,4 +51,4 @@ static_resources:
address:
socket_address:
address: node-server
port_value: 9090
port_value: 9090
4 changes: 2 additions & 2 deletions net/grpc/gateway/examples/helloworld/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -294,15 +294,15 @@ run the 3 processes all in the background.

```sh
$ docker run -d -v "$(pwd)"/envoy.yaml:/etc/envoy/envoy.yaml:ro \
--network=host envoyproxy/envoy:v1.16.1
--network=host envoyproxy/envoy:v1.17.0
```

> NOTE: As per [this issue](https://github.com/grpc/grpc-web/issues/436):
> if you are running Docker on Mac/Windows, remove the `--network=host` option:
>
> ```sh
> $ docker run -d -v "$(pwd)"/envoy.yaml:/etc/envoy/envoy.yaml:ro \
> -p 8080:8080 -p 9901:9901 envoyproxy/envoy:v1.16.1
> -p 8080:8080 -p 9901:9901 envoyproxy/envoy:v1.17.0
> ```

3. Run the simple Web Server. This hosts the static file `index.html` and
Expand Down
7 changes: 4 additions & 3 deletions net/grpc/gateway/examples/helloworld/envoy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ static_resources:
- filters:
- name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
codec_type: auto
stat_prefix: ingress_http
route_config:
Expand All @@ -24,7 +24,8 @@ static_resources:
- match: { prefix: "/" }
route:
cluster: greeter_service
max_grpc_timeout: 0s
max_stream_duration:
grpc_timeout_header_max: 0s
cors:
allow_origin_string_match:
- prefix: "*"
Expand All @@ -51,4 +52,4 @@ static_resources:
address:
socket_address:
address: 0.0.0.0
port_value: 9090
port_value: 9090
2 changes: 1 addition & 1 deletion scripts/run_interop_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ docker-compose build common prereqs node-interop-server interop-client java-inte
# Run interop tests
pid1=$(docker run -d \
-v "$(pwd)"/test/interop/envoy.yaml:/etc/envoy/envoy.yaml:ro \
--network=host envoyproxy/envoy:v1.16.1)
--network=host envoyproxy/envoy:v1.17.0)
pid2=$(docker run -d --network=host grpcweb/node-interop-server)

run_tests
Expand Down
2 changes: 1 addition & 1 deletion test/interop/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ tests.

```sh
$ docker run -d -v $(pwd)/test/interop/envoy.yaml:/etc/envoy/envoy.yaml:ro \
--network=host envoyproxy/envoy:v1.16.1
--network=host envoyproxy/envoy:v1.17.0
```


Expand Down
7 changes: 4 additions & 3 deletions test/interop/envoy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ static_resources:
- filters:
- name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
codec_type: auto
stat_prefix: ingress_http
route_config:
Expand All @@ -24,7 +24,8 @@ static_resources:
- match: { prefix: "/" }
route:
cluster: interop_service
max_grpc_timeout: 0s
max_stream_duration:
grpc_timeout_header_max: 0s
cors:
allow_origin_string_match:
- prefix: "*"
Expand All @@ -50,4 +51,4 @@ static_resources:
address:
socket_address:
address: localhost
port_value: 7074
port_value: 7074

0 comments on commit f960b8f

Please sign in to comment.