-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[Blog post] Introduce new OTel features in Envoy and Istio #4553
[Blog post] Introduce new OTel features in Envoy and Istio #4553
Conversation
joaopgrassi
commented
May 28, 2024
•
edited by chalin
Loading
edited by chalin
- Fixes New Blog Post: Observability on the Edge: Enhancing OTel support in Envoy and Istio #4534
- I also added Envoy and Istio to the integrations list.
- Preview: https://deploy-preview-4553--opentelemetry.netlify.app/blog/2024/new-otel-features-envoy-istio/
CC @jsuereth |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few small copy edit suggestions. The docs maintainers might have more to say. Thanks!
Co-authored-by: Tiffany Hrabusa <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
I merged |
We updated some of our tooling just yesterday, might be the case that something slipped here, cc @chalin |
We just published another blog post, so let's give it another 2 days and then we can merge this one in. |
Co-authored-by: Patrice Chalin <[email protected]>
Ah I see, it was because I didn't |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very well written post @joaopgrassi! 🙌🏻 I enjoyed reading it.
I made another copyedit pass, see inline comments.
Co-authored-by: Patrice Chalin <[email protected]>
/fix:refcache |
You triggered fix:refcache action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/9383659493 |
fix:refcache run failed, please check https://github.com/open-telemetry/opentelemetry.io/actions/runs/9383659493 for details |
Not sure why it's failing. /cc @svrnm |
Seems to have been fixed, I did it manually. :) |
Ah, it seems to have failed because you pushed a commit before it was able to. I'm glad that it's all fixed. @svrnm - this seems ready to merge. If it lands today, we'll need to adjust the date. |
Thanks for all the help everyone! |