Skip to content

Consolidate checks for ad-hoc TUNNEL_PROTOCOL setting #25769

@rgo3

Description

@rgo3

We currently have multiple copies of the "special handling for ad-hoc tunnel" to create parameters for init.sh or set config options:

if !option.Config.TunnelingEnabled() &&
if !c.TunnelingEnabled() &&
and will add another one after completion of #24422 to write a correct value to node_config.h.

As a follow-up we should find a way to consolidate these checks in one place as explained by @julianwiedmann in this #24422 (comment)_

Metadata

Metadata

Labels

area/datapathImpacts bpf/ or low-level forwarding details, including map management and monitor messages.area/loaderImpacts the loading of BPF programs into the kernel.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions