connect: Add Envoy 1.21.1 to support matrix, remove 1.17.4 (#12777)

This commit is contained in:
Evan Culver 2022-04-14 10:44:42 -07:00 committed by GitHub
parent acf9a9799e
commit e62745c82c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
218 changed files with 25 additions and 17 deletions

3
.changelog/12777.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:improvement
connect: Add Envoy 1.21.1 to support matrix, remove 1.17.4
```

View File

@ -845,13 +845,13 @@ jobs:
command: make test-coverage-ci command: make test-coverage-ci
- run: *notify-slack-failure - run: *notify-slack-failure
envoy-integration-test-1_17_4: &ENVOY_TESTS envoy-integration-test-1_18_6: &ENVOY_TESTS
machine: machine:
image: ubuntu-2004:202101-01 image: ubuntu-2004:202101-01
parallelism: 4 parallelism: 4
resource_class: medium resource_class: medium
environment: environment:
ENVOY_VERSION: "1.17.4" ENVOY_VERSION: "1.18.6"
steps: &ENVOY_INTEGRATION_TEST_STEPS steps: &ENVOY_INTEGRATION_TEST_STEPS
- checkout - checkout
# Get go binary from workspace # Get go binary from workspace
@ -884,11 +884,6 @@ jobs:
path: *TEST_RESULTS_DIR path: *TEST_RESULTS_DIR
- run: *notify-slack-failure - run: *notify-slack-failure
envoy-integration-test-1_18_6:
<<: *ENVOY_TESTS
environment:
ENVOY_VERSION: "1.18.6"
envoy-integration-test-1_19_3: envoy-integration-test-1_19_3:
<<: *ENVOY_TESTS <<: *ENVOY_TESTS
environment: environment:
@ -899,6 +894,11 @@ jobs:
environment: environment:
ENVOY_VERSION: "1.20.2" ENVOY_VERSION: "1.20.2"
envoy-integration-test-1_21_1:
<<: *ENVOY_TESTS
environment:
ENVOY_VERSION: "1.21.1"
# run integration tests for the connect ca providers # run integration tests for the connect ca providers
test-connect-ca-providers: test-connect-ca-providers:
docker: docker:
@ -1140,9 +1140,6 @@ workflows:
- nomad-integration-0_8: - nomad-integration-0_8:
requires: requires:
- dev-build - dev-build
- envoy-integration-test-1_17_4:
requires:
- dev-build
- envoy-integration-test-1_18_6: - envoy-integration-test-1_18_6:
requires: requires:
- dev-build - dev-build
@ -1152,6 +1149,9 @@ workflows:
- envoy-integration-test-1_20_2: - envoy-integration-test-1_20_2:
requires: requires:
- dev-build - dev-build
- envoy-integration-test-1_21_1:
requires:
- dev-build
website: website:
unless: << pipeline.parameters.trigger-load-test >> unless: << pipeline.parameters.trigger-load-test >>

View File

@ -11,7 +11,7 @@ import (
var ( var (
// minSupportedVersion is the oldest mainline version we support. This should always be // minSupportedVersion is the oldest mainline version we support. This should always be
// the zero'th point release of the last element of proxysupport.EnvoyVersions. // the zero'th point release of the last element of proxysupport.EnvoyVersions.
minSupportedVersion = version.Must(version.NewVersion("1.17.0")) minSupportedVersion = version.Must(version.NewVersion("1.18.0"))
minVersionToForceLDSandCDSToAlwaysUseWildcardsOnReconnect = version.Must(version.NewVersion("1.19.0")) minVersionToForceLDSandCDSToAlwaysUseWildcardsOnReconnect = version.Must(version.NewVersion("1.19.0"))

View File

@ -123,7 +123,6 @@ func TestDetermineSupportedProxyFeaturesFromString(t *testing.T) {
// Insert a bunch of valid versions. // Insert a bunch of valid versions.
// Populate feature flags here when appropriate. See consul 1.10.x for reference. // Populate feature flags here when appropriate. See consul 1.10.x for reference.
for _, v := range []string{ for _, v := range []string{
"1.17.0", "1.17.1", "1.17.2", "1.17.3", "1.17.4",
"1.18.0", "1.18.1", "1.18.2", "1.18.3", "1.18.4", "1.18.5", "1.18.6", "1.18.0", "1.18.1", "1.18.2", "1.18.3", "1.18.4", "1.18.5", "1.18.6",
} { } {
cases[v] = testcase{expect: supportedProxyFeatures{ cases[v] = testcase{expect: supportedProxyFeatures{
@ -133,6 +132,7 @@ func TestDetermineSupportedProxyFeaturesFromString(t *testing.T) {
for _, v := range []string{ for _, v := range []string{
"1.19.0", "1.19.1", "1.19.2", "1.19.3", "1.19.0", "1.19.1", "1.19.2", "1.19.3",
"1.20.0", "1.20.1", "1.20.2", "1.20.0", "1.20.1", "1.20.2",
"1.21.1",
} { } {
cases[v] = testcase{expect: supportedProxyFeatures{}} cases[v] = testcase{expect: supportedProxyFeatures{}}
} }

View File

@ -35,7 +35,11 @@ func goldenEnvoy(t *testing.T, name, envoyVersion, latestEnvoyVersion, got strin
// coalescing works below when there is no xDS generated skew across envoy // coalescing works below when there is no xDS generated skew across envoy
// versions. // versions.
subname := goldenEnvoyVersionName(t, envoyVersion) subname := goldenEnvoyVersionName(t, envoyVersion)
latestSubname := goldenEnvoyVersionName(t, latestEnvoyVersion)
latestSubname := "latest"
if envoyVersion == latestEnvoyVersion {
subname = "latest"
}
return golden(t, name, subname, latestSubname, got) return golden(t, name, subname, latestSubname, got)
} }
@ -85,7 +89,7 @@ func golden(t *testing.T, name, subname, latestSubname, got string) string {
if latestSubname != "" && subname != latestSubname { if latestSubname != "" && subname != latestSubname {
latestGolden := filepath.Join("testdata", fmt.Sprintf("%s.%s.golden", name, latestSubname)) latestGolden := filepath.Join("testdata", fmt.Sprintf("%s.%s.golden", name, latestSubname))
raw, err := ioutil.ReadFile(latestGolden) raw, err := ioutil.ReadFile(latestGolden)
require.NoError(t, err) require.NoError(t, err, "%q %q %q", name, subname, latestSubname)
latestExpected = string(raw) latestExpected = string(raw)
} }

View File

@ -7,8 +7,8 @@ package proxysupport
// //
// see: https://www.consul.io/docs/connect/proxies/envoy#supported-versions // see: https://www.consul.io/docs/connect/proxies/envoy#supported-versions
var EnvoyVersions = []string{ var EnvoyVersions = []string{
"1.21.1",
"1.20.2", "1.20.2",
"1.19.3", "1.19.3",
"1.18.6", "1.18.6",
"1.17.4",
} }

Some files were not shown because too many files have changed in this diff Show More