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 option INCLUDE_PUBSUB_METRICS to manage verbose metrics #758

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andriusm
Copy link
Contributor

@andriusm andriusm commented Feb 10, 2025

This implements #757

Changes proposed

Added INCLUDE_PUBSUB_METRICS option to the server config so that pubsub data in metrics can be excluded.

Check List (Check all the applicable boxes)

  • I sign off on contributing this submission to open-source
  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Copy link

netlify bot commented Feb 10, 2025

Deploy Preview for opal-docs canceled.

Name Link
🔨 Latest commit 3ef4d1a
🔍 Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/67ab12bd16da00000843562f

@andriusm andriusm force-pushed the feature/include_pubsub_metrics branch from 9961250 to ed3d6e6 Compare February 10, 2025 10:50
@andriusm andriusm force-pushed the feature/include_pubsub_metrics branch 10 times, most recently from ab2ea33 to 6ecf10b Compare February 11, 2025 08:38
@andriusm andriusm force-pushed the feature/include_pubsub_metrics branch from 6ecf10b to 3ef4d1a Compare February 11, 2025 09:04
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.

1 participant