Skip to content

Conversation

@nickolaev
Copy link
Contributor

This adds the bazel and make infrastructure to build release images with included symbols information.
It would be useful in production environments when debugging issues.

To leverage the build with CI, we move the core Release build functionality to build-envoy-images-release-base.yaml and then include it in the two new files:

  • build-envoy-images-release.yaml - invoked on push in the branch
  • build-envoy-images-release-debug.yaml - invoked on demand in the Actions UI

@nickolaev nickolaev requested a review from a team as a code owner December 3, 2025 09:32
@nickolaev nickolaev requested a review from mhofstetter December 3, 2025 09:32
@nickolaev
Copy link
Contributor Author

Should be merged before #1655

This woula allow calling `bazel build` with `--configure=release_debug`.
The resulting binary will retain all the symbols information.
It also is big, around 2GB.

Signed-off-by: Nikolay Nikolaev <[email protected]>
We're passing this to the Makefile and then also in the Dockerfile.
The result is calling the `bazel build --configure=release_debug`

Signed-off-by: Nikolay Nikolaev <[email protected]>
We move the common image release workflow to a shareadble YAML
and parametrize it. Then we include it in a release and
release-debug YAML files.

Signed-off-by: Nikolay Nikolaev <[email protected]>
@nickolaev nickolaev force-pushed the pr/nickolaev/debug_builds branch from e8b859c to e4e37ef Compare December 3, 2025 09:37
@mhofstetter mhofstetter requested a review from nezdolik December 3, 2025 14:47
@mhofstetter
Copy link
Member

@sayboras looks like Run integration tests on amd64 fails with Error when evaluating 'runs-on' for job 'tests'. .github/workflows/ci-tests.yaml (Line: 41, Col: 14): Unexpected value ''

IMO this is related to this change #1578 - seems like the variable is missing in forks.

do we need to change the trigger from pull_request to pull_request_target ?

Copy link
Member

@sayboras sayboras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sayboras sayboras merged commit 2648aa2 into cilium:main Dec 10, 2025
14 checks passed
@nezdolik
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants