Skip to content

Document OTEL empty upstream response metric#1751

Merged
wozniakjan merged 2 commits into
kedacore:mainfrom
aliaqel-stripe:add-otel-empty-upstream-response-metric
May 12, 2026
Merged

Document OTEL empty upstream response metric#1751
wozniakjan merged 2 commits into
kedacore:mainfrom
aliaqel-stripe:add-otel-empty-upstream-response-metric

Conversation

@aliaqel-stripe

@aliaqel-stripe aliaqel-stripe commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds documentation for the OpenTelemetry metric introduced by kedacore/keda#7671:

  • keda.scaler.empty.upstream.responses
  • documents that it is emitted when a scaler query returns an empty upstream result
  • documents the emitted attributes: namespace, scaledResource, triggerName, metricName, resourceType, and ignoreNullValues

Requested from the review thread on kedacore/keda#7671: kedacore/keda#7671 (comment)

@aliaqel-stripe aliaqel-stripe requested a review from a team as a code owner April 26, 2026 16:44
@netlify

netlify Bot commented Apr 26, 2026

Copy link
Copy Markdown

Deploy Preview for keda ready!

Name Link
🔨 Latest commit c683a81
🔍 Latest deploy log https://app.netlify.com/projects/keda/deploys/69ee41521035580008d8ef79
😎 Deploy Preview https://deploy-preview-1751--keda.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.

@github-actions

Copy link
Copy Markdown

Thank you for your contribution! 🙏 We will review your PR as soon as possible.

While you are waiting, make sure to:

  • Add your contribution to all applicable KEDA versions
  • GitHub checks are passing
  • Is the DCO check failing? Here is how you can fix DCO issues

Learn more about:

@wozniakjan wozniakjan merged commit 7d8e45a into kedacore:main May 12, 2026
9 checks passed
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.

3 participants