Skip to content

WIP build: disable inlining for nodeup builds#18265

Open
hakman wants to merge 3 commits intokubernetes:masterfrom
hakman:nodeup-disable-inlining
Open

WIP build: disable inlining for nodeup builds#18265
hakman wants to merge 3 commits intokubernetes:masterfrom
hakman:nodeup-disable-inlining

Conversation

@hakman
Copy link
Copy Markdown
Member

@hakman hakman commented May 3, 2026

Apply -gcflags=all=-l only to nodeup release builds to reduce the nodeup binary size without changing compiler flags for kops, protokube, or channels.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 3, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zetaab for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot requested review from olemarkus and zetaab May 3, 2026 06:30
@hakman hakman changed the title build: disable inlining for nodeup builds WIP build: disable inlining for nodeup builds May 3, 2026
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 3, 2026
Apply -gcflags=all=-l only to nodeup release builds to reduce the
nodeup binary size without changing compiler flags for kops, protokube,
or channels.

Signed-off-by: Ciprian Hacman <ciprian@hakman.dev>
@hakman hakman force-pushed the nodeup-disable-inlining branch from cc3a500 to e9acd03 Compare May 3, 2026 09:41
Signed-off-by: Ciprian Hacman <ciprian@hakman.dev>
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. area/nodeup and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 3, 2026
Signed-off-by: Ciprian Hacman <ciprian@hakman.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/nodeup cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants