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

Move kbn-alert-load tool into Kibana alerting #88389

Closed
mikecote opened this issue Jan 14, 2021 · 7 comments
Closed

Move kbn-alert-load tool into Kibana alerting #88389

mikecote opened this issue Jan 14, 2021 · 7 comments
Labels
estimate:small Small Estimated Level of Effort Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) technical debt Improvement of the software architecture and operational architecture

Comments

@mikecote
Copy link
Contributor

This issue is to move the kbn-alert-load tool into Kibana.

@mikecote mikecote added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Jan 14, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@pmuellr
Copy link
Member

pmuellr commented Jan 14, 2021

for whoever picks this up, I did a quick pass in Dec of moving the package into kibana, here - #84003 . I can't remember what all I had to change, but remember I had to switch out the meow CLI arg parser for whatever we use in Kibana (didn't want to add another dependency).

@gmmorris gmmorris added the loe:medium Medium Level of Effort label Jul 14, 2021
@gmmorris gmmorris added technical debt Improvement of the software architecture and operational architecture estimate:small Small Estimated Level of Effort labels Aug 13, 2021
@gmmorris gmmorris removed the loe:medium Medium Level of Effort label Sep 2, 2021
@mikecote
Copy link
Contributor Author

mikecote commented Dec 9, 2021

@kobelb mentioned that if the versioning of the kbn-alert-load tool isn't the same as Kibana, it's ok to have it in a separate repository. But, if we wanted to couple the tool and Kibana version together, it's ok to integrate with Kibana.

@mikecote
Copy link
Contributor Author

mikecote commented Dec 9, 2021

cc @ymao1 this issue may be something we want to do early in the performance work (#119845).

@ymao1
Copy link
Contributor

ymao1 commented Dec 9, 2021

I believe @YulNaumenko is working on this.

@EricDavisX
Copy link
Contributor

@EricDavisX tagging himself, may take this over as nobody is working it at this point afaik

@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
@mikecote
Copy link
Contributor Author

mikecote commented Jul 21, 2022

Closing as already completed 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
estimate:small Small Estimated Level of Effort Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants