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

Added EUI Usage Analytics Script #8296

Merged
merged 7 commits into from
Jan 29, 2025
Merged

Conversation

JasonStoltz
Copy link
Member

@JasonStoltz JasonStoltz commented Jan 27, 2025

Summary

This PR adds a script used to collect EUI usage data from our primary product suite.

Screenshot 2025-01-27 at 5 00 51 PM

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:

  • Add Cloud UI to this script as another repo
  • Schedule a weekly job to run - This PR did some initial setup
  • Figure out a way to store the Elastic dashboard configuration here as well, so it is easier to reproduce elsewhere.

QA

Run the scripts according to the instructions README.

Log in to the Dashboard and verify the results.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@JasonStoltz JasonStoltz changed the title Scramble Added EUI Usage Analytics Script= Jan 28, 2025
@JasonStoltz JasonStoltz marked this pull request as ready for review January 28, 2025 21:00
@JasonStoltz JasonStoltz requested a review from a team as a code owner January 28, 2025 21:00
@JasonStoltz JasonStoltz changed the title Added EUI Usage Analytics Script= Added EUI Usage Analytics Script Jan 28, 2025
const escodegen = require('escodegen-wallaby');
const Codeowners = require('codeowners');

const codeowners = new Codeowners('../../../kibana');
Copy link
Member

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.

Copy link
Member

@tkajtoch tkajtoch left a 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

@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

Copy link
Member

@tkajtoch tkajtoch left a 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

@JasonStoltz JasonStoltz merged commit c558b32 into elastic:main Jan 29, 2025
5 checks passed
@JasonStoltz JasonStoltz deleted the scramble branch January 29, 2025 22:38
weronikaolejniczak pushed a commit to cee-chen/eui that referenced this pull request Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants