Skip to content

Commit 1ef552d

Browse files
nfudenjbohanonben-taussig-solo
authored
1.27 | bazel: Update to a newer version of envoy-fork with http2 continuation cve (#321)
* bazel: Update to a newer version of envoy-fork with http2 continuation cve * bazel: Leading space in hash * Use private pool cloud build worker (#316) * use 16core private worker * mimic build-bot's repo tag * sub-job tag * add creds for build cache (#313) * export BAZEL_BUILD_EXTRA_OPTIONS, GCP_SERVICE_ACCOUNT_KEY_PATH from do_ci.sh * Update ci/do_ci.sh Co-authored-by: Jacob Bohanon <[email protected]> * update BAZEL_BUILD_EXTRA_OPTIONS line --------- Co-authored-by: Jacob Bohanon <[email protected]> * actually save the changes to doci resolution --------- Co-authored-by: Jacob Bohanon <[email protected]> Co-authored-by: Ben Taussig <[email protected]>
1 parent 79b2816 commit 1ef552d

File tree

5 files changed

+33
-9
lines changed

5 files changed

+33
-9
lines changed

bazel/repository_locations.bzl

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
REPOSITORY_LOCATIONS = dict(
22
envoy = dict(
3-
# envoy 1.27.3 from release v1.27.3-fork1
4-
commit = "ebfb8ff1944e55756318f89bea189e0470daaed0",
3+
# envoy 1.27.4 from release v1.27.4-fork2
4+
commit = "31f980b46a9ee24c545655be82b6849d6b9a16a8",
55
remote = "https://github.com/solo-io/envoy-fork",
66
),
77
inja = dict(
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
changelog:
2+
- type: DEPENDENCY_BUMP
3+
dependencyOwner: solo-io
4+
dependencyRepo: envoy-fork
5+
dependencyTag: v1.27.4-fork1
6+
issueLink: https://github.com/solo-io/solo-projects/issues/6008
7+
resolvesIssue: false
8+
description: >
9+
Bump to upstream envoy v1.27.4
10+
Tackles the http2 crazy cve CVE-2024-30255

ci/cloudbuild.yaml

+10-1
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,23 @@ steps:
5353
- 'TAGGED_VERSION=$TAG_NAME'
5454

5555
options:
56-
machineType: 'N1_HIGHCPU_32'
56+
pool:
57+
name: 'projects/solo-public/locations/us-central1/workerPools/envoy-gloo-runner'
5758
timeout: 20000s
5859

5960
artifacts:
6061
objects:
6162
location: 'gs://solo-public-artifacts.solo.io/envoy/$COMMIT_SHA/'
6263
paths: ['linux/amd64/build_envoy_release/envoy']
6364

65+
tags:
66+
- "repo_envoy-gloo"
67+
# This tag can be used to filter for or out jobs which are spawned by the main job
68+
# submitted by build-bot. It's somewhat redundant as one could filter on `tags~^pr`
69+
# to achieve the same effect since that tag is added to main jobs by build-bot,
70+
# but this is somewhat less esoteric
71+
- "sub-job"
72+
6473
availableSecrets:
6574
inline:
6675
- kmsKeyName: projects/solo-public/locations/global/keyRings/build/cryptoKeys/build-key

ci/do_ci.sh

+7-4
Original file line numberDiff line numberDiff line change
@@ -32,17 +32,20 @@ fi
3232

3333
export ENVOY_SRCDIR=$SOURCE_DIR
3434

35-
# google cloud build times out when using full throttle.
36-
export NUM_CPUS=10
37-
3835
# google cloud build doesn't like ipv6
39-
export BAZEL_EXTRA_TEST_OPTIONS="--test_env=ENVOY_IP_TEST_VERSIONS=v4only --test_output=errors --jobs=${NUM_CPUS}"
36+
export BAZEL_EXTRA_TEST_OPTIONS="--test_env=ENVOY_IP_TEST_VERSIONS=v4only --test_output=errors"
4037

4138
# We do not need/want to build the Envoy contrib filters so we replace the
4239
# associated targets with the ENVOY_BUILD values
4340
export ENVOY_CONTRIB_BUILD_TARGET="//source/exe:envoy-static"
4441
export ENVOY_CONTRIB_BUILD_DEBUG_INFORMATION="//source/exe:envoy-static.dwp"
4542

43+
BAZEL_BUILD_EXTRA_OPTIONS+=" --remote_cache=${BAZEL_REMOTE_CACHE}"
44+
45+
export GCP_SERVICE_ACCOUNT_KEY_PATH=$(mktemp -t gcp_service_account.XXXXXX.json)
46+
echo "${GCP_SERVICE_ACCOUNT_KEY}" | base64 --decode > "${GCP_SERVICE_ACCOUNT_KEY_PATH}"
47+
BAZEL_BUILD_EXTRA_OPTIONS+=" --google_credentials=${GCP_SERVICE_ACCOUNT_KEY_PATH}"
48+
4649
if [ "${BUILD_TYPE:-}" != "" ] ; then
4750
BUILD_CONFIG="--config=$BUILD_TYPE"
4851
fi

cloudbuild.yaml

+4-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
steps:
22

33
- name: 'gcr.io/cloud-builders/gcloud'
4-
args: ['builds','submit','--config=ci/cloudbuild.yaml','--substitutions','TAG_NAME=$TAG_NAME,COMMIT_SHA=$COMMIT_SHA,_BUILD_TYPE=']
4+
id: "standard"
5+
args: ['builds','submit','--region=us-central1','--config=ci/cloudbuild.yaml','--substitutions','TAG_NAME=$TAG_NAME,COMMIT_SHA=$COMMIT_SHA,_BUILD_TYPE=']
56

67
- name: 'gcr.io/cloud-builders/gcloud'
7-
args: ['builds','submit','--config=ci/cloudbuild.yaml','--substitutions','COMMIT_SHA=$COMMIT_SHA,_BUILD_TYPE=clang-asan']
8+
id: "asan"
9+
args: ['builds','submit','--region=us-central1','--config=ci/cloudbuild.yaml','--substitutions','COMMIT_SHA=$COMMIT_SHA,_BUILD_TYPE=clang-asan']
810
waitFor: ['-']
911

1012
timeout: 20000s

0 commit comments

Comments
 (0)