-
Notifications
You must be signed in to change notification settings - Fork 7.8k
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
No services available through Jaeger on Istio 0.2.4 #936
Comments
We should repro w/o any Zipkin components - I wonder if there's something funky in Envoy that makes the swap from Zipkin to Jaegar at runtime break things. |
@redhotpenguin The issue is that the istio.yaml has been changed to use So just need to specify the namespace when deploying the all-in-one template:
as mentioned in istio/old_issues_repo#73. However this has shown up a different problem affecting zipkin instrumentation (whether viewed on zipkin or jaeger UI) - also reported by a user here: istio/old_issues_repo#72 This is caused because the way spans are classified in Envoy changed post 1.4.0, to use local configuration (i.e. OperationName: Egress/Ingress) to determine whether the span was client or server. @ZackButcher depending on when you wanted to release 0.2, you may want to revert to Envoy 1.4.0. Should I create a separate issue in this repo to outline the issue? btw - I had a look at the envoy config in the proxy container, and it had no listeners defined. Where does the config for these get added? |
@objectiser I don't think we're likely to roll back to Envoy's 1.4 release - we actually just moved forward to a more recent (post-1.4) Envoy SHA a few days ago to pick up some fixes. The listener config is provided by Pilot: here's the code, here's some testdata examples. |
@ZackButcher Thanks - yeah found it, and submitted istio/old_pilot_repo#1358 |
Awesome, thanks! |
expected to be fixed in 0.2.6, please confirm and close ? |
@objectiser @ZackButcher is this issue done? |
We have confirmation that Zipkin is working now, I have not had time to look at Jaeger. |
Yes, confirmed 0.2.6 works with Jaeger. |
Thanks! |
Automatic merge from submit-queue. [DO NOT MERGE] Auto PR to update dependencies of proxy This PR will be merged automatically once checks are successful. ```release-note none ```
…ute (istio#936) Co-authored-by: Julien DELACROIX <[email protected]>
…gocd.argoproj.io/instance` (istio#980) * Fix IOR wrongly copying argocd internal annotation from Gateway to route (istio#936) * Fix IOR wrongly copying argocd label from Gateway to route (istio#957) * OSSM-6295: Copy ArgoCD annotations from Gateway to Route Signed-off-by: Jacek Ewertowski <[email protected]> * OSSM-6295: Copy all annotations and labels added by ArgoCD except argocd.argoproj.io/instance Signed-off-by: Jacek Ewertowski <[email protected]> * Fix lint errors Signed-off-by: Jacek Ewertowski <[email protected]> --------- Co-authored-by: Julien DELACROIX <[email protected]> Co-authored-by: Jacek Ewertowski <[email protected]> Signed-off-by: Yann Liu <[email protected]>
After having setup Jaeger via, https://istio.io/docs/tasks/telemetry/distributed-tracing.html, the dashboard is accessible but the services select box only contains a
-
.Environment - macOS Sierra. Versions of relevant brew packages are:
jaeger-deployment pod is running:
Jaeger services are running
The text was updated successfully, but these errors were encountered: