-
Notifications
You must be signed in to change notification settings - Fork 244
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
Use odo with OpenShift Service Mesh (aka Istio) #2623
Comments
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
Can we keep this open? I think it's still important. |
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
/lifecycle frozen |
Yes, sorry about not following up on that. |
Being able to use annotations with odo would also be useful for dapr.io which also uses a sidecar. |
Devfile API was just updated to allow specifying annotations. (devfile/api#661) Before we can levarege this in odo we still need to wait for github.com/devfile/library to implement it. |
[kind/Feature]
Which functionality do you think we should add?
I think it'd be great to have the ability to use odo in conjunction with Istio/OpenShift Service Mesh. Service Mesh requires an annotation on the DeploymentConfigs created for each component. This is in order for the platform to inject a sidecar container with each pod. (spec/template/metadata/annotations of
sidecar.istio.io/inject: "true"
).See ref docs here:
https://docs.openshift.com/container-platform/4.3/service_mesh/service_mesh_day_two/prepare-to-deploy-applications-ossm.html#ossm-automatic-sidecar-injection_deploying-applications-ossm
Additionally, Services that get created should have port names that follow the naming listed here (for full support of routing and metrics):
https://istio.io/docs/ops/configuration/traffic-management/protocol-selection/
e.g.
Why is this needed?
To allow odo to be used in conjunction with OpenShift Service Mesh
The text was updated successfully, but these errors were encountered: