-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 Silk integration #10783
Add Silk integration #10783
Conversation
The |
1 similar comment
The |
The |
3 similar comments
The |
The |
The |
Codecov Report
Flags with carried forward coverage won't be shown. Click here to find out more. |
The |
/azp run PR changes |
Azure Pipelines successfully started running 1 pipeline(s). |
The |
11 similar comments
The |
The |
The |
The |
The |
The |
The |
The |
The |
The |
The |
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.
💯
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.
🎉
* New integration: Silk * Add e2e environment with mocked endpoints * Fix docker e2e env * Add configuration options * Add to CI * Add check logic * User server instead * Fix api format * Create Metrics class and parse metrics * Fix state * Comment * Add short desc * Update * Avoid redundant dictionary modification * Add additional metrics * Comment test out * Add additional system metrics and test * Add events * Make service checks more verbose, remove port and use host_address * Add metrics metadata * Fix url in test env, clean up sc * Document service checks * Update e2e mock responses * Add metrics to assert * Add e2e test * FIx style * Add event support for Silk (#10818) * refactor service check * Move test fixture to fixture dir * Clarify connect sc from state sc * Rename get_metrics to get_data * Support version metadata * Change from datetime to timestamp (#10877) * Add silk host tag to service checks * Add bs and rw breakdown * Support replication stats * Fix test * Fix metadata * Add units * Clean up * Refactor error msg and server state service checks * Refactor * Remove resolution metric * Fix tests * Fix test from get_data refactor * Fix format * clean up * Add new blocksize metrics * document block size metrics * Extract field to name * Add recommended monitor and fix rw volumes endpoint * Add config options for blocksize and read write stats * Update README * Move recommended monitors * Fix blocksize and readwrite config option * Update description for service check * Move blocksize and readwrite options up * Fix silk tests to account for blocksize and read/write metrics * Fix tests for CI * Update e2e tests * Import metrics * Fix CI * Validate config * Rename events.json fixture to fix CI * Update silk/README.md Co-authored-by: Austin Lai <76412946+alai97@users.noreply.github.com> * Apply suggestions * Remove empty silk dashboard until dashboard is ready * Add note about e2e testing * Apply suggestions * Apply suggestions * Sync validations * Add test for optional metrics * Fix test * Address comments * Add file * Add curated_metric section * Remove requirements.in * Address comments * Fix style * Address comments * Fix style * Address comments Co-authored-by: Andrew Zhang <31313038+yzhan289@users.noreply.github.com> Co-authored-by: Andrew Zhang <yzhan289@jhu.edu> Co-authored-by: Andrew Zhang <andrew.zhang@datadoghq.com> Co-authored-by: Austin Lai <76412946+alai97@users.noreply.github.com> bd217c5
What does this PR do?
This PR adds the Silk integration
Motivation
Additional Notes
Review checklist (to be filled by reviewers)
changelog/
andintegration/
labels attached