Skip to content

add zipkin server address to service mesh config #270

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

Merged
merged 3 commits into from
May 15, 2017
Merged

Conversation

rshriram
Copy link
Member

@rshriram rshriram commented May 14, 2017

This PR enables Zipkin tracing in the service mesh (enables it by default). In future, we can look for ways to dynamically enable/disable this [requires manager to watch for updates to config maps].


This change is Reviewable

@rshriram rshriram requested a review from kyessenov May 14, 2017 15:52
@rshriram
Copy link
Member Author

jenkins rebuild istio/presubmit

1 similar comment
@rshriram
Copy link
Member Author

jenkins rebuild istio/presubmit

@istio-testing
Copy link
Collaborator

Jenkins job istio/presubmit passed

@sebastienvas sebastienvas merged commit 564d8c9 into master May 15, 2017
@sebastienvas sebastienvas deleted the enable_zipkin branch May 15, 2017 03:45
rshriram added a commit that referenced this pull request Oct 30, 2017
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
Automatic merge from submit-queue.

Fix systemd issue

For now, without putting generic named binaries in user’s /usr/local/bin

Replacement for #268 

```release-note
Standalone node agent path fix (debian)
```
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
* More coverage for config gen

* rename sds-version

* restructure kube tests

* Use different header values
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
Automatic merge from submit-queue.

Fix systemd issue

For now, without putting generic named binaries in user’s /usr/local/bin

Replacement for #268 

```release-note
Standalone node agent path fix (debian)
```
howardjohn pushed a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
This is a general cleanup of the readme. Fixes:
* Small grammatical errors
* Lots of adding backticks because my eyes hurt otherwise
* Some sections were unclear, particularly the bits around how the
'environment' name is determined
* Some sections weren't really 'noob friendly' eg. the references to
'kube-inject' instead of `istioctl kube-inject`
howardjohn pushed a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
luksa pushed a commit to luksa/istio that referenced this pull request Apr 26, 2021
MAISTRA-2051: Add extended TLS options
howardjohn added a commit that referenced this pull request Sep 7, 2022
* hbone: add `hboneInsecureSkipVerify` to CLI (#40535)

* hbone: add `hboneInsecureSkipVerify` to CLI

This matches the proto. Its helpful for local testing since spiffe
certificate verification is typically not happening

* lint

* Add tests with direct HBONE clients

* fix for new improvements
aniketsingh03 pushed a commit to aniketsingh03/istio-1 that referenced this pull request Oct 4, 2022
Signed-off-by: Yaroslav Skopets <yaroslav@tetrate.io>
antonioberben pushed a commit to antonioberben/istio that referenced this pull request Jan 29, 2024
…1-24

[jaeger-operator] Update version jaeger-operator to v1.24.0
fjglira pushed a commit to fjglira/istio that referenced this pull request Feb 10, 2025
…ter-merge_upstream_istio_master-6253864e

Automator: merge upstream changes to openshift-service-mesh/istio@master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants