-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
Enable real-time metrics reporting #1368
Conversation
2b52d5a
to
311e29a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested with the file-sink only - this works great from a user's perspective.
This commit contains three changes: - It replaces underscores with dashes in vast.metrics.* options to be consistent with other options. - It removes the unused vast.metrics.enable option. Use vast.enable-metrics instead. - It adds a new options vast.metrics.real-time to enable real-time metrics reporting. This is disabled by default.
311e29a
to
f9bdb13
Compare
I rebased this onto recent master to fix the merge conflict in the changelog file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found a duplicate line, looks fine otherwise.
📔 Description
This PR contains three changes:
vast.metrics.*
options to be consistent with other options.vast.metrics.enable
option. Usevast.enable-metrics
instead.vast.metrics.{file,uds}-sink.real-time
to enable real-time metrics reporting. This is disabled by default.📝 Checklist
🎯 Review Instructions
For @tenzir/backend: Review code file-by-file. Please also review the related documentation PR.
For @0snap, who requested this feature: Run locally and see if this suits your needs. You can create a static binary for this branch on-demand with the new "Run workflow" button on the actions page for the
VAST Static
workflow. Edit: I've done this, you can find the run here.