Skip to content

Commit

Permalink
Update Envoy to 8cd214d (Sep 13th 2024). (#1220)
Browse files Browse the repository at this point in the history
- synced `.bazelrc`, `tools/gen_compilation_database.py`, `tools/code_format/config.yaml` from Envoy's version.
- no changes in `.bazelversion`, `ci/run_envoy_docker.sh`.
- replaced macro `THROW_IF_STATUS_NOT_OK` with `THROW_IF_NOT_OK_REF` as per envoyproxy/envoy#35694.

Signed-off-by: Jakub Sobon <[email protected]>
  • Loading branch information
mum4k committed Sep 17, 2024
1 parent 3402af1 commit 09e7fd9
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 28 deletions.
37 changes: 13 additions & 24 deletions .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -272,24 +272,23 @@ build:cache-local --remote_cache=grpc://localhost:9092
build:rbe-toolchain --action_env=BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=1

build:rbe-toolchain-clang --config=rbe-toolchain
build:rbe-toolchain-clang --platforms=@envoy_build_tools//toolchains:rbe_linux_clang_platform
build:rbe-toolchain-clang --host_platform=@envoy_build_tools//toolchains:rbe_linux_clang_platform
build:rbe-toolchain-clang --crosstool_top=@envoy_build_tools//toolchains/configs/linux/clang/cc:toolchain
build:rbe-toolchain-clang --extra_toolchains=@envoy_build_tools//toolchains/configs/linux/clang/config:cc-toolchain
build:rbe-toolchain-clang --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_clang_platform
build:rbe-toolchain-clang --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_clang_platform
build:rbe-toolchain-clang --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/clang/cc:toolchain
build:rbe-toolchain-clang --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/clang/config:cc-toolchain
build:rbe-toolchain-clang --action_env=CC=clang --action_env=CXX=clang++

build:rbe-toolchain-clang-libc++ --config=rbe-toolchain
build:rbe-toolchain-clang-libc++ --platforms=@envoy_build_tools//toolchains:rbe_linux_clang_libcxx_platform
build:rbe-toolchain-clang-libc++ --host_platform=@envoy_build_tools//toolchains:rbe_linux_clang_libcxx_platform
build:rbe-toolchain-clang-libc++ --crosstool_top=@envoy_build_tools//toolchains/configs/linux/clang_libcxx/cc:toolchain
build:rbe-toolchain-clang-libc++ --extra_toolchains=@envoy_build_tools//toolchains/configs/linux/clang_libcxx/config:cc-toolchain
build:rbe-toolchain-clang-libc++ --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_clang_libcxx_platform
build:rbe-toolchain-clang-libc++ --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_clang_libcxx_platform
build:rbe-toolchain-clang-libc++ --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/clang_libcxx/cc:toolchain
build:rbe-toolchain-clang-libc++ --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/clang_libcxx/config:cc-toolchain
build:rbe-toolchain-clang-libc++ --action_env=CC=clang --action_env=CXX=clang++
build:rbe-toolchain-clang-libc++ --action_env=CXXFLAGS=-stdlib=libc++
build:rbe-toolchain-clang-libc++ --action_env=LDFLAGS=-stdlib=libc++
build:rbe-toolchain-clang-libc++ --define force_libcpp=enabled

# Do not inherit from "clang-asan" to avoid picking up flags from local clang.bazelrc.
build:rbe-toolchain-asan --config=asan
build:rbe-toolchain-asan --config=clang-asan
build:rbe-toolchain-asan --linkopt -fuse-ld=lld
build:rbe-toolchain-asan --action_env=ENVOY_UBSAN_VPTR=1
build:rbe-toolchain-asan --copt=-fsanitize=vptr,function
Expand All @@ -307,20 +306,10 @@ build:rbe-toolchain-tsan --linkopt=-Wl,-rpath,/opt/libcxx_tsan/lib
build:rbe-toolchain-tsan --config=clang-tsan

build:rbe-toolchain-gcc --config=rbe-toolchain
build:rbe-toolchain-gcc --platforms=@envoy_build_tools//toolchains:rbe_linux_gcc_platform
build:rbe-toolchain-gcc --host_platform=@envoy_build_tools//toolchains:rbe_linux_gcc_platform
build:rbe-toolchain-gcc --crosstool_top=@envoy_build_tools//toolchains/configs/linux/gcc/cc:toolchain
build:rbe-toolchain-gcc --extra_toolchains=@envoy_build_tools//toolchains/configs/linux/gcc/config:cc-toolchain

build:rbe-toolchain-msvc-cl --host_platform=@envoy_build_tools//toolchains:rbe_windows_msvc_cl_platform
build:rbe-toolchain-msvc-cl --platforms=@envoy_build_tools//toolchains:rbe_windows_msvc_cl_platform
build:rbe-toolchain-msvc-cl --crosstool_top=@envoy_build_tools//toolchains/configs/windows/msvc-cl/cc:toolchain
build:rbe-toolchain-msvc-cl --extra_toolchains=@envoy_build_tools//toolchains/configs/windows/msvc-cl/config:cc-toolchain

build:rbe-toolchain-clang-cl --host_platform=@envoy_build_tools//toolchains:rbe_windows_clang_cl_platform
build:rbe-toolchain-clang-cl --platforms=@envoy_build_tools//toolchains:rbe_windows_clang_cl_platform
build:rbe-toolchain-clang-cl --crosstool_top=@envoy_build_tools//toolchains/configs/windows/clang-cl/cc:toolchain
build:rbe-toolchain-clang-cl --extra_toolchains=@envoy_build_tools//toolchains/configs/windows/clang-cl/config:cc-toolchain
build:rbe-toolchain-gcc --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_gcc_platform
build:rbe-toolchain-gcc --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_gcc_platform
build:rbe-toolchain-gcc --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/gcc/cc:toolchain
build:rbe-toolchain-gcc --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/gcc/config:cc-toolchain

build:remote --spawn_strategy=remote,sandboxed,local
build:remote --strategy=Javac=remote,sandboxed,local
Expand Down
4 changes: 2 additions & 2 deletions bazel/repositories.bzl
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")

ENVOY_COMMIT = "fbc6ee2ed5b858c842999c688504fd133008868a"
ENVOY_SHA = "5afbcfb875d41f7ecf7851b04ba43d1e1810bc1bc5a0da813cd82a0ee579ad2f"
ENVOY_COMMIT = "8cd214daaf858a5e698494a5a667f31978531c18"
ENVOY_SHA = "7d17c7aaa675d22abdb1cecb800e7a9a83c61c01d64639a62397570d32db868b"

HDR_HISTOGRAM_C_VERSION = "0.11.2" # October 12th, 2020
HDR_HISTOGRAM_C_SHA = "637f28b5f64de2e268131e4e34e6eef0b91cf5ff99167db447d9b2825eae6bad"
Expand Down
2 changes: 1 addition & 1 deletion source/common/request_source_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ void RemoteRequestSourceImpl::connectToRequestStreamGrpcService() {
absl::StatusOr<Envoy::Grpc::AsyncClientFactoryPtr> cluster_manager =
cluster_manager_->grpcAsyncClientManager().factoryForGrpcService(grpc_service, scope_,
/*skip_cluster_check=*/true);
THROW_IF_STATUS_NOT_OK(cluster_manager, throw);
THROW_IF_NOT_OK_REF(cluster_manager.status());
grpc_client_ = std::make_unique<RequestStreamGrpcClientImpl>(
(*cluster_manager)->createUncachedRawAsyncClient(), dispatcher_, *base_header_,
header_buffer_length_);
Expand Down
2 changes: 2 additions & 0 deletions tools/code_format/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -204,6 +204,7 @@ paths:
- source/common/protobuf/utility.cc
- source/common/protobuf/protobuf.h
- source/common/protobuf/yaml_utility.cc
- source/extensions/filters/http/proto_message_extraction/extraction_util/extraction_util.cc
- test/extensions/bootstrap/wasm/test_data/speed_cpp.cc

namespace_check:
Expand All @@ -225,6 +226,7 @@ paths:
- ci/prebuilt
- source/common/protobuf
- source/extensions/filters/http/grpc_field_extraction
- source/extensions/filters/http/proto_message_extraction/extraction_util
- test/extensions/bootstrap/wasm/test_data
- . # unique
# TODO(915): Remove this inclusion when all direct references removed # unique
Expand Down
1 change: 0 additions & 1 deletion tools/gen_compilation_database.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,6 @@ def fix_compilation_database(args, db):
help=
'Use `clang++` instead of the bazel wrapper for commands. This may help if `clangd` cannot find/run the tools.'
)
parser.add_argument('--bazel', default='bazel')
parser.add_argument(
'bazel_targets',
nargs='*',
Expand Down

0 comments on commit 09e7fd9

Please sign in to comment.