-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Open
Labels
area/agentCilium agent related.Cilium agent related.area/proxyImpacts proxy components, including DNS, Kafka, Envoy and/or XDS servers.Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers.kind/cfpCilium Feature ProposalCilium Feature Proposalkind/community-reportThis was reported by a user in the Cilium community, eg via Slack.This was reported by a user in the Cilium community, eg via Slack.kind/featureThis introduces new functionality.This introduces new functionality.pinnedThese issues are not marked stale by our issue bot.These issues are not marked stale by our issue bot.roadmapThis functionality is planned for a future release of Cilium.This functionality is planned for a future release of Cilium.
Description
Proposal / RFE
It is known that agent upgrades will disrupt connectivity if L7 policy is in place. I'm wondering if there's an opportunity to improve this though, perhaps by leveraging the pre-flight check process and the SO_REUSEPORT
socket option.
Today, the preflight-check runs a dummy agent: perhaps, that agent could take over handling the traffic (HTTP, DNS, etc.), listening on the same port(s) as the "real" cilium-agent. An upgrade process could look like this:
- run the pre-flight check (and leave it running)
- upgrade/restart the cilium-agent
- un-install the pre-flight check
I haven't really looked at what this would mean from code perspective (cilium-agent, envoy etc), but in concept, it seems it may work and get rid of the downtime altogether, so I thought I'd throw this here and get some feedback.
jaydp17, splattner, Davee02, omallo and flblajoestringer
Metadata
Metadata
Assignees
Labels
area/agentCilium agent related.Cilium agent related.area/proxyImpacts proxy components, including DNS, Kafka, Envoy and/or XDS servers.Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers.kind/cfpCilium Feature ProposalCilium Feature Proposalkind/community-reportThis was reported by a user in the Cilium community, eg via Slack.This was reported by a user in the Cilium community, eg via Slack.kind/featureThis introduces new functionality.This introduces new functionality.pinnedThese issues are not marked stale by our issue bot.These issues are not marked stale by our issue bot.roadmapThis functionality is planned for a future release of Cilium.This functionality is planned for a future release of Cilium.