Skip to content

docs: add note about Tempo appProtocol in proxy-trace.md - #5963

Open
rtomadpg wants to merge 3 commits into
envoyproxy:mainfrom
rtomadpg:patch-1
Open

docs: add note about Tempo appProtocol in proxy-trace.md#5963
rtomadpg wants to merge 3 commits into
envoyproxy:mainfrom
rtomadpg:patch-1

Conversation

@rtomadpg

@rtomadpg rtomadpg commented May 8, 2025

Copy link
Copy Markdown

Resolves #5887

Signed-off-by: Renzo Tomà @ DPG <151731240+rtomadpg@users.noreply.github.com>
Comment thread site/content/en/latest/tasks/observability/proxy-trace.md Outdated
@codecov

codecov Bot commented May 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.83%. Comparing base (2bbbdf8) to head (1927037).
⚠️ Report is 1248 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5963      +/-   ##
==========================================
+ Coverage   65.80%   65.83%   +0.02%     
==========================================
  Files         217      217              
  Lines       36013    36013              
==========================================
+ Hits        23699    23708       +9     
+ Misses      10834    10829       -5     
+ Partials     1480     1476       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Renzo Tomà @ DPG <151731240+rtomadpg@users.noreply.github.com>
zirain
zirain previously approved these changes May 8, 2025

@zirain zirain left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

shawnh2
shawnh2 previously approved these changes May 8, 2025
```yaml
distributor:
appProtocol:
grpc: grpc

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of grpc in the key, can we use kubernetes.io/h2c instead ?
this is what upstream would ideally like
https://gateway-api.sigs.k8s.io/guides/backend-protocol/

@arkodg

arkodg commented May 8, 2025

Copy link
Copy Markdown
Contributor

thanks @rtomadpg, added one comment, can you also copy over the same content into v1.3/tasks/observability/proxy-trace.md

@github-actions

github-actions Bot commented Jun 8, 2025

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions Bot added the stale label Jun 8, 2025
@acelinkio

Copy link
Copy Markdown

I'm coming here from using an otelcollector and not seeing really how to configure gatewayapi to send via grpc. i see commends around appProtocol.

i'll continue looking around, jsut want to mention its not readily apparent

@github-actions github-actions Bot removed the stale label Mar 7, 2026
Signed-off-by: Isaac Wilson <isaac.wilson514@gmail.com>
@jukie
jukie dismissed stale reviews from shawnh2 and zirain via 8afd879 April 23, 2026 07:00
@netlify

netlify Bot commented Apr 23, 2026

Copy link
Copy Markdown

Deploy Preview for cerulean-figolla-1f9435 ready!

Name Link
🔨 Latest commit 8afd879
🔍 Latest deploy log https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/69e9c3ad31266e00080cd991
😎 Deploy Preview https://deploy-preview-5963--cerulean-figolla-1f9435.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jukie
jukie requested review from a team, arkodg, shawnh2 and zirain April 23, 2026 07:01
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions Bot added the stale label Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenTelemetry tracing to Tempo fails using backendRef, works using host/port (HTTP used instead of GRPC?)

6 participants