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

Task manager to become a system index in Elasticsearch? #58397

Closed
mikecote opened this issue Feb 24, 2020 · 9 comments
Closed

Task manager to become a system index in Elasticsearch? #58397

mikecote opened this issue Feb 24, 2020 · 9 comments
Labels
discuss Feature:Task Manager Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@mikecote
Copy link
Contributor

No description provided.

@mikecote mikecote added discuss Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Feb 24, 2020
@elasticmachine
Copy link
Contributor

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

@pmuellr
Copy link
Member

pmuellr commented Feb 25, 2020

So many opportunities here!

The biggest is that since ES has a notion of a cluster of nodes (unlike Kibana which doesn't know if there are other Kibana's running as peers), we can likely control the writes and perhaps reads to route through a single node, and avoid some of the concurrency issues we've dealt with, in the existing TM implementation.

I was also wondering if we could arrange some alternate coupling on the transport, between ES and Kibana, so we don't have to actually "poll" in Kibana. Either use websockets, or some long-poll story, or make-it-slow async returns, or even have ES send requests to Kibana that a task needs to be run (this last one seems fragile, but kinda fun).

@pmuellr
Copy link
Member

pmuellr commented Jun 4, 2021

@mikecote Should we re-prioritize this? Do we even have a choice given saved objects are going to eventually become system indices?

@mikecote
Copy link
Contributor Author

mikecote commented Jun 4, 2021

@pmuellr good question! @elastic/kibana-core regarding saved object indices, is the core team planning to handle the conversion for .kibana_task_manager, or is there some work necessary from the @elastic/kibana-alerting-services team to complete?

@mikecote
Copy link
Contributor Author

mikecote commented Jun 4, 2021

I added this issue to the alerting project board.

@pgayvallet
Copy link
Contributor

regarding saved object indices, is the core team planning to handle the conversion for .kibana_task_manager

Sorry, but conversion of what exactly? Isn't .kibana_task_manager de facto already a system index by following the .kibana* pattern?

@mikecote
Copy link
Contributor Author

mikecote commented Jun 8, 2021

regarding saved object indices, is the core team planning to handle the conversion for .kibana_task_manager

Sorry, but conversion of what exactly? Isn't .kibana_task_manager de facto already a system index by following the .kibana* pattern?

With the system indices project, from what I understand, we need to convert Kibana owned indices to either system or hidden by 8.0? I'm wondering if it's the core team or alerting team who will convert the task manager index. It's a saved object index. See #81536 for more details.

@pgayvallet
Copy link
Contributor

pgayvallet commented Jun 8, 2021

we need to convert Kibana owned indices to either system or hidden by 8.0?

Nothing to be done on our side, it was done in ES in https://github.com/elastic/elasticsearch/pull/68919/files#diff-ae6fc8fbb1c04f7efb1c5672ec0bb072f796b2d5968a2a34783d471a466aa099L57

.kibana_task_manager matches the .kibana_* pattern, so it's already an external system index. The associated restrictions will only apply on 8.0+ though

@mikecote
Copy link
Contributor Author

mikecote commented Jun 9, 2021

@pgayvallet awesome, thank you! I will close this issue 🙏 8.0+ aligns with my thinking as well

@mikecote mikecote closed this as completed Jun 9, 2021
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Feature:Task Manager Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

No branches or pull requests

5 participants