Skip to content

Fix for broken FIPS builds#41226

Merged
phlax merged 2 commits into
envoyproxy:mainfrom
grnmeira:broken-fipss-build
Sep 25, 2025
Merged

Fix for broken FIPS builds#41226
phlax merged 2 commits into
envoyproxy:mainfrom
grnmeira:broken-fipss-build

linter fix

6b8db0f
Select commit
Loading
Failed to load commit list.
CI (Envoy) / Mobile/Android skipped Sep 25, 2025 in 0s

Check was skipped

This check was not triggered in this CI run

Details

Request (pr/41226/main@6b8db0f)

grnmeira @grnmeira 6b8db0f #41226 merge main@6ffb4f7

Fix for broken FIPS builds

This PR moved the Go dependency in FIPS builds to a Bazel http_archive. Once that was done, Bazel's filegroup doesn't seem to be dealing so well with non-UTF-8 characters present in a file name in Go's test suite, causing the build to fail.

It's not fully understood what circumstances cause Bazel to behave like that, but it seems reasonable to remove the test directory from the exports. Once a bug is filed to the Bazel project I'll update the PR with the issue number.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Environment

Request variables

Key Value
ref 416dd09
sha 6b8db0f
pr 41226
base-sha 6ffb4f7
actor grnmeira @grnmeira
message Fix for broken FIPS builds...
started 1758816265.1327
target-branch main
trusted false
Build image

Container image/s (as used in this CI run)

Key Value
default envoyproxy/envoy-build-ubuntu:f4a881a1205e8e6db1a57162faf3df7aed88eae8
mobile envoyproxy/envoy-build-ubuntu:mobile-f4a881a1205e8e6db1a57162faf3df7aed88eae8
Version

Envoy version (as used in this CI run)

Key Value
major 1
minor 36
patch 0
dev true