Skip to content
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

Add export metrics #2340

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Conversation

chancez
Copy link
Contributor

@chancez chancez commented Apr 17, 2024

This is useful to figure out how much data is being exported and if the log pipeline is behind in processing the incoming events.

@chancez chancez requested a review from a team as a code owner April 17, 2024 18:24
@chancez chancez requested a review from olsajiri April 17, 2024 18:24
cmd/tetragon/main.go Outdated Show resolved Hide resolved
@chancez chancez force-pushed the pr/chancez/export_metrics branch from 218c97d to f44fac4 Compare April 17, 2024 18:52
Copy link
Contributor

@olsajiri olsajiri left a comment

Choose a reason for hiding this comment

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

lgtm, left some comments

pkg/exporter/metrics.go Show resolved Hide resolved
pkg/exporter/exporter.go Show resolved Hide resolved
pkg/exporter/metrics.go Show resolved Hide resolved
pkg/exporter/metrics.go Show resolved Hide resolved
@olsajiri
Copy link
Contributor

cc @lambdanis

@chancez chancez self-assigned this Apr 23, 2024
@chancez chancez added the release-note/minor This PR introduces a minor user-visible change label Apr 23, 2024
this is useful to figure out how much data is being exported and if the log
pipeline is behind in processing the incoming events.

Signed-off-by: Chance Zibolski <[email protected]>
@chancez chancez force-pushed the pr/chancez/export_metrics branch from f44fac4 to 8f1671b Compare April 23, 2024 22:00
@chancez chancez requested a review from mtardy as a code owner April 23, 2024 22:00
Copy link

netlify bot commented Apr 23, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 8f1671b
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/66282f652ec32e0008b3de41
😎 Deploy Preview https://deploy-preview-2340--tetragon.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 site configuration.

@mtardy mtardy requested a review from lambdanis April 24, 2024 08:51
@lambdanis lambdanis added the area/metrics Related to prometheus metrics label Apr 25, 2024
@lambdanis lambdanis merged commit 92a0d58 into cilium:main Apr 25, 2024
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/metrics Related to prometheus metrics release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants