-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
loader: Don't compile .asm
files by default
#24769
loader: Don't compile .asm
files by default
#24769
Conversation
Today we always compile a .asm files for endpoints, even though we rarely use them. They take a lot of space in the sysdumps and increase the overall compile time. This commit changes it to only compile those files if debugging mode is enabled. Reported-by: Sebastian Wicki <sebastian@isovalent.com> Signed-off-by: Paul Chaignon <paul@cilium.io>
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The net-next job hit known flake #24687. |
k8s-1.26-kernel-net-next hit a timeout while downloading the Docker images as part of the VM provisioning. I'm guessing due to some transient connectivity issue. |
ConformanceEKS hit known CI breakage #24774. Other tests are passing. Marking ready to merge. |
Today, we always compile a
.asm
files for endpoints, even though we rarely use them. They take a lot of space in the sysdumps and increase the overall compile time.This pull request changes it to only compile those files if debugging mode is enabled.
Marking for backports to all branches because the change here is unlikely to conflict and should help with sysdump collection.