From 66cd15d34b9d3d81e2b0b1daf131dd8b1958eb97 Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Fri, 7 Feb 2020 14:21:53 +0100 Subject: [PATCH 1/8] add 1.12.2 --- .circleci/config.yml | 24 ++++++++++---------- test/integration/connect/envoy/run-tests.sh | 2 +- website/source/docs/connect/proxies/envoy.md | 3 ++- 3 files changed, 15 insertions(+), 14 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index dbe98e788bbc..c7302dd0b343 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -510,13 +510,13 @@ jobs: path: ui-v2/test-results # Envoy integration tests. Require docker dev binary to be built already - envoy-integration-test-1.8.0: + envoy-integration-test-1.9.1: docker: # We only really need bash and docker-compose which is installed on all # Circle images but pick Go since we have to pick one of them. - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.8.0" + ENVOY_VERSIONS: "1.9.1" steps: &ENVOY_INTEGRATION_TEST_STEPS - checkout # Get go binary from workspace @@ -536,25 +536,25 @@ jobs: path: ./test/integration/connect/envoy/workdir/logs destination: container-logs - envoy-integration-test-1.9.1: + envoy-integration-test-1.10.0: docker: - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.9.1" + ENVOY_VERSIONS: "1.10.0" steps: *ENVOY_INTEGRATION_TEST_STEPS - envoy-integration-test-1.10.0: + envoy-integration-test-1.11.2: docker: - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.10.0" + ENVOY_VERSIONS: "1.11.2" steps: *ENVOY_INTEGRATION_TEST_STEPS - envoy-integration-test-1.11.1: + envoy-integration-test-1.12.2: docker: - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.11.1" + ENVOY_VERSIONS: "1.12.2" steps: *ENVOY_INTEGRATION_TEST_STEPS # run tests on vault ca provider integration tests @@ -640,16 +640,16 @@ workflows: - nomad-integration-0_8: requires: - dev-build - - envoy-integration-test-1.8.0: - requires: - - dev-build - envoy-integration-test-1.9.1: requires: - dev-build - envoy-integration-test-1.10.0: requires: - dev-build - - envoy-integration-test-1.11.1: + - envoy-integration-test-1.11.2: + requires: + - dev-build + - envoy-integration-test-1.12.2: requires: - dev-build - vault-ca-provider: diff --git a/test/integration/connect/envoy/run-tests.sh b/test/integration/connect/envoy/run-tests.sh index 8ca3358074d6..cad3c2e87eab 100755 --- a/test/integration/connect/envoy/run-tests.sh +++ b/test/integration/connect/envoy/run-tests.sh @@ -17,7 +17,7 @@ FILTER_TESTS=${FILTER_TESTS:-} STOP_ON_FAIL=${STOP_ON_FAIL:-} # ENVOY_VERSIONS is the list of envoy versions to run each test against -ENVOY_VERSIONS=${ENVOY_VERSIONS:-"1.10.0 1.9.1 1.8.0 1.11.2"} +ENVOY_VERSIONS=${ENVOY_VERSIONS:-"1.9.1 1.10.0 1.11.2 1.12.2"} if [ ! -z "$DEBUG" ] ; then set -x diff --git a/website/source/docs/connect/proxies/envoy.md b/website/source/docs/connect/proxies/envoy.md index d8ac9fdb85bf..e2e62a46df1e 100644 --- a/website/source/docs/connect/proxies/envoy.md +++ b/website/source/docs/connect/proxies/envoy.md @@ -36,7 +36,8 @@ compatible Envoy versions. | Consul Version | Compatible Envoy Versions | |---|---| -| 1.5.2 and higher | 1.11.1 1.10.0, 1.9.1, 1.8.0† | +| 1.7.0 and higher | 1.12.2, 1.11.2, 1.10.0, 1.9.1 | +| 1.5.2, 1.5.3, 1.6.x | 1.11.1, 1.10.0, 1.9.1, 1.8.0† | | 1.5.0, 1.5.1 | 1.9.1, 1.8.0† | | 1.3.x, 1.4.x | 1.9.1, 1.8.0†, 1.7.0† | From 42aeb2d527b2b6d674dbc7ec631ae95a5b942c04 Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Fri, 7 Feb 2020 22:44:17 +0100 Subject: [PATCH 2/8] add envoy 1.13.0 --- .circleci/config.yml | 23 ++++++++++----------- command/connect/envoy/bootstrap_tpl.go | 12 +++++++++++ test/integration/connect/envoy/run-tests.sh | 3 +-- 3 files changed, 24 insertions(+), 14 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index c7302dd0b343..45a9255eeb74 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -509,14 +509,13 @@ jobs: - store_test_results: path: ui-v2/test-results - # Envoy integration tests. Require docker dev binary to be built already - envoy-integration-test-1.9.1: + envoy-integration-test-1.10.0: docker: # We only really need bash and docker-compose which is installed on all # Circle images but pick Go since we have to pick one of them. - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.9.1" + ENVOY_VERSIONS: "1.10.0" steps: &ENVOY_INTEGRATION_TEST_STEPS - checkout # Get go binary from workspace @@ -536,25 +535,25 @@ jobs: path: ./test/integration/connect/envoy/workdir/logs destination: container-logs - envoy-integration-test-1.10.0: + envoy-integration-test-1.11.2: docker: - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.10.0" + ENVOY_VERSIONS: "1.11.2" steps: *ENVOY_INTEGRATION_TEST_STEPS - envoy-integration-test-1.11.2: + envoy-integration-test-1.12.2: docker: - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.11.2" + ENVOY_VERSIONS: "1.12.2" steps: *ENVOY_INTEGRATION_TEST_STEPS - envoy-integration-test-1.12.2: + envoy-integration-test-1.13.0: docker: - image: *GOLANG_IMAGE environment: - ENVOY_VERSIONS: "1.12.2" + ENVOY_VERSIONS: "1.13.0" steps: *ENVOY_INTEGRATION_TEST_STEPS # run tests on vault ca provider integration tests @@ -640,9 +639,6 @@ workflows: - nomad-integration-0_8: requires: - dev-build - - envoy-integration-test-1.9.1: - requires: - - dev-build - envoy-integration-test-1.10.0: requires: - dev-build @@ -652,6 +648,9 @@ workflows: - envoy-integration-test-1.12.2: requires: - dev-build + - envoy-integration-test-1.13.0: + requires: + - dev-build - vault-ca-provider: requires: - dev-build diff --git a/command/connect/envoy/bootstrap_tpl.go b/command/connect/envoy/bootstrap_tpl.go index d646bfe9301f..80e3e69a8974 100644 --- a/command/connect/envoy/bootstrap_tpl.go +++ b/command/connect/envoy/bootstrap_tpl.go @@ -190,6 +190,18 @@ const bootstrapTemplate = `{ } } } + }, + "layered_runtime":{ + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } ` diff --git a/test/integration/connect/envoy/run-tests.sh b/test/integration/connect/envoy/run-tests.sh index cad3c2e87eab..e40201a31ddd 100755 --- a/test/integration/connect/envoy/run-tests.sh +++ b/test/integration/connect/envoy/run-tests.sh @@ -17,7 +17,7 @@ FILTER_TESTS=${FILTER_TESTS:-} STOP_ON_FAIL=${STOP_ON_FAIL:-} # ENVOY_VERSIONS is the list of envoy versions to run each test against -ENVOY_VERSIONS=${ENVOY_VERSIONS:-"1.9.1 1.10.0 1.11.2 1.12.2"} +ENVOY_VERSIONS=${ENVOY_VERSIONS:-"1.10.0 1.11.2 1.12.2 1.13.0"} if [ ! -z "$DEBUG" ] ; then set -x @@ -27,7 +27,6 @@ DIR=$(cd -P -- "$(dirname -- "$0")" && pwd -P) cd $DIR -FILTER_TESTS=${FILTER_TESTS:-} LEAVE_CONSUL_UP=${LEAVE_CONSUL_UP:-} PROXY_LOGS_ON_FAIL=${PROXY_LOGS_ON_FAIL:-} From 943af3a762c7b5a41db3c0f99139b136caf3e707 Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Mon, 10 Feb 2020 11:06:04 +0100 Subject: [PATCH 3/8] Introduce -envoy-version to get 1.10.0 passing. --- command/connect/envoy/bootstrap_tpl.go | 4 ++++ command/connect/envoy/envoy.go | 5 +++++ test/integration/connect/envoy/helpers.bash | 1 + 3 files changed, 10 insertions(+) diff --git a/command/connect/envoy/bootstrap_tpl.go b/command/connect/envoy/bootstrap_tpl.go index 80e3e69a8974..d5043acd370d 100644 --- a/command/connect/envoy/bootstrap_tpl.go +++ b/command/connect/envoy/bootstrap_tpl.go @@ -96,6 +96,8 @@ type BootstrapTplArgs struct { // Namespace is the Consul Enterprise Namespace of the proxy service instance as // registered with the Consul agent. Namespace string + + EnvoyVersion string } const bootstrapTemplate = `{ @@ -190,6 +192,7 @@ const bootstrapTemplate = `{ } } } + {{- if ne .EnvoyVersion "1.10.0"}} }, "layered_runtime":{ "layers": [ @@ -202,6 +205,7 @@ const bootstrapTemplate = `{ } } ] + {{- end }} } } ` diff --git a/command/connect/envoy/envoy.go b/command/connect/envoy/envoy.go index 4bc5432ab426..0aaee2787e8f 100644 --- a/command/connect/envoy/envoy.go +++ b/command/connect/envoy/envoy.go @@ -56,6 +56,7 @@ type cmd struct { bootstrap bool disableCentralConfig bool grpcAddr string + envoyVersion string // mesh gateway registration information register bool @@ -110,6 +111,9 @@ func (c *cmd) init() { "Set the agent's gRPC address and port (in http(s)://host:port format). "+ "Alternatively, you can specify CONSUL_GRPC_ADDR in ENV.") + c.flags.StringVar(&c.envoyVersion, "envoy-version", "1.13.0", + "Sets the envoy-version that the envoy binary has.") + c.flags.BoolVar(&c.register, "register", false, "Register a new Mesh Gateway service before configuring and starting Envoy") @@ -519,6 +523,7 @@ func (c *cmd) templateArgs() (*BootstrapTplArgs, error) { Token: httpCfg.Token, LocalAgentClusterName: xds.LocalAgentClusterName, Namespace: httpCfg.Namespace, + EnvoyVersion: c.envoyVersion, }, nil } diff --git a/test/integration/connect/envoy/helpers.bash b/test/integration/connect/envoy/helpers.bash index a5cfdfdffca7..134c9974996e 100755 --- a/test/integration/connect/envoy/helpers.bash +++ b/test/integration/connect/envoy/helpers.bash @@ -508,6 +508,7 @@ function gen_envoy_bootstrap { if output=$(docker_consul "$DC" connect envoy -bootstrap \ -proxy-id $PROXY_ID \ + -envoy-version "$ENVOY_VERSION" \ -admin-bind 0.0.0.0:$ADMIN_PORT ${EXTRA_ENVOY_BS_ARGS} 2>&1); then # All OK, write config to file From 4da1be6b86b338182187b24e00ba65fdfa8271fa Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Mon, 10 Feb 2020 12:00:23 +0100 Subject: [PATCH 4/8] update old version and fix consul-exec case --- .../connect/envoy/docker-compose.yml | 30 ++++++++++--------- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/test/integration/connect/envoy/docker-compose.yml b/test/integration/connect/envoy/docker-compose.yml index a89745395f8c..a2ffebd7540d 100644 --- a/test/integration/connect/envoy/docker-compose.yml +++ b/test/integration/connect/envoy/docker-compose.yml @@ -175,7 +175,7 @@ services: s1-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -196,7 +196,7 @@ services: s2-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -217,7 +217,7 @@ services: s2-v1-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -238,7 +238,7 @@ services: s2-v2-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -259,7 +259,7 @@ services: s3-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -280,7 +280,7 @@ services: s3-v1-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -301,7 +301,7 @@ services: s3-v2-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -322,7 +322,7 @@ services: s3-alt-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -363,14 +363,16 @@ services: context: . dockerfile: Dockerfile-consul-envoy args: - ENVOY_VERSION: ${ENVOY_VERSION:-1.8.0} - image: consul-dev-envoy:${ENVOY_VERSION:-1.8.0} + ENVOY_VERSION: ${ENVOY_VERSION:-1.13.0} + image: consul-dev-envoy:${ENVOY_VERSION:-1.13.0} command: - "consul" - "connect" - "envoy" - "-sidecar-for" - "s1" + - "-envoy-version" + - ${ENVOY_VERSION:-1.13.0} - "--" - "-l" - "debug" @@ -480,7 +482,7 @@ services: s1-sidecar-proxy-secondary: depends_on: - consul-secondary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -501,7 +503,7 @@ services: s2-sidecar-proxy-secondary: depends_on: - consul-secondary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -522,7 +524,7 @@ services: gateway-primary: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" @@ -543,7 +545,7 @@ services: gateway-secondary: depends_on: - consul-secondary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.8.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" command: - "envoy" - "-c" From d091e8d6316208b548e94c6172827eb01fe4be3b Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Mon, 10 Feb 2020 14:24:35 +0100 Subject: [PATCH 5/8] add envoy_version and fix check --- command/connect/envoy/bootstrap_tpl.go | 5 +++-- test/integration/connect/envoy/helpers.bash | 9 ++++++++- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/command/connect/envoy/bootstrap_tpl.go b/command/connect/envoy/bootstrap_tpl.go index d5043acd370d..25034e39757d 100644 --- a/command/connect/envoy/bootstrap_tpl.go +++ b/command/connect/envoy/bootstrap_tpl.go @@ -114,7 +114,8 @@ const bootstrapTemplate = `{ "cluster": "{{ .ProxyCluster }}", "id": "{{ .ProxyID }}", "metadata": { - "namespace": "{{if ne .Namespace ""}}{{ .Namespace }}{{else}}default{{end}}" + "namespace": "{{if ne .Namespace ""}}{{ .Namespace }}{{else}}default{{end}}", + "envoy_version": "{{ .EnvoyVersion }}" } }, "static_resources": { @@ -198,7 +199,7 @@ const bootstrapTemplate = `{ "layers": [ { "name": "static_layer", - "static_layer": { + "static_layer": { "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true diff --git a/test/integration/connect/envoy/helpers.bash b/test/integration/connect/envoy/helpers.bash index 134c9974996e..4ce0a117f40a 100755 --- a/test/integration/connect/envoy/helpers.bash +++ b/test/integration/connect/envoy/helpers.bash @@ -144,7 +144,14 @@ function get_envoy_listener_filters { local HOSTPORT=$1 run retry_default curl -s -f $HOSTPORT/config_dump [ "$status" -eq 0 ] - echo "$output" | jq --raw-output '.configs[2].dynamic_active_listeners[].listener | "\(.name) \( .filter_chains[0].filters | map(.name) | join(","))"' + local ENVOY_VERSION=$(echo $output | jq --raw-output '.configs[0].bootstrap.node.metadata.envoy_version') + local QUERY='' + if [ "$ENVOY_VERSION" == "1.13.0" ]; then + QUERY='.configs[2].dynamic_listeners[].active_state.listener | "\(.name) \( .filter_chains[0].filters | map(.name) | join(","))"' + else + QUERY='.configs[2].dynamic_active_listeners[].listener | "\(.name) \( .filter_chains[0].filters | map(.name) | join(","))"' + fi + echo "$output" | jq --raw-output "$QUERY" } function get_envoy_cluster_threshold { From 7d4a26731eb911feeba1bff91b1fda3ab29ecc55 Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Mon, 10 Feb 2020 14:26:34 +0100 Subject: [PATCH 6/8] better json --- command/connect/envoy/bootstrap_tpl.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/command/connect/envoy/bootstrap_tpl.go b/command/connect/envoy/bootstrap_tpl.go index 25034e39757d..440c7ac1e6c8 100644 --- a/command/connect/envoy/bootstrap_tpl.go +++ b/command/connect/envoy/bootstrap_tpl.go @@ -200,9 +200,9 @@ const bootstrapTemplate = `{ { "name": "static_layer", "static_layer": { - "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, - "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, - "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true } } ] From 3bbda9a0077f9dd30dc3cc2fa925ea11a9438c72 Mon Sep 17 00:00:00 2001 From: Hans Hasselberg Date: Mon, 10 Feb 2020 16:51:32 +0100 Subject: [PATCH 7/8] feedback --- command/connect/envoy/bootstrap_tpl.go | 2 ++ .../connect/envoy/docker-compose.yml | 30 +++++++++---------- .../docs/commands/connect/envoy.html.md.erb | 3 ++ 3 files changed, 20 insertions(+), 15 deletions(-) diff --git a/command/connect/envoy/bootstrap_tpl.go b/command/connect/envoy/bootstrap_tpl.go index 440c7ac1e6c8..54287069be8f 100644 --- a/command/connect/envoy/bootstrap_tpl.go +++ b/command/connect/envoy/bootstrap_tpl.go @@ -97,6 +97,8 @@ type BootstrapTplArgs struct { // registered with the Consul agent. Namespace string + // EnvoyVersion is the envoy version, which is necessary to generate the + // correct configuration. EnvoyVersion string } diff --git a/test/integration/connect/envoy/docker-compose.yml b/test/integration/connect/envoy/docker-compose.yml index a2ffebd7540d..2df2a12c815c 100644 --- a/test/integration/connect/envoy/docker-compose.yml +++ b/test/integration/connect/envoy/docker-compose.yml @@ -175,7 +175,7 @@ services: s1-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -196,7 +196,7 @@ services: s2-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -217,7 +217,7 @@ services: s2-v1-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -238,7 +238,7 @@ services: s2-v2-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -259,7 +259,7 @@ services: s3-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -280,7 +280,7 @@ services: s3-v1-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -301,7 +301,7 @@ services: s3-v2-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -322,7 +322,7 @@ services: s3-alt-sidecar-proxy: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -363,8 +363,8 @@ services: context: . dockerfile: Dockerfile-consul-envoy args: - ENVOY_VERSION: ${ENVOY_VERSION:-1.13.0} - image: consul-dev-envoy:${ENVOY_VERSION:-1.13.0} + ENVOY_VERSION: ${ENVOY_VERSION} + image: consul-dev-envoy:${ENVOY_VERSION} command: - "consul" - "connect" @@ -372,7 +372,7 @@ services: - "-sidecar-for" - "s1" - "-envoy-version" - - ${ENVOY_VERSION:-1.13.0} + - ${ENVOY_VERSION} - "--" - "-l" - "debug" @@ -482,7 +482,7 @@ services: s1-sidecar-proxy-secondary: depends_on: - consul-secondary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -503,7 +503,7 @@ services: s2-sidecar-proxy-secondary: depends_on: - consul-secondary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -524,7 +524,7 @@ services: gateway-primary: depends_on: - consul-primary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" @@ -545,7 +545,7 @@ services: gateway-secondary: depends_on: - consul-secondary - image: "envoyproxy/envoy:v${ENVOY_VERSION:-1.13.0}" + image: "envoyproxy/envoy:v${ENVOY_VERSION}" command: - "envoy" - "-c" diff --git a/website/source/docs/commands/connect/envoy.html.md.erb b/website/source/docs/commands/connect/envoy.html.md.erb index 51ae8628abef..aab568caa886 100644 --- a/website/source/docs/commands/connect/envoy.html.md.erb +++ b/website/source/docs/commands/connect/envoy.html.md.erb @@ -73,6 +73,9 @@ proxy configuration needed. for and so can be used to access any upstream service that that service is allowed to access by [Connect intentions](/docs/connect/intentions.html). + * `-envoy-version` - The version of envoy that is being started. Default is + `1.13.0`. This is required so that the correct configuration can be generated. + * `-- [pass-through options]` - Any options given after a double dash are passed directly through to the `envoy` invocation. See [Envoy's documentation](https://www.envoyproxy.io/docs) for more details. The command From e42d665fcf0f8abafe3241df9369e6a6a1919ae6 Mon Sep 17 00:00:00 2001 From: Matt Keeler Date: Mon, 10 Feb 2020 14:09:26 -0500 Subject: [PATCH 8/8] Update Envoy CLI tests to account for the 1.13 compatibility changes. --- command/connect/envoy/envoy_test.go | 17 +++++++++++++++++ .../envoy/testdata/access-log-path.golden | 15 ++++++++++++++- command/connect/envoy/testdata/defaults.golden | 15 ++++++++++++++- .../envoy/testdata/existing-ca-file.golden | 15 ++++++++++++++- .../envoy/testdata/extra_-multiple.golden | 15 ++++++++++++++- .../connect/envoy/testdata/extra_-single.golden | 15 ++++++++++++++- .../envoy/testdata/grpc-addr-config.golden | 15 ++++++++++++++- .../connect/envoy/testdata/grpc-addr-env.golden | 15 ++++++++++++++- .../envoy/testdata/grpc-addr-flag.golden | 15 ++++++++++++++- .../envoy/testdata/grpc-addr-unix.golden | 15 ++++++++++++++- .../envoy/testdata/stats-config-override.golden | 15 ++++++++++++++- command/connect/envoy/testdata/token-arg.golden | 15 ++++++++++++++- command/connect/envoy/testdata/token-env.golden | 15 ++++++++++++++- .../envoy/testdata/token-file-arg.golden | 15 ++++++++++++++- .../envoy/testdata/token-file-env.golden | 15 ++++++++++++++- .../envoy/testdata/zipkin-tracing-config.golden | 15 ++++++++++++++- 16 files changed, 227 insertions(+), 15 deletions(-) diff --git a/command/connect/envoy/envoy_test.go b/command/connect/envoy/envoy_test.go index 3e58f8679f59..a3ceaf9e739b 100644 --- a/command/connect/envoy/envoy_test.go +++ b/command/connect/envoy/envoy_test.go @@ -83,6 +83,7 @@ func TestGenerateConfig(t *testing.T) { Flags: []string{"-proxy-id", "test-proxy"}, Env: []string{}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -99,6 +100,7 @@ func TestGenerateConfig(t *testing.T) { "-token", "c9a52720-bf6c-4aa6-b8bc-66881a5ade95"}, Env: []string{}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -117,6 +119,7 @@ func TestGenerateConfig(t *testing.T) { "CONSUL_HTTP_TOKEN=c9a52720-bf6c-4aa6-b8bc-66881a5ade95", }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -138,6 +141,7 @@ func TestGenerateConfig(t *testing.T) { "token.txt": "c9a52720-bf6c-4aa6-b8bc-66881a5ade95", }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -159,6 +163,7 @@ func TestGenerateConfig(t *testing.T) { "token.txt": "c9a52720-bf6c-4aa6-b8bc-66881a5ade95", }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -176,6 +181,7 @@ func TestGenerateConfig(t *testing.T) { "-grpc-addr", "localhost:9999"}, Env: []string{}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", // Should resolve IP, note this might not resolve the same way @@ -196,6 +202,7 @@ func TestGenerateConfig(t *testing.T) { "CONSUL_GRPC_ADDR=localhost:9999", }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", // Should resolve IP, note this might not resolve the same way @@ -215,6 +222,7 @@ func TestGenerateConfig(t *testing.T) { "-grpc-addr", "unix:///var/run/consul.sock"}, Env: []string{}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentSocket: "/var/run/consul.sock", @@ -229,6 +237,7 @@ func TestGenerateConfig(t *testing.T) { Flags: []string{"-proxy-id", "test-proxy"}, GRPCPort: 9999, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", // Should resolve IP, note this might not resolve the same way @@ -247,6 +256,7 @@ func TestGenerateConfig(t *testing.T) { Flags: []string{"-proxy-id", "test-proxy", "-admin-access-log-path", "/some/path/access.log"}, Env: []string{}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", // Should resolve IP, note this might not resolve the same way @@ -265,6 +275,7 @@ func TestGenerateConfig(t *testing.T) { Flags: []string{"-proxy-id", "test-proxy", "-ca-file", "some/path"}, Env: []string{}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", // Should resolve IP, note this might not resolve the same way @@ -280,6 +291,7 @@ func TestGenerateConfig(t *testing.T) { Flags: []string{"-proxy-id", "test-proxy", "-ca-file", "../../../test/ca/root.cer"}, Env: []string{"CONSUL_HTTP_SSL=1"}, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", // Should resolve IP, note this might not resolve the same way @@ -322,6 +334,7 @@ func TestGenerateConfig(t *testing.T) { }`, }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -354,6 +367,7 @@ func TestGenerateConfig(t *testing.T) { }`, }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -391,6 +405,7 @@ func TestGenerateConfig(t *testing.T) { } , { "name": "fake_sink_2" }`, }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -415,6 +430,7 @@ func TestGenerateConfig(t *testing.T) { }`, }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", @@ -469,6 +485,7 @@ func TestGenerateConfig(t *testing.T) { }`, }, WantArgs: BootstrapTplArgs{ + EnvoyVersion: "1.13.0", ProxyCluster: "test-proxy", ProxyID: "test-proxy", AgentAddress: "127.0.0.1", diff --git a/command/connect/envoy/testdata/access-log-path.golden b/command/connect/envoy/testdata/access-log-path.golden index 2b8851377c3f..fc704bd6a5d0 100644 --- a/command/connect/envoy/testdata/access-log-path.golden +++ b/command/connect/envoy/testdata/access-log-path.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/defaults.golden b/command/connect/envoy/testdata/defaults.golden index 6d8c7980f615..b81fc2b1e9ab 100644 --- a/command/connect/envoy/testdata/defaults.golden +++ b/command/connect/envoy/testdata/defaults.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/existing-ca-file.golden b/command/connect/envoy/testdata/existing-ca-file.golden index 9b8b656ea3da..264b0dc667d5 100644 --- a/command/connect/envoy/testdata/existing-ca-file.golden +++ b/command/connect/envoy/testdata/existing-ca-file.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -108,5 +109,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/extra_-multiple.golden b/command/connect/envoy/testdata/extra_-multiple.golden index 726828984712..845361ec13a3 100644 --- a/command/connect/envoy/testdata/extra_-multiple.golden +++ b/command/connect/envoy/testdata/extra_-multiple.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -121,5 +122,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/extra_-single.golden b/command/connect/envoy/testdata/extra_-single.golden index 57e660f1b2f2..7ab52ab78e02 100644 --- a/command/connect/envoy/testdata/extra_-single.golden +++ b/command/connect/envoy/testdata/extra_-single.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -112,5 +113,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/grpc-addr-config.golden b/command/connect/envoy/testdata/grpc-addr-config.golden index c58d4d32cd17..f51834610f73 100644 --- a/command/connect/envoy/testdata/grpc-addr-config.golden +++ b/command/connect/envoy/testdata/grpc-addr-config.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/grpc-addr-env.golden b/command/connect/envoy/testdata/grpc-addr-env.golden index c58d4d32cd17..f51834610f73 100644 --- a/command/connect/envoy/testdata/grpc-addr-env.golden +++ b/command/connect/envoy/testdata/grpc-addr-env.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/grpc-addr-flag.golden b/command/connect/envoy/testdata/grpc-addr-flag.golden index c58d4d32cd17..f51834610f73 100644 --- a/command/connect/envoy/testdata/grpc-addr-flag.golden +++ b/command/connect/envoy/testdata/grpc-addr-flag.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/grpc-addr-unix.golden b/command/connect/envoy/testdata/grpc-addr-unix.golden index 76ccbc23cade..a302087c92aa 100644 --- a/command/connect/envoy/testdata/grpc-addr-unix.golden +++ b/command/connect/envoy/testdata/grpc-addr-unix.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -98,5 +99,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/stats-config-override.golden b/command/connect/envoy/testdata/stats-config-override.golden index 8b83fbba04a6..b793637bffd2 100644 --- a/command/connect/envoy/testdata/stats-config-override.golden +++ b/command/connect/envoy/testdata/stats-config-override.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -57,5 +58,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/token-arg.golden b/command/connect/envoy/testdata/token-arg.golden index 72693933b99d..6f5fd0d56ad4 100644 --- a/command/connect/envoy/testdata/token-arg.golden +++ b/command/connect/envoy/testdata/token-arg.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/token-env.golden b/command/connect/envoy/testdata/token-env.golden index 72693933b99d..6f5fd0d56ad4 100644 --- a/command/connect/envoy/testdata/token-env.golden +++ b/command/connect/envoy/testdata/token-env.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/token-file-arg.golden b/command/connect/envoy/testdata/token-file-arg.golden index 72693933b99d..6f5fd0d56ad4 100644 --- a/command/connect/envoy/testdata/token-file-arg.golden +++ b/command/connect/envoy/testdata/token-file-arg.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/token-file-env.golden b/command/connect/envoy/testdata/token-file-env.golden index 72693933b99d..6f5fd0d56ad4 100644 --- a/command/connect/envoy/testdata/token-file-env.golden +++ b/command/connect/envoy/testdata/token-file-env.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -99,5 +100,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } } diff --git a/command/connect/envoy/testdata/zipkin-tracing-config.golden b/command/connect/envoy/testdata/zipkin-tracing-config.golden index 74bcab3be044..ca9a24eb9a89 100644 --- a/command/connect/envoy/testdata/zipkin-tracing-config.golden +++ b/command/connect/envoy/testdata/zipkin-tracing-config.golden @@ -12,7 +12,8 @@ "cluster": "test-proxy", "id": "test-proxy", "metadata": { - "namespace": "default" + "namespace": "default", + "envoy_version": "1.13.0" } }, "static_resources": { @@ -132,5 +133,17 @@ } } } + }, + "layered_runtime": { + "layers": [ + { + "name": "static_layer", + "static_layer": { + "envoy.deprecated_features:envoy.api.v2.Cluster.tls_context": true, + "envoy.deprecated_features:envoy.config.trace.v2.ZipkinConfig.HTTP_JSON_V1": true, + "envoy.deprecated_features:envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager.Tracing.operation_name": true + } + } + ] } }