Skip to content

Conversation

@JasonStoltz
Copy link
Member

Summary

There are two parts to this PR:

Refactoring (7617a16)

  • Added a single unit test to aid refactoring.
  • Added an override for kibana root, which we'll need for CI.
  • Added a dry run option.

Added support for Cloud UI (924d957)

QA

Remove or strikethrough items that do not apply to your PR.

- Added a unit test to aid refaxtoring.
- Added an override for kibana root, which we'll need for CI
- Added a dry run option and removed test script
@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💔 Build Failed

Failed CI Steps

@JasonStoltz JasonStoltz changed the title Scramble Update EUI Analytics Scripts Feb 4, 2025
@github-actions
Copy link

github-actions bot commented May 6, 2025

👋 Hey there. This PR hasn't had any activity for 90 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

@github-actions github-actions bot added the stale-pr (Don't delete - used for automation) label May 6, 2025
@JasonStoltz
Copy link
Member Author

Oof, really need to get back to this one.

@github-actions github-actions bot removed the stale-pr (Don't delete - used for automation) label May 10, 2025
@weronikaolejniczak
Copy link
Contributor

@JasonStoltz what is there left to do? Can I help somehow?

@JasonStoltz
Copy link
Member Author

@weronikaolejniczak The main thing:

It tries to ship like 120k records to Elasticsearch, which is 429ing, so I'm getting rate limited. Just need to configure the bulk API for the javascript client to do some incremental backoff and retries, etc.

@JasonStoltz
Copy link
Member Author

Also needed a follow-up PR, is actually getting this running weekly as a job in CI.

@JasonStoltz
Copy link
Member Author

And I'm not sure when I'll get to this, so if someone has time it is definitely something we could benefit from as a team, I believe.

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.

4 participants