Skip to content

loader: migrate ENABLE_TPROXY to load time variable#44719

Closed
smagnani96 wants to merge 1 commit intopr/smagnani96/fix-l7lbfrom
pr/smagnani96/fix-l7lb-followup
Closed

loader: migrate ENABLE_TPROXY to load time variable#44719
smagnani96 wants to merge 1 commit intopr/smagnani96/fix-l7lbfrom
pr/smagnani96/fix-l7lb-followup

Conversation

@smagnani96
Copy link
Copy Markdown
Contributor

This commit migrates ENABLE_TPROXY macro to a previously introduced proxy_redirect_via_cilium_net load time variable.

Reference: #42646

@smagnani96 smagnani96 self-assigned this Mar 10, 2026
@smagnani96 smagnani96 added area/loader Impacts the loading of BPF programs into the kernel. area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/misc This PR makes changes that have no direct user impact. labels Mar 10, 2026
@smagnani96 smagnani96 force-pushed the pr/smagnani96/fix-l7lb branch from 473a9e6 to 19b1d6a Compare March 11, 2026 00:08
This commit migrates ENABLE_TPROXY macro to a previously introduced
`proxy_redirect_via_cilium_net` load time variable.

Reference: #42646

Signed-off-by: Simone Magnani <simone.magnani@isovalent.com>
@smagnani96 smagnani96 force-pushed the pr/smagnani96/fix-l7lb-followup branch from 9397b50 to b1063ad Compare March 11, 2026 00:09
@smagnani96 smagnani96 added the dont-merge/blocked Another PR must be merged before this one. label Mar 11, 2026
@julianwiedmann
Copy link
Copy Markdown
Member

Maybe rebase on-top of #44649 😁

@smagnani96
Copy link
Copy Markdown
Contributor Author

lol 😅

@smagnani96 smagnani96 closed this Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. area/loader Impacts the loading of BPF programs into the kernel. dont-merge/blocked Another PR must be merged before this one. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants