Skip to content

deps: Deprecate/disable opentracing by default#32421

Merged
phlax merged 2 commits intoenvoyproxy:mainfrom
phlax:deprecate-opentracing-default
Feb 21, 2024
Merged

deps: Deprecate/disable opentracing by default#32421
phlax merged 2 commits intoenvoyproxy:mainfrom
phlax:deprecate-opentracing-default

Conversation

@phlax
Copy link
Member

@phlax phlax commented Feb 15, 2024

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @lizan
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #32421 was opened by phlax.

see: more, trace.

Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the deprecate-opentracing-default branch from c3c9f16 to caa9624 Compare February 15, 2024 14:25
lizan
lizan previously approved these changes Feb 15, 2024
@phlax
Copy link
Member Author

phlax commented Feb 16, 2024

this requires updates to jaeger example/s from #28851

Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax
Copy link
Member Author

phlax commented Feb 16, 2024

/retest

@phlax phlax merged commit 1ad862d into envoyproxy:main Feb 21, 2024
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.

2 participants