Skip to content

k8s: migrate CiliumNode and CiliumEndpoint watcher to Resource[T] #28159

New issue
New issue
Closed
Closed
k8s: migrate CiliumNode and CiliumEndpoint watcher to Resource[T]#28159
Assignees
pippolo84
Labels
area/k8sImpacts the kubernetes API, or kubernetes -> cilium internals translation layers.area/modularizationRelates to code modularization and maintenance.feature/egress-gatewayImpacts the egress IP gateway feature.kind/performanceThere is a performance impact of this.pinnedThese issues are not marked stale by our issue bot.
@lmb

Description

@lmb
lmb
opened on Sep 14, 2023

Once #28091 goes in we'll be pulling CiliumNode and CiliumEndpoint twice: once via the old k8s watchers and once via the new Resource[T] framework. This has an obvious performance impact.

Migrate the watchers in pkg/k8s/watchers/cilium_endpoint.go and pkg/k8s/watchers/cilium_node.go to use Resource[T] internally. There is a simple example for this in pkg/k8s/watchers/service.go.

Metadata

Metadata

Assignees

Labels

area/k8sImpacts the kubernetes API, or kubernetes -> cilium internals translation layers.area/modularizationRelates to code modularization and maintenance.feature/egress-gatewayImpacts the egress IP gateway feature.kind/performanceThere is a performance impact of this.pinnedThese issues are not marked stale by our issue bot.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions