envoy: Bump envoy version to v1.39.1 - #1998
Open
sayboras wants to merge 8 commits into
Open
Conversation
sayboras
force-pushed
the
pr/tammach/envoy-1.39
branch
2 times, most recently
from
August 17, 2026 12:02
0cbd6a5 to
9169629
Compare
sayboras
force-pushed
the
pr/tammach/envoy-1.39
branch
2 times, most recently
from
August 29, 2026 05:15
7274f6c to
ce354be
Compare
sayboras
changed the title
sayboras
force-pushed
the
pr/tammach/envoy-1.39
branch
2 times, most recently
from
August 30, 2026 15:01
46c0cfb to
820cf2e
Compare
Sync up the following files with upstream Envoy v1.39.1: - `.bazelversion` (Bazel 7.7.1 -> 8.7.0) - `envoy.bazelrc` - `envoy_build_config/extensions_build_config.bzl` Adjust patch 0003 for the upstream backport in original_dst_cluster.cc that replaced copyInternetAddressAndPort(*dst_ip) with getAddressWithPort(dst_addr, dst_ip->port()). Signed-off-by: Tam Mach <tam.mach@cilium.io>
Since Bazel 8, `git_repository` carries a `patch_strip` attribute, and `bazel_tools`' `patch()` helper unconditionally prepends the corresponding `-pN` to `patch_args`. With `patch_tool = "git"` and `patch_args = ["apply"]` that produces `git -p0 apply < patchfile`, and `git` rejects `-p0` as an unknown global option. Signed-off-by: Tam Mach <tam.mach@cilium.io>
Envoy 1.39 brings a number of API changes that the Cilium filters and their tests need to follow: - `check_format` now bans `absl::optional`, `absl::nullopt` and `absl::make_optional` in favour of the `std` equivalents. - `FilterState::StateType` is gone and `setData()` lost its `state_type` parameter. - `Http::UpstreamCallbacks` gained a pure `onHostSelected()`. Cilium enforces upstream host authorization in the network filter, so this is a no-op here. - `TransportSocket::closeSocket()` takes an `abort_reset` flag, which `SslSocketWrapper` forwards so that policy denials still reset the connection. - Protobuf 35 marks `SerializeToString()` and `Any::PackFrom()` as `[[nodiscard]]`. Failing to serialize an access log entry or a health check event is now logged and the message dropped instead of being sent truncated. - `TestUtility::waitForCounterEq()`/`Ge()` and the gauge equivalents were removed in favour of the matcher based `waitForCounter()`/`waitForGauge()`. - Buffer watermark callbacks are `absl::AnyInvocable<void()>` and therefore move-only. - Extension lookup by name only is gone, so the router filter needs an explicit `typed_config`. - spdlog is built with `SPDLOG_USE_STD_FORMAT`, so `ENVOY_LOG()` arguments have to be `std::format`-able. Turn the `RuleVerdict` formatter into a `std::formatter` and replace `fmt::join()` with `absl::StrJoin()`. - `cilium_network_policy_test` needs an explicit dependency on the upstream secret mocks, which it used to pick up transitively. Signed-off-by: Tam Mach <tam.mach@cilium.io>
API update for Envoy 1.39, which bumps protobuf from 33.2 to 35.1. Only changes comments in generated files. Signed-off-by: Tam Mach <tam.mach@cilium.io>
`main` is on Envoy 1.39 now, so widen its `allowedVersions` accordingly. Otherwise Renovate keeps the constraint at `<=1.38` and stops proposing 1.39.x patch releases for `main`. Signed-off-by: Tam Mach <tam.mach@cilium.io>
Protobuf bundled with Envoy 1.39 marks Any::PackFrom as [[nodiscard]], so the unchecked call in cilium_network_policy_benchmark.cc fails under -Werror=unused-result and breaks the integration test build. Assign the result to std::ignore, matching the other test files.
Bazel 8 turns @bazel_tools//tools/cpp/runfiles into an alias for @rules_cc//cc/runfiles, so the per_file_copt regex keyed on the old bazel_tools path no longer matched anything. Without it, MSAN flags the stat() buffer in Runfiles::Create and every //tests:* binary aborts before running a single test. Signed-off-by: Tam Mach <tam.mach@cilium.io>
sayboras
force-pushed
the
pr/tammach/envoy-1.39
branch
from
August 30, 2026 15:10
820cf2e to
d8f8c62
Compare
Signed-off-by: Tam Mach <tam.mach@cilium.io>
sayboras
force-pushed
the
pr/tammach/envoy-1.39
branch
from
August 31, 2026 00:00
77f9e98 to
0b37004
Compare
sayboras
marked this pull request as ready for review
September 2, 2026 12:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.