-
Notifications
You must be signed in to change notification settings - Fork 842
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
Added EUI Usage Analytics Script #8296
Conversation
const escodegen = require('escodegen-wallaby'); | ||
const Codeowners = require('codeowners'); | ||
|
||
const codeowners = new Codeowners('../../../kibana'); |
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.
[Not a blocker] This should eventually be configurable to make our lives easier when automating this job. Our agent configuration needs specific paths to be used that are mounted to the right volumes for performance reasons.
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.
Overall, the changes look solid! 🎉 I added a few minor comments, but they're not critical by any means
Preview staging links for this PR:
|
💚 Build Succeeded
History
|
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.
Thanks for addressing my comments! LGTM
Co-authored-by: Tomasz Kajtoch <[email protected]>
Summary
This PR adds a script used to collect EUI usage data from our primary product suite.
The intent it to run this script automatically in a weekly job. For now, it would need to be run manually from someone's workstation.
Things to consider doing in the future:
repo
QA
Run the scripts according to the instructions README.
Log in to the Dashboard and verify the results.