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

Remove prom scrape annotations from code-executor, which does not have metrics #156

Closed
wants to merge 1 commit into from

Conversation

erulabs
Copy link
Contributor

@erulabs erulabs commented Mar 4, 2024

This prevents error-spam from failures to scrape metrics.

Resolves #155

@samsternatretool
Copy link

@erulabs thank you for submitting this PR! Let me find the right person to review it.

@ryanartecona
Copy link
Contributor

ah yes, good find. this happened to be noticed independently and will be fixed (in the same way as here) in #154, which should be the next release to go out, soon. cc @avimoondra

@avimoondra
Copy link
Contributor

ah yes, good find. this happened to be noticed independently and will be fixed (in the same way as here) in #154, which should be the next release to go out, soon. cc @avimoondra

🎯 should be resolved in a day or two!

@samsternatretool
Copy link

Sounds like we can close this one, thanks @erulabs for the contribution though!

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.

Code executor does not expose metrics endpoint but does have metrics annotations
4 participants