Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

servicemesh: proxylib image for the separated cilium-envoy image #25226

Closed
jrajahalme opened this issue May 2, 2023 · 2 comments
Closed

servicemesh: proxylib image for the separated cilium-envoy image #25226

jrajahalme opened this issue May 2, 2023 · 2 comments
Labels
area/servicemesh GH issues or PRs regarding servicemesh sig/agent Cilium agent related. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.

Comments

@jrajahalme
Copy link
Member

While #25081 allows cilium-envoy to run as a separate container, we still have to figure out how/if to pair a proxylib image (libcilium.so) with the external container. Proxylib uses a our stable Go extensions for Envoy API so that a Cilium agent build can add additional L7 parsers to Envoy without re-building the Envoy image. Cilium agent can then build control plane (e.g., policy) support for a given L7 protocol and ship an implementation of it as libcilium.so and configure Cilium Envoy filters to use it. These proxylib images are not useful by themselves, outside of them being used with Cilium. While we were shipping cilium-envoy within the Cilium agent image there was no problem with this, but now that we start supporting a separate cilium-envoy container image we need to figure out how to ship proxylib for it.

@youngnick youngnick added sig/agent Cilium agent related. area/servicemesh GH issues or PRs regarding servicemesh labels May 8, 2023
@github-actions
Copy link

github-actions bot commented Jul 8, 2023

This issue has been automatically marked as stale because it has not
had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Jul 8, 2023
@mhofstetter
Copy link
Member

Fixed with cilium/proxy#232 & #26101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/servicemesh GH issues or PRs regarding servicemesh sig/agent Cilium agent related. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Projects
None yet
Development

No branches or pull requests

3 participants