Skip to content
This repository has been archived by the owner on Feb 16, 2019. It is now read-only.

make tracing endpoint configuration straightforward #388

Open
visheyra opened this issue Jun 13, 2018 · 2 comments
Open

make tracing endpoint configuration straightforward #388

visheyra opened this issue Jun 13, 2018 · 2 comments

Comments

@visheyra
Copy link

visheyra commented Jun 13, 2018

Is this a BUG or FEATURE REQUEST?:

Feature Request:
Yes

Describe the feature:

It would be nice if one could simply configure the zipkin endpoint in the different component that needs it.

Use case

I want to change my jaeger instance namespace (let's say tracing instead of istio-system)

Current "working so far" alternatives

The good one

I have to do the following operations:

  1. Update the istio configuration in the istio configmap
  • kind: ConfigMap
  • name: istio
  • namespace: istio-system
  1. Update the mixer deployment and change a command line argument
  • Kind: Deployment
  • Name: istio-mixer
  • Namespace: istio-system
  • cli arg: trace-zipkin-url
  1. Update the ingress deployment and change a command line argument
  • Kind: Deployment
  • Name: istio-ingress
  • Namespace: istio-system
  • cli arg: zipkinAddress

The bad one

One could make a dirty fix by adding an externalName in the istio-system namespace and make it point to jaegertracing.<YOUR_NS>.svc.cluster.local. I think this dirty fix would not work with mTLS enabled.

thoughts

Since this configuration is not available in the helm chart, doing it manually with the above methods are the only that would works.

peers

@jsenon

@visheyra
Copy link
Author

made a fix for it here. If any can review

@visheyra
Copy link
Author

maybe I should reopen this issue in istio/istio ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants